| Message ID | 1310680394-18093-1-git-send-email-lmr@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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" %
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@redhat.com> --- client/tests/kvm/tests/physical_resources_check.py | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)