fix: correct test relaying on jasmine timeout

mhevery · mhevery · commit 4f7d6ae1b9cc · 2015-12-15T18:52:03.000-08:00
The tests run slow, because afterEach was async but never called done().
because it was afterEach any errors are ignored and so it did not show
up as failure. Instead the test ran very slow until jasmine timed out.

This fix corrects the afterEach and decreases the timeout to 1,000 ms.
diff --git a/test/patch/WebSocket.spec.js b/test/patch/WebSocket.spec.js
@@ -20,6 +20,7 @@ describe('WebSocket', ifEnvSupports('WebSocket', function () {
   afterEach(function (done) {
     socket.addEventListener('close', done);
     socket.close();
+    done();
   });
 
 
@@ -102,6 +103,6 @@ describe('WebSocket', ifEnvSupports('WebSocket', function () {
     setTimeout(function() {
       expect(log).toEqual('');
       done();
-    }, 500);
+    }, 100);
   });
 }));
diff --git a/test/setup.js b/test/setup.js
@@ -1,3 +1,5 @@
+jasmine.DEFAULT_TIMEOUT_INTERVAL = 2000;
+
 // Setup tests for Zone without microtask support
 require('../lib/browser/zone.js');
 require('../lib/browser/long-stack-trace-zone.js');

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+jasmine.DEFAULT_TIMEOUT_INTERVAL = 2000;`
	`2`	`+`
`1`	`3`	`// Setup tests for Zone without microtask support`
`2`	`4`	`require('../lib/browser/zone.js');`
`3`	`5`	`require('../lib/browser/long-stack-trace-zone.js');`