Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
baileympearson committed Jun 30, 2023
1 parent 91e3365 commit 8d644eb
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 3 deletions.
1 change: 0 additions & 1 deletion test/integration/crud/misc_cursors.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -3601,7 +3601,6 @@ describe('Cursor', function () {

await collection.insertMany(docs);

// TODO - talk to Neal about this test
const cursor = await collection.find({}, { batchSize: 3 });
for (let i = 0; i < 3; ++i) {
await cursor.next();
Expand Down
1 change: 0 additions & 1 deletion test/integration/node-specific/abstract_cursor.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,6 @@ describe('class AbstractCursor', function () {
expect(doc.name).to.equal('JOHN DOE');
});

// skipped because these tests fail after throwing uncaught exceptions
it(`when the transform throws, Cursor.stream() propagates the error to the user`, async () => {
const cursor = collection
.find()
Expand Down
1 change: 0 additions & 1 deletion test/integration/node-specific/cursor_stream.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,6 @@ describe('Cursor Streams', function () {
const stream = cursor.stream();
stream.on('error', err => (error = err));
cursor.on('close', function () {
// NOTE: use `setImmediate` here because the stream implementation uses `nextTick` to emit the error
setTimeout(() => {
expect(error).to.exist;
client.close(done);
Expand Down

0 comments on commit 8d644eb

Please sign in to comment.