mirror of
https://github.com/samba-team/samba.git
synced 2024-12-23 17:34:34 +03:00
subunit: Don't abort when receiving test results from tests that weren't
announced.
This commit is contained in:
parent
64bf8c400c
commit
664eacc53a
@ -68,16 +68,34 @@ def parse_results(msg_ops, statistics, fh):
|
||||
else:
|
||||
reason = None
|
||||
if result in ("success", "successful"):
|
||||
open_tests.pop() #FIXME: Check that popped value == $testname
|
||||
try:
|
||||
open_tests.remove(testname)
|
||||
except KeyError:
|
||||
statistics['TESTS_ERROR']+=1
|
||||
msg_ops.end_test(testname, "error", True,
|
||||
"Test was never started")
|
||||
else:
|
||||
statistics['TESTS_EXPECTED_OK']+=1
|
||||
msg_ops.end_test(testname, "success", False, reason)
|
||||
elif result in ("xfail", "knownfail"):
|
||||
open_tests.pop() #FIXME: Check that popped value == $testname
|
||||
try:
|
||||
open_tests.remove(testname)
|
||||
except KeyError:
|
||||
statistics['TESTS_ERROR']+=1
|
||||
msg_ops.end_test(testname, "error", True,
|
||||
"Test was never started")
|
||||
else:
|
||||
statistics['TESTS_EXPECTED_FAIL']+=1
|
||||
msg_ops.end_test(testname, "xfail", False, reason)
|
||||
expected_fail+=1
|
||||
elif result in ("failure", "fail"):
|
||||
open_tests.pop() #FIXME: Check that popped value == $testname
|
||||
try:
|
||||
open_tests.remove(testname)
|
||||
except KeyError:
|
||||
statistics['TESTS_ERROR']+=1
|
||||
msg_ops.end_test(testname, "error", True,
|
||||
"Test was never started")
|
||||
else:
|
||||
statistics['TESTS_UNEXPECTED_FAIL']+=1
|
||||
msg_ops.end_test(testname, "failure", True, reason)
|
||||
elif result == "skip":
|
||||
@ -89,7 +107,10 @@ def parse_results(msg_ops, statistics, fh):
|
||||
msg_ops.end_test(testname, "skip", False, reason)
|
||||
elif result == "error":
|
||||
statistics['TESTS_ERROR']+=1
|
||||
open_tests.pop() #FIXME: Check that popped value == $testname
|
||||
try:
|
||||
open_tests.remove(testname)
|
||||
except KeyError:
|
||||
pass
|
||||
msg_ops.end_test(testname, "error", True, reason)
|
||||
elif result == "skip-testsuite":
|
||||
msg_ops.skip_testsuite(testname)
|
||||
|
Loading…
Reference in New Issue
Block a user