openstack/swift
Code Issues Proposed changes

Fix test comment and remove extra parameter

Browse Source 
Fixed the comment in the test to match exactly what's
being removed and what the expected result is.

Also, removed that extra '/' parameter which was causing
the assert to test at the wrong directory level.

Change-Id: I2f27f0d12c08375c61047a3f861c94a3dd3915c6
Signed-off-by: Thiago da Silva <thiago@redhat.com>
This commit is contained in:
Thiago da Silva 2017-02-10 14:49:50 -05:00
parent c8a2b77313
commit 04502a9f64
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
test/probe/test_replication_servers_working.py
View File

@ -149,7 +149,9 @@ class TestReplicatorFunctions(ReplProbeTest):
raise
time.sleep(1)
# Check behavior by deleting hashes.pkl file
# Delete directories and files in objects storage without
# deleting file "hashes.pkl".
# Check, that files not replicated.
for directory in os.listdir(os.path.join(test_node, data_dir)):
for input_dir in os.listdir(os.path.join(
test_node, data_dir, directory)):
@ -168,13 +170,15 @@ class TestReplicatorFunctions(ReplProbeTest):
test_node, data_dir, directory)):
self.assertFalse(os.path.isdir(
os.path.join(test_node, data_dir,
directory, '/', input_dir)))
directory, input_dir)))
break
except Exception:
if time.time() - begin > 60:
raise
time.sleep(1)
# Now, delete file "hashes.pkl".
# Check, that all files were replicated.
for directory in os.listdir(os.path.join(test_node, data_dir)):
os.remove(os.path.join(
test_node, data_dir, directory, 'hashes.pkl'))