Fixed a broken ref (npm-sinon is not directly compatible with browsers).

Fixed a brokenly slow test-runner (browser tests are now ignoring leaks).
fizker · Jun 26, 2012 · bc7bb24 · bc7bb24
1 parent 6c6d694
commit bc7bb24
Show file tree

Hide file tree

Showing 2 changed files with 3,566 additions and 2 deletions.
diff --git a/test/browser.mustache b/test/browser.mustache
@@ -4,11 +4,20 @@
 
 <link rel=stylesheet href=../node_modules/mocha/mocha.css>
 <script src=../node_modules/mocha/mocha.js></script>
-<script src=../node_modules/sinon/lib/sinon.js></script>
+<script src=lib/sinon-1.3.4.js></script>
 <script src=../node_modules/chai/chai.js></script>
 <script src=../node_modules/sinon-chai/lib/sinon-chai.js></script>
 <script>
-mocha.setup('bdd')
+mocha.setup(
+	{ ui: 'bdd'
+	// Checking for leaks is extremely slow in Safari (all tests run in > 78s
+	// vs < 0.90s with `ignoreLeaks: true`, and both settings run < 0.6s in
+	// chrome and < 80ms in node.
+	// So: We are checking for leaks in node, not in the browser (which is for
+	// cross-browser testing anyway!
+	, ignoreLeaks: true
+	}
+);
 var expect = chai.expect
 </script>