Skip to content

Commit

Permalink
Merge pull request socketio#1999 from nkzawa/patch-3
Browse files Browse the repository at this point in the history
Fix a few assertions are not executed properly
  • Loading branch information
rauchg committed Nov 22, 2015
2 parents d4fb6a5 + 56fe266 commit 988d9d2
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/socket.io.js
Original file line number Diff line number Diff line change
Expand Up @@ -668,7 +668,7 @@ describe('socket.io', function(){
});
function getClients() {
sio.of('/chat').clients(function(error, sids) {
expect(error).to.be.undefined;
expect(error).to.not.be.ok();
expect(sids).to.contain(chatSids[0]);
expect(sids).to.contain(chatSids[1]);
expect(sids).to.not.contain(otherSid);
Expand Down Expand Up @@ -711,7 +711,7 @@ describe('socket.io', function(){
});
function getClients() {
sio.of('/chat').in('foo').clients(function(error, sids) {
expect(error).to.be.undefined;
expect(error).to.not.be.ok();
expect(sids).to.contain(chatFooSid);
expect(sids).to.not.contain(chatBarSid);
expect(sids).to.not.contain(otherSid);
Expand Down Expand Up @@ -754,7 +754,7 @@ describe('socket.io', function(){
});
function getClients() {
sio.of('/chat').clients(function(error, sids) {
expect(error).to.be.undefined;
expect(error).to.not.be.ok();
expect(sids).to.contain(chatFooSid);
expect(sids).to.contain(chatBarSid);
expect(sids).to.not.contain(otherSid);
Expand Down

0 comments on commit 988d9d2

Please sign in to comment.