From a6ec3197c40205b1ac91bb7bb9a5dec4bb3901f6 Mon Sep 17 00:00:00 2001 From: Ehsan Akhgari Date: Tue, 18 Sep 2012 21:54:53 -0400 Subject: [PATCH] Improve the failure message when a mochitest ends without a call to SimpleTest.finish, no bug, irc-r=vlad DONTBUILD --- testing/mochitest/tests/SimpleTest/TestRunner.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/testing/mochitest/tests/SimpleTest/TestRunner.js b/testing/mochitest/tests/SimpleTest/TestRunner.js index c95534887d20..980930ba1c7f 100644 --- a/testing/mochitest/tests/SimpleTest/TestRunner.js +++ b/testing/mochitest/tests/SimpleTest/TestRunner.js @@ -427,8 +427,9 @@ TestRunner.testFinished = function(tests) { if (TestRunner.currentTestURL != TestRunner.getLoadedTestURL()) { TestRunner.error("TEST-UNEXPECTED-FAIL | " + TestRunner.currentTestURL + - " | finished in a non-clean fashion (in " + - TestRunner.getLoadedTestURL() + ")"); + " | " + TestRunner.getLoadedTestURL() + + " finished in a non-clean fashion, probably" + + " because it didn't call SimpleTest.finish()"); tests.push({ result: false }); }