Skip to content

Commit 8d644eb

Browse files
Apply suggestions from code review
1 parent 91e3365 commit 8d644eb

File tree

3 files changed

+0
-3
lines changed

3 files changed

+0
-3
lines changed

test/integration/crud/misc_cursors.test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3601,7 +3601,6 @@ describe('Cursor', function () {
36013601

36023602
await collection.insertMany(docs);
36033603

3604-
// TODO - talk to Neal about this test
36053604
const cursor = await collection.find({}, { batchSize: 3 });
36063605
for (let i = 0; i < 3; ++i) {
36073606
await cursor.next();

test/integration/node-specific/abstract_cursor.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,6 @@ describe('class AbstractCursor', function () {
117117
expect(doc.name).to.equal('JOHN DOE');
118118
});
119119

120-
// skipped because these tests fail after throwing uncaught exceptions
121120
it(`when the transform throws, Cursor.stream() propagates the error to the user`, async () => {
122121
const cursor = collection
123122
.find()

test/integration/node-specific/cursor_stream.test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,6 @@ describe('Cursor Streams', function () {
296296
const stream = cursor.stream();
297297
stream.on('error', err => (error = err));
298298
cursor.on('close', function () {
299-
// NOTE: use `setImmediate` here because the stream implementation uses `nextTick` to emit the error
300299
setTimeout(() => {
301300
expect(error).to.exist;
302301
client.close(done);

0 commit comments

Comments
 (0)