Skip to content
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

PHPLIB-1392: Convert transaction tests and support schema 1.9 #1226

Merged
merged 9 commits into from Feb 16, 2024

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Feb 1, 2024

https://jira.mongodb.org/browse/PHPLIB-1392
https://jira.mongodb.org/browse/PHPLIB-1397

This branch is being used to POC converted unified spec tests in:

Patch build on all topologies and serverless: https://spruce.mongodb.com/version/65cbe6b732f4174551b0143a/tasks

This PR implements the createEntities operation from UTF schema 1.9, as it was necessary for some spec tests. The following 1.9 schema features have been left unimplemented:

  • createOptions field to initialData
  • timeoutMS field in collectionOrDatabaseOptions
  • isTimeoutError field to expectedError

@jmikola jmikola marked this pull request as ready for review February 5, 2024 22:11
@jmikola jmikola changed the title Support schema 1.9 in unified test runner PHPLIB-1392: Convert transaction tests and support schema 1.9 Feb 7, 2024
@jmikola
Copy link
Member Author

jmikola commented Feb 7, 2024

Evergreen failures all seem related to rotating Azure credentials for CSFLE (see also: #1228 (comment)). @alcaeus: please let me know when that's done and I'll rebase.

@jmikola jmikola force-pushed the utf-schema-1.9 branch 4 times, most recently from e62d4b5 to 388dc59 Compare February 7, 2024 21:46
Copy link
Member

@alcaeus alcaeus left a comment

Choose a reason for hiding this comment

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

Changes to the unified spec runner look fine. I did not review changes in the individual test files (doing that in the corresponding spec PR).

Confirmed that test failures are due to Azure credentials, but again I didn't check all the builds. We'll have to wait until the new credentials are available.

UnifiedTestRunner::preventStaleDbVersionError() already handles this. The Operation code was likely always redundant and dates back to the original test runner implementation in 4608a73.
See: mongodb#1228

Also updates a reference to a moved JIRA issue
@jmikola jmikola force-pushed the utf-schema-1.9 branch 2 times, most recently from e91893c to 1a28efa Compare February 13, 2024 22:01
@jmikola jmikola merged commit 4192095 into mongodb:master Feb 16, 2024
14 checks passed
@jmikola jmikola deleted the utf-schema-1.9 branch February 16, 2024 05:36
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.

None yet

2 participants