openstack
/
fuel-qa
Code Issues Proposed changes

Change logic for checking failed tests in assert_ostf_run(). 

If in list actual_failed names we have something else that
 we are ordered in failed_test_name test must be failed.
But if in list actual_failed_names we haven't all tests
 that we are ordered in failed_test_name test must be passed.

Closes bug: 1615517

Change-Id: Ibaafe43039e96d6a39b7c405ac6a0ef55d373c8c
This commit is contained in:
Mikhail Samoylov 2016-08-22 16:14:09 +03:00
parent 55662870e8
commit ac1e3e9adc
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
fuelweb_test/models/fuel_web_client.py
View File

@ -267,11 +267,12 @@ class FuelWebClient29(object):
pretty_log(test_result, indent=1)))
if failed_test_name:
for test_name in failed_test_name:
assert_true(test_name in actual_failed_names,
for test_name in actual_failed_names:
assert_true(test_name in failed_test_name,
'WARNING! Unexpected fail: '
'expected {0}, actual {1}'.format(
failed_test_name, actual_failed_names))
failed_test_name, actual_failed_names)
)
assert_true(
failed <= should_fail, 'Failed {0} OSTF tests; should fail'