diff options
author | Yaniv Kaul <ykaul@redhat.com> | 2018-07-03 00:45:30 +0300 |
---|---|---|
committer | Nigel Babu <nigelb@redhat.com> | 2018-07-17 04:14:01 +0000 |
commit | 02dbb7a68f828863e5b71dc15488e665d484ab6e (patch) | |
tree | 4ae10586c3f26f9e73a6d533bbd4af88094c6ef5 /tests/functional/glusterd/test_readonly_option_on_volume.py | |
parent | 87f9679588c54c550447acdc8f0cc15626c7d881 (diff) |
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 <ykaul@redhat.com>
Diffstat (limited to 'tests/functional/glusterd/test_readonly_option_on_volume.py')
-rw-r--r-- | tests/functional/glusterd/test_readonly_option_on_volume.py | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/tests/functional/glusterd/test_readonly_option_on_volume.py b/tests/functional/glusterd/test_readonly_option_on_volume.py index 09e0b4378..064d1ac8a 100644 --- a/tests/functional/glusterd/test_readonly_option_on_volume.py +++ b/tests/functional/glusterd/test_readonly_option_on_volume.py @@ -114,10 +114,11 @@ class TestReadOnlyOptionOnVolume(GlusterBaseClass): self.counter = self.counter + 10 # Validate IO - g.log.info("Wait for IO to complete and validate IO ...") - ret = validate_io_procs(self.all_mounts_procs, self.mounts) - self.assertFalse(ret, "IO should fail on mount points of readonly " - "volumes but IO success") + self.assertFalse( + validate_io_procs(self.all_mounts_procs, self.mounts), + "IO should fail on mount points of readonly " + "volumes but IO succeeded" + ) g.log.info("IO failed on mount points of read only volumes " "as expected") @@ -150,7 +151,7 @@ class TestReadOnlyOptionOnVolume(GlusterBaseClass): self.counter = self.counter + 10 # 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") - g.log.info("IO is successful on all mounts") + self.assertTrue( + validate_io_procs(self.all_mounts_procs, self.mounts), + "IO failed on some of the clients" + ) |