Skip to content

Commit ea29928

Browse files
committed
test(cursor): reduce single test run by four seconds
NODE-1132
1 parent 43e3a17 commit ea29928

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

test/functional/cursor_tests.js

+16-16
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
var test = require('./shared').assert;
33
var setupDatabase = require('./shared').setupDatabase;
44
var fs = require('fs');
5+
var expect = require('chai').expect;
56

67
describe('Cursor', function() {
78
before(function() {
@@ -2285,7 +2286,9 @@ describe('Cursor', function() {
22852286
collection
22862287
) {
22872288
test.equal(null, err);
2288-
var closed = false;
2289+
2290+
var closeCount = 0;
2291+
var errorOccurred = false;
22892292

22902293
var count = 100;
22912294
// Just hammer the server
@@ -2302,16 +2305,20 @@ describe('Cursor', function() {
23022305
});
23032306

23042307
stream.on('error', function(err) {
2305-
test.ok(err != null);
2308+
expect(err).to.exist;
2309+
errorOccurred = true;
23062310
});
23072311

2308-
stream.on('end', function() {
2309-
closed = true;
2310-
});
2312+
var validator = () => {
2313+
closeCount++;
2314+
if (closeCount === 2) {
2315+
expect(errorOccurred).to.equal(true);
2316+
done();
2317+
}
2318+
};
23112319

2312-
stream.on('close', function() {
2313-
closed = true;
2314-
});
2320+
stream.on('end', validator);
2321+
stream.on('close', validator);
23152322

23162323
// Just hammer the server
23172324
for (var i = 0; i < 100; i++) {
@@ -2322,14 +2329,7 @@ describe('Cursor', function() {
23222329
});
23232330
}
23242331

2325-
setTimeout(function() {
2326-
client.close();
2327-
2328-
setTimeout(function() {
2329-
test.equal(true, closed);
2330-
done();
2331-
}, 5000);
2332-
}, 800);
2332+
setTimeout(() => client.close(), 800);
23332333
}
23342334
});
23352335
}

0 commit comments

Comments
 (0)