From 04502a9f6424048121224a423a341c220de1b0d9 Mon Sep 17 00:00:00 2001 From: Thiago da Silva Date: Fri, 10 Feb 2017 14:49:50 -0500 Subject: [PATCH] Fix test comment and remove extra parameter 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 --- test/probe/test_replication_servers_working.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/test/probe/test_replication_servers_working.py b/test/probe/test_replication_servers_working.py index c486f93c67..8062825aea 100644 --- a/test/probe/test_replication_servers_working.py +++ b/test/probe/test_replication_servers_working.py @@ -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'))