From 02dbb7a68f828863e5b71dc15488e665d484ab6e Mon Sep 17 00:00:00 2001 From: Yaniv Kaul Date: Tue, 3 Jul 2018 00:45:30 +0300 Subject: Shorten all the logs around verify_io_procs No functional change, just make the tests a bit more readable. It could be moved to a decorator later on, wrapping tests. Change-Id: I484bb8b46907ee8f33dfcf4c960737a21819cd6a Signed-off-by: Yaniv Kaul --- .../snapshot/test_validate_snaps_dir_over_uss.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) (limited to 'tests/functional/snapshot/test_validate_snaps_dir_over_uss.py') diff --git a/tests/functional/snapshot/test_validate_snaps_dir_over_uss.py b/tests/functional/snapshot/test_validate_snaps_dir_over_uss.py index 5b91f8b40..8fcce0096 100644 --- a/tests/functional/snapshot/test_validate_snaps_dir_over_uss.py +++ b/tests/functional/snapshot/test_validate_snaps_dir_over_uss.py @@ -139,11 +139,11 @@ class TestValidateUss(GlusterBaseClass): self.io_validation_complete = False # Validate IO - g.log.info("Wait for IO to complete and validate IO ...") - ret = validate_io_procs(self.all_mounts_procs, self.mounts) - self.assertTrue(ret, "IO failed on some of the clients") + self.assertTrue( + validate_io_procs(self.all_mounts_procs, self.mounts), + "IO failed on some of the clients" + ) self.io_validation_complete = True - g.log.info("I/O successful on clients") # get the snapshot list. snap_list = get_snap_list(self.mnode) @@ -212,8 +212,10 @@ class TestValidateUss(GlusterBaseClass): # IO should fail g.log.info("IO should Fail with ROFS error.....") - ret = validate_io_procs(all_mounts_procs, self.mounts) - self.assertFalse(ret, "Unexpected: IO successfully completed") + self.assertFalse( + validate_io_procs(all_mounts_procs, self.mounts), + "Unexpected: IO successfully completed" + ) g.log.info("Expected: IO failed to complete") # validate snap-0 present in mountpoint -- cgit