path: root/tests/functional/bvt
diff options
authorNigel Babu <>2016-12-20 12:52:25 +0530
committerNigel Babu <>2017-02-01 12:48:22 +0530
commited2dfbe2e0b10a8595d0161dbb2190f0f702c0ab (patch)
tree3061897b209ff46f65ca97dd37c6fb05a539560e /tests/functional/bvt
parent86f5e76e38e6aefce88917ee88c2eeaf18431046 (diff)
Raise custom exceptions rather assert failures
The setup should never raise an assert failure in a test. Only tests should fail an assert. If an essential test setup doesn't work, we should be raising custom exceptions instead. Change-Id: I6d5cce448132b71b6fde3a39fef894be8b1216d3 Signed-off-by: Nigel Babu <>
Diffstat (limited to 'tests/functional/bvt')
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/functional/bvt/ b/tests/functional/bvt/
index 55db725b9..074d57910 100644
--- a/tests/functional/bvt/
+++ b/tests/functional/bvt/
@@ -45,7 +45,8 @@ class BvtTestsClass(GlusterVolumeBaseClass):
cls.script_upload_path = "/tmp/"
ret = os.path.exists(cls.script_local_path)
- assert (ret is True), ("Unable to find the io scripts")
+ if not ret:
+ raise Exception("Unable to find the io scripts")
for client in cls.clients:
g.upload(client, cls.script_local_path, cls.script_upload_path)