selftest: list.remove raises ValueError rather than KeyError.

diff --git a/selftest/subunithelper.py b/selftest/subunithelper.py
index d03a1d04419efa2b676a01dcd31955cb393999a9..d2054a94987766c8b5874f61f007d6d0f308520a 100644 (file)
--- 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":