-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test(NODE-3412): sync spec tests for serverless testing #2911
Conversation
b557e83
to
b68e844
Compare
3c559ef
to
92d09f5
Compare
…verless-spec-tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a question on one of these changes
operations.set('findOneAndDelete', async ({ entities, operation }) => { | ||
const collection = entities.getEntity('collection', operation.object); | ||
const { filter, ...opts } = operation.arguments; | ||
return (await collection.findOneAndDelete(filter, opts)).value; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious, how did this work before without extracting the value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we just didn't have any unified tests for this operation yet.
Syncing various spec tests for serverless testing:
NODE-3412, NODE-3353