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 <[email protected]> --- 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
