Skip to content

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Apr 21, 2023

https://jira.mongodb.org/browse/PHPLIB-1116

Opening this PR preemptively as I noticed some test runner changes were necessary to POC mongodb/specifications#1389. This PR will be updated once the spec is merged and we have a final sync commit.

@jmikola jmikola marked this pull request as ready for review April 21, 2023 16:39
@jmikola jmikola requested a review from alcaeus April 21, 2023 16:40
@jmikola
Copy link
Member Author

jmikola commented Apr 22, 2023

This PR should wait on mongodb/specifications#1410.

Comment on lines +689 to +694
/* runCommand spec tests may execute commands that return a
* cursor (e.g. aggregate). PHPC creates a cursor automatically
* so cannot return the original command result. Existing spec
* tests do not evaluate the result, so we can return null here.
* If that changes down the line, we can use command monitoring
* to capture and return the command result. */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jmikola
Copy link
Member Author

jmikola commented May 8, 2023

Updated with follow-up commit for DRIVERS-2577: mongodb/specifications@146964a

Will merge after CI completes.

@jmikola
Copy link
Member Author

jmikola commented May 8, 2023

Found a separate issue with the spec tests, so this will need to wait on mongodb/specifications#1416 to be resolved. Pushing a POC commit in the meantime.

@jmikola jmikola force-pushed the phplib-1116 branch 2 times, most recently from 848b948 to 69d507f Compare May 8, 2023 15:15
@jmikola jmikola merged commit f50276a into mongodb:master May 11, 2023
@jmikola jmikola deleted the phplib-1116 branch May 11, 2023 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants