changeset 32952:6123a5267119

tests: remove WarnTest We would raise this if a test didn't return a result code. AFAICT this can only occur if there is a logic error in the test harness itself. I don't think it is worth the code complexity to distinguish this failure scenario from a regular test failure.
author Gregory Szorc <gregory.szorc@gmail.com>
date Sat, 03 Jun 2017 17:13:35 -0700
parents f83b7376a300
children 69d1c3ea0d6f
files tests/run-tests.py
diffstat 1 files changed, 1 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/tests/run-tests.py	Sat Jun 03 17:09:13 2017 -0700
+++ b/tests/run-tests.py	Sat Jun 03 17:13:35 2017 -0700
@@ -717,8 +717,6 @@
                 # test we "ran", but we want to exclude skipped tests
                 # from those we count towards those run.
                 result.testsRun -= 1
-            except WarnTest as e:
-                result.addWarn(self, str(e))
             except ReportedTest as e:
                 pass
             except self.failureException as e:
@@ -788,7 +786,7 @@
         elif ret == 'timeout':
             self.fail('timed out')
         elif ret is False:
-            raise WarnTest('no result code from test')
+            self.fail('no result code from test')
         elif out != self._refout:
             # Diff generation may rely on written .err file.
             if (ret != 0 or out != self._refout) and not self._skipped \
@@ -1481,9 +1479,6 @@
 
 iolock = threading.RLock()
 
-class WarnTest(Exception):
-    """Raised to indicate that a test warned."""
-
 class ReportedTest(Exception):
     """Raised to indicate that a test already reported."""