-
Notifications
You must be signed in to change notification settings - Fork 266
PHPLIB-694: Spec tests for snapshot sessions #851
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
Conversation
1de8c49
to
2c7aaf8
Compare
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.
LGTM, good to merge once tests have been updated.
What is a prose test as opposed to a spec test (referred to in the JIRA ticket)? |
Spec tests are written in YML/JSON and executed using a test runner. Prose tests are expressed in written text and implemented manually. The corresponding prose test for this work was defined in https://github.com/mongodb/specifications/blob/master/source/sessions/tests/README.rst#prose-tests and implemented in mongodb/mongo-php-driver@ac49353#diff-f526132129853b0d00c150317c2cb972ac2d17d32e05fed83a6313d91fadcecf |
LGTMs noted. Will leave this open until mongodb/specifications#1046 is merged and I can update the commit reference. |
This is necessary to pull in changes from PHPC-1761 for testing snapshot sessions.
https://jira.mongodb.org/browse/PHPLIB-694
Syncs tests with mongodb/specifications/pull/1046 (will be updated once merged).
Temporarily reduces the
driver-versions
axis in the Evergreen matrix tolatest-dev
to pull in changes from PHPC-1761.