Bug 781116 - "finished in a non-clean fashion" message should use error() instead of log(); r=ted
authorMs2ger <ms2ger@gmail.com>
Thu, 09 Aug 2012 09:11:33 +0200
changeset 101934 9c0617ad9de331534d7c325d4d73ea63bff7acde
parent 101933 7532bf93435cae8243fd6543852380b28215035f
child 101935 c77acb256aec13024f61adaf174c4d152b1c381e
push id13237
push useremorley@mozilla.com
push dateThu, 09 Aug 2012 11:51:38 +0000
treeherdermozilla-inbound@f799dd180292 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersted
bugs781116
milestone17.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 781116 - "finished in a non-clean fashion" message should use error() instead of log(); r=ted
testing/mochitest/tests/SimpleTest/TestRunner.js
--- a/testing/mochitest/tests/SimpleTest/TestRunner.js
+++ b/testing/mochitest/tests/SimpleTest/TestRunner.js
@@ -420,20 +420,20 @@ TestRunner.testFinished = function(tests
                 TestRunner.log("TEST-INFO | Found unexpected crash dump file " +
                                aFilename + ".");
             });
         }
     }
 
     function runNextTest() {
         if (TestRunner.currentTestURL != TestRunner.getLoadedTestURL()) {
-            TestRunner.log("TEST-UNEXPECTED-FAIL | " +
-                           TestRunner.currentTestURL +
-                           " | finished in a non-clean fashion (in " +
-                           TestRunner.getLoadedTestURL() + ")");
+            TestRunner.error("TEST-UNEXPECTED-FAIL | " +
+                             TestRunner.currentTestURL +
+                             " | finished in a non-clean fashion (in " +
+                             TestRunner.getLoadedTestURL() + ")");
             tests.push({ result: false });
         }
 
         var runtime = new Date().valueOf() - TestRunner._currentTestStartTime;
         TestRunner.log("TEST-END | " +
                        TestRunner.currentTestURL +
                        " | finished in " + runtime + "ms");