You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by sv...@apache.org on 2022/03/27 18:03:48 UTC

svn commit: r1899259 - in /subversion/branches/1.14.x: ./ STATUS build/run_tests.py

Author: svn-role
Date: Sun Mar 27 18:03:48 2022
New Revision: 1899259

URL: http://svn.apache.org/viewvc?rev=1899259&view=rev
Log:
Merge r1897449 from trunk:

 * r1897449
   tests: Include additional information in an error message.
   Justification:
     Trivial test harness change that may help investigate unreproducible
     errors.
   Votes:
     +1: danielsh, markphip, hartmannathan

Modified:
    subversion/branches/1.14.x/   (props changed)
    subversion/branches/1.14.x/STATUS
    subversion/branches/1.14.x/build/run_tests.py

Propchange: subversion/branches/1.14.x/
------------------------------------------------------------------------------
  Merged /subversion/trunk:r1897449

Modified: subversion/branches/1.14.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.14.x/STATUS?rev=1899259&r1=1899258&r2=1899259&view=diff
==============================================================================
--- subversion/branches/1.14.x/STATUS (original)
+++ subversion/branches/1.14.x/STATUS Sun Mar 27 18:03:48 2022
@@ -115,11 +115,3 @@ Veto-blocked changes:
 
 Approved changes:
 =================
-
- * r1897449
-   tests: Include additional information in an error message.
-   Justification:
-     Trivial test harness change that may help investigate unreproducible
-     errors.
-   Votes:
-     +1: danielsh, markphip, hartmannathan

Modified: subversion/branches/1.14.x/build/run_tests.py
URL: http://svn.apache.org/viewvc/subversion/branches/1.14.x/build/run_tests.py?rev=1899259&r1=1899258&r2=1899259&view=diff
==============================================================================
--- subversion/branches/1.14.x/build/run_tests.py (original)
+++ subversion/branches/1.14.x/build/run_tests.py Sun Mar 27 18:03:48 2022
@@ -776,10 +776,12 @@ class TestHarness:
     # ### Even if failure==1 it could be that the test didn't run at all.
     if test_failed and test_failed != 1:
       if self.log:
-        log.write('FAIL:  %s: Unknown test failure; see tests.log.\n' % progbase)
+        log.write('FAIL:  %s: Unknown test failure (%s); see tests.log.\n'
+                  % (progbase, test_failed))
         log.flush()
       else:
-        log.write('FAIL:  %s: Unknown test failure.\n' % progbase)
+        log.write('FAIL:  %s: Unknown test failure (%s).\n'
+                  % (progbase, test_failed))
 
   def _run_c_test(self, progabs, progdir, progbase, test_nums, dot_count):
     'Run a c test, escaping parameters as required.'