openstack/swift
Code Issues Proposed changes

Merge "Add tests for X-Backend-Clean-Expiring-Object-Queue true"

Browse Source
This commit is contained in:
Zuul 2018-01-17 05:30:45 +00:00 committed by Gerrit Code Review
commit 7227d9967b
1 changed files with 89 additions and 47 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

72
test/unit/obj/test_server.py
View File

@ -5427,27 +5427,31 @@ class TestObjectController(unittest.TestCase):
self.assertTrue('chost,badhost' in msg) self.assertTrue('chost,badhost' in msg)
self.assertTrue('cdevice' in msg) self.assertTrue('cdevice' in msg)
def test_delete_at_update_on_put(self): def test_delete_at_update_on_put_and_post(self):
# Test how delete_at_update works when issued a delete for old # Test how delete_at_update works when issued a delete for old
# expiration info after a new put with no new expiration info. # expiration info after a new put/post with no new expiration info.
policy = random.choice(list(POLICIES)) policy = random.choice(list(POLICIES))
def do_test(method, headers, expected_args):
given_args = [] given_args = []
def fake_async_update(*args): def fake_async_update(*args):
given_args.extend(args) given_args.extend(args)
req = Request.blank( headers.update({'X-Timestamp': 1,
'/v1/a/c/o',
environ={'REQUEST_METHOD': 'PUT'},
headers={'X-Timestamp': 1,
'X-Trans-Id': '123', 'X-Trans-Id': '123',
'X-Backend-Storage-Policy-Index': int(policy)}) 'X-Backend-Storage-Policy-Index': int(policy)})
req = Request.blank(
'/v1/a/c/o',
environ={'REQUEST_METHOD': method},
headers=headers)
with mock.patch.object(self.object_controller, 'async_update', with mock.patch.object(self.object_controller, 'async_update',
fake_async_update): fake_async_update):
self.object_controller.delete_at_update( self.object_controller.delete_at_update(
'DELETE', 2, 'a', 'c', 'o', req, 'sda1', policy) 'DELETE', 2, 'a', 'c', 'o', req, 'sda1', policy)
self.assertEqual( self.assertEqual(expected_args, given_args)
given_args, [
expected_args = [
'DELETE', '.expiring_objects', '0000000000', 'DELETE', '.expiring_objects', '0000000000',
'0000000002-a/c/o', None, None, None, '0000000002-a/c/o', None, None, None,
HeaderKeyDict({ HeaderKeyDict({
@ -5455,7 +5459,33 @@ class TestObjectController(unittest.TestCase):
'x-timestamp': utils.Timestamp('1').internal, 'x-timestamp': utils.Timestamp('1').internal,
'x-trans-id': '123', 'x-trans-id': '123',
'referer': 'PUT http://localhost/v1/a/c/o'}), 'referer': 'PUT http://localhost/v1/a/c/o'}),
'sda1', policy]) 'sda1', policy]
do_test('PUT', {}, expected_args)
do_test('PUT', {'X-Backend-Clean-Expiring-Object-Queue': 'true'},
expected_args)
do_test('PUT', {'X-Backend-Clean-Expiring-Object-Queue': 't'},
expected_args)
expected_args = [
'DELETE', '.expiring_objects', '0000000000',
'0000000002-a/c/o', None, None, None,
HeaderKeyDict({
'X-Backend-Storage-Policy-Index': 0,
'x-timestamp': utils.Timestamp('1').internal,
'x-trans-id': '123',
'referer': 'POST http://localhost/v1/a/c/o'}),
'sda1', policy]
do_test('POST', {}, expected_args)
do_test('POST', {'X-Backend-Clean-Expiring-Object-Queue': 'true'},
expected_args)
do_test('POST', {'X-Backend-Clean-Expiring-Object-Queue': 't'},
expected_args)
do_test('PUT', {'X-Backend-Clean-Expiring-Object-Queue': 'false'}, [])
do_test('PUT', {'X-Backend-Clean-Expiring-Object-Queue': 'f'}, [])
do_test('POST', {'X-Backend-Clean-Expiring-Object-Queue': 'false'}, [])
do_test('POST', {'X-Backend-Clean-Expiring-Object-Queue': 'f'}, [])
def test_delete_at_negative(self): def test_delete_at_negative(self):
# Test how delete_at_update works when issued a delete for old # Test how delete_at_update works when issued a delete for old
@ -5590,22 +5620,27 @@ class TestObjectController(unittest.TestCase):
def test_delete_at_update_delete(self): def test_delete_at_update_delete(self):
policy = random.choice(list(POLICIES)) policy = random.choice(list(POLICIES))
def do_test(headers, expected_args):
given_args = [] given_args = []
def fake_async_update(*args): def fake_async_update(*args):
given_args.extend(args) given_args.extend(args)
self.object_controller.async_update = fake_async_update self.object_controller.async_update = fake_async_update
headers.update({'X-Timestamp': 1,
'X-Trans-Id': '1234',
'X-Backend-Storage-Policy-Index': int(policy)})
req = Request.blank( req = Request.blank(
'/v1/a/c/o', '/v1/a/c/o',
environ={'REQUEST_METHOD': 'DELETE'}, environ={'REQUEST_METHOD': 'DELETE'},
headers={'X-Timestamp': 1, headers=headers)
'X-Trans-Id': '1234',
'X-Backend-Storage-Policy-Index': int(policy)})
self.object_controller.delete_at_update('DELETE', 2, 'a', 'c', 'o', self.object_controller.delete_at_update('DELETE', 2, 'a', 'c', 'o',
req, 'sda1', policy) req, 'sda1', policy)
self.assertEqual( self.assertEqual(expected_args, given_args)
given_args, [
expected_args = [
'DELETE', '.expiring_objects', '0000000000', 'DELETE', '.expiring_objects', '0000000000',
'0000000002-a/c/o', None, None, '0000000002-a/c/o', None, None,
None, HeaderKeyDict({ None, HeaderKeyDict({
@ -5613,7 +5648,14 @@ class TestObjectController(unittest.TestCase):
'x-timestamp': utils.Timestamp('1').internal, 'x-timestamp': utils.Timestamp('1').internal,
'x-trans-id': '1234', 'x-trans-id': '1234',
'referer': 'DELETE http://localhost/v1/a/c/o'}), 'referer': 'DELETE http://localhost/v1/a/c/o'}),
'sda1', policy]) 'sda1', policy]
do_test({}, expected_args)
do_test({'X-Backend-Clean-Expiring-Object-Queue': 'true'},
expected_args)
do_test({'X-Backend-Clean-Expiring-Object-Queue': 't'}, expected_args)
do_test({'X-Backend-Clean-Expiring-Object-Queue': 'false'}, [])
do_test({'X-Backend-Clean-Expiring-Object-Queue': 'f'}, [])
def test_delete_backend_replication(self): def test_delete_backend_replication(self):
# If X-Backend-Replication: True delete_at_update should completely # If X-Backend-Replication: True delete_at_update should completely