openstack/swift
Code Issues Proposed changes

py3: Fix expirer container generation

Browse Source 
Closes-Bug: #1883561
Change-Id: Idbc13b90a565c4e64cef9035dbcd60872576727a
This commit is contained in:
Tim Burke 2020-06-11 15:37:56 -07:00 committed by Tim Burke
parent 3c0a448f06
commit 9bc8c82587
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
swift/common/utils.py
View File

@ -4402,7 +4402,7 @@ def get_expirer_container(x_delete_at, expirer_divisor, acc, cont, obj):
"""
shard_int = int(hash_path(acc, cont, obj), 16) % 100
return normalize_delete_at_timestamp(
int(x_delete_at) / expirer_divisor * expirer_divisor - shard_int)
int(x_delete_at) // expirer_divisor * expirer_divisor - shard_int)
class _MultipartMimeFileLikeObject(object):

10
test/unit/common/test_utils.py
View File

@ -6607,6 +6607,16 @@ class TestParseContentDisposition(unittest.TestCase):
self.assertEqual(attrs, {'name': 'somefile', 'filename': 'test.html'})
class TestGetExpirerContainer(unittest.TestCase):
@mock.patch.object(utils, 'hash_path', return_value=hex(101)[2:])
def test_get_expirer_container(self, mock_hash_path):
container = utils.get_expirer_container(1234, 20, 'a', 'c', 'o')
self.assertEqual(container, '0000001219')
container = utils.get_expirer_container(1234, 200, 'a', 'c', 'o')
self.assertEqual(container, '0000001199')
class TestIterMultipartMimeDocuments(unittest.TestCase):
def test_bad_start(self):