Keep tests from calling done() twice.

feathersjs · Feb 12, 2015 · 9bd878c · 9bd878c
1 parent 30346d0
commit 9bd878c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/test/providers/primus.test.js b/test/providers/primus.test.js
@@ -438,10 +438,16 @@ describe('Primus provider', function () {
       socket.send('tasks::remove', 1, {}, function() {});
       socket.send('tasks::remove', 23, {}, function() {});
 
+      var ready = false;
+
       socket.on('tasks removed', function (data) {
         service.removed = oldRemoved;
         assert.equal(data.id, 23);
-        done();
+        console.log('Ready: ' + ready);
+        if (ready) {
+          done();
+        }
+        ready = true;
       });
     });
   });

diff --git a/test/providers/socketio.test.js b/test/providers/socketio.test.js
@@ -437,10 +437,15 @@ describe('SocketIO provider', function () {
       socket.emit('tasks::remove', 1, {}, function() {});
       socket.emit('tasks::remove', 23, {}, function() {});
 
+      var ready = false;
+
       socket.on('tasks removed', function (data) {
         service.removed = oldRemoved;
         assert.equal(data.id, 23);
-        done();
+        if (ready) {
+          done();
+        }
+        ready = true;
       });
     });
   });