Skip to content
This repository has been archived by the owner on Feb 4, 2022. It is now read-only.

Commit

Permalink
fix(wire-protocol): 2.6 killCursor should not way for reply
Browse files Browse the repository at this point in the history
NODE-1088
  • Loading branch information
mbroadst committed Oct 6, 2017
1 parent 31dbaf4 commit 7337d91
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 4 deletions.
1 change: 0 additions & 1 deletion lib/connection/pool.js
Expand Up @@ -287,7 +287,6 @@ function connectionFailureHandler(self, event) {
// Flush all work Items on this connection
while (this.workItems.length > 0) {
var workItem = this.workItems.shift();
// if(workItem.cb) workItem.cb(err);
if (workItem.cb) workItem.cb(err);
}

Expand Down
2 changes: 0 additions & 2 deletions lib/wireprotocol/2_6_support.js
Expand Up @@ -106,8 +106,6 @@ WireProtocol.prototype.killCursor = function(bson, ns, cursorState, pool, callba
} catch (err) {
callback(err, null);
}

return;
}

// Callback
Expand Down
2 changes: 1 addition & 1 deletion test/tests/unit/single/sessions_tests.js
Expand Up @@ -217,7 +217,7 @@ describe('Sessions (Single)', function() {
}
});

it.only('should return server sessions to the pool on `endSession`', {
it('should return server sessions to the pool on `endSession`', {
metadata: { requires: { topology: 'single' } },
test: function(done) {
let sentIsMaster = false;
Expand Down

0 comments on commit 7337d91

Please sign in to comment.