Bug 676556 - GCLI requirejs tests log when they shouldn't; r=rcampbell
authorJoe Walker <jwalker@mozilla.com>
Fri, 05 Aug 2011 09:59:20 -0300
changeset 73951 4ec2052b0839249d086bdaffb34546cc58039906
parent 73950 183aba2199bfae752d3f7c406e8627ddac0b68de
child 73952 eef25ec2d58e3f7d845438209ba2d95c44f0e441
push id2
push userbsmedberg@mozilla.com
push dateFri, 19 Aug 2011 14:38:13 +0000
reviewersrcampbell
bugs676556
milestone8.0a1
Bug 676556 - GCLI requirejs tests log when they shouldn't; r=rcampbell
browser/devtools/webconsole/test/browser/browser_webconsole_gcli_require.js
--- a/browser/devtools/webconsole/test/browser/browser_webconsole_gcli_require.js
+++ b/browser/devtools/webconsole/test/browser/browser_webconsole_gcli_require.js
@@ -102,21 +102,21 @@ function testMultiImport() {
   ok(r1 === r2, 'double require was strict equal');
 }
 
 function testUncompilable() {
   // It's not totally clear how a module loader should perform with unusable
   // modules, however at least it should go into a flat spin ...
   // GCLI mini_require reports an error as it should
   try {
-      let unrequireable = require('gclitest/unrequirable');
-      fail();
+    let unrequireable = require('gclitest/unrequirable');
+    fail();
   }
   catch (ex) {
-      console.log(ex);
+    // an exception is expected
   }
 }
 
 function testRecursive() {
   // See Bug 658583
   // require('gclitest/recurse');
   // Also see the comments in the testRecursive() function
 }