openstack/swift
Code Issues Proposed changes

Removing commented out code in test/unit/account/test_backend.py

Browse Source 
Noticed this while reviewing another change. Looks like the test itself already
ensures correct functionality of the reclaim() method in AccountBroker without
the commented code, thus removing this stale code.

Change-Id: I6a26a7591adef9fd794ca68a4e9c493d1127f93c
This commit is contained in:
Christian Schwede 2015-07-22 10:43:17 +00:00
parent cd7b2db550
commit a65e9db875
1 changed files with 0 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
test/unit/account/test_backend.py
View File

@ -160,23 +160,9 @@ class TestAccountBroker(unittest.TestCase):
broker.put_container('y', 0, 0, 0, 0, POLICIES.default.idx)
broker.put_container('z', 0, 0, 0, 0, POLICIES.default.idx)
broker.reclaim(Timestamp(time()).internal, time())
# self.assertEqual(len(res), 2)
# self.assertTrue(isinstance(res, tuple))
# containers, account_name = res
# self.assertTrue(containers is None)
# self.assertTrue(account_name is None)
# Now delete the account
broker.delete_db(Timestamp(time()).internal)
broker.reclaim(Timestamp(time()).internal, time())
# self.assertEqual(len(res), 2)
# self.assertTrue(isinstance(res, tuple))
# containers, account_name = res
# self.assertEqual(account_name, 'test_account')
# self.assertEqual(len(containers), 3)
# self.assertTrue('x' in containers)
# self.assertTrue('y' in containers)
# self.assertTrue('z' in containers)
# self.assertTrue('a' not in containers)
def test_delete_db_status(self):
ts = (Timestamp(t).internal for t in itertools.count(int(time())))