In r5451, several changes to the test were made,
but a very simple one was forgotten: Change the
check made in the end of the test to verify whether
the test has passed or fail. n_fail is now a list
of failures, so we need to test whether it is an
empty list or not.
Signed-off-by: Lucas Meneghel Rodrigues <lmr@xxxxxxxxxx>
---
client/tests/kvm/tests/physical_resources_check.py | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/client/tests/kvm/tests/physical_resources_check.py b/client/tests/kvm/tests/physical_resources_check.py
index 9691d9e..4e83814 100644
--- a/client/tests/kvm/tests/physical_resources_check.py
+++ b/client/tests/kvm/tests/physical_resources_check.py
@@ -173,7 +173,7 @@ def run_physical_resources_check(test, params, env):
catch_serial_cmd)
n_fail.extend(f_fail)
- if n_fail != 0:
+ if n_fail:
session.close()
raise error.TestFail("Physical resources check test "
"reported %s failures:\n%s" %
--
1.7.6
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
