summaryrefslogtreecommitdiff
path: root/selftest
diff options
context:
space:
mode:
authorJelmer Vernooij <jelmer@samba.org>2010-05-30 17:05:07 +0200
committerJelmer Vernooij <jelmer@samba.org>2010-05-30 17:05:49 +0200
commit598656985aeb1a1ac11d246bb942c2248dfbfd8c (patch)
tree1e6234525f918baf5a628ad7feeaa8098b26f977 /selftest
parenta1b44f4335f41a0c11505f1552d9173ca0b469f4 (diff)
downloadsamba-598656985aeb1a1ac11d246bb942c2248dfbfd8c.tar.gz
selftest: list.remove raises ValueError rather than KeyError.
Diffstat (limited to 'selftest')
-rw-r--r--selftest/subunithelper.py8
1 files changed, 4 insertions, 4 deletions
diff --git a/selftest/subunithelper.py b/selftest/subunithelper.py
index d03a1d04419..d2054a94987 100644
--- a/selftest/subunithelper.py
+++ b/selftest/subunithelper.py
@@ -76,7 +76,7 @@ def parse_results(msg_ops, statistics, fh):
if result in ("success", "successful"):
try:
open_tests.remove(testname)
- except KeyError:
+ except ValueError:
statistics['TESTS_ERROR']+=1
msg_ops.end_test(testname, "error", True,
"Test was never started")
@@ -86,7 +86,7 @@ def parse_results(msg_ops, statistics, fh):
elif result in ("xfail", "knownfail"):
try:
open_tests.remove(testname)
- except KeyError:
+ except ValueError:
statistics['TESTS_ERROR']+=1
msg_ops.end_test(testname, "error", True,
"Test was never started")
@@ -97,7 +97,7 @@ def parse_results(msg_ops, statistics, fh):
elif result in ("failure", "fail"):
try:
open_tests.remove(testname)
- except KeyError:
+ except ValueError:
statistics['TESTS_ERROR']+=1
msg_ops.end_test(testname, "error", True,
"Test was never started")
@@ -115,7 +115,7 @@ def parse_results(msg_ops, statistics, fh):
statistics['TESTS_ERROR']+=1
try:
open_tests.remove(testname)
- except KeyError:
+ except ValueError:
pass
msg_ops.end_test(testname, "error", True, reason)
elif result == "skip-testsuite":