Skip to content
This repository has been archived by the owner on Feb 26, 2022. It is now read-only.

Commit

Permalink
Bug 581109 - cfx test should classify all errors and warnings to nsIC…
Browse files Browse the repository at this point in the history
…onsoleService as warnings, not errors
  • Loading branch information
Atul Varma [:atul] committed Jul 26, 2010
1 parent b291cde commit eaedda7
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions packages/test-harness/lib/harness.js
Expand Up @@ -223,12 +223,19 @@ function cleanup() {

sandbox.unload();

if ((sandbox.console.errorsLogged ||
consoleListener.errorsLogged) && !results.failed) {
if (sandbox.console.errorsLogged && !results.failed) {
results.failed++;
console.error("warnings and/or errors were logged.");
}

if (consoleListener.errorsLogged && !results.failed) {
console.warn(consoleListener.errorsLogged + " " +
"warnings or errors were logged to the " +
"platform's nsIConsoleService, which could " +
"be of no consequence; however, they could also " +
"be indicative of aberrant behavior.");
}

consoleListener.errorsLogged = 0;
sandbox = null;

Expand Down

0 comments on commit eaedda7

Please sign in to comment.