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

DRIVERS-2407 update expected error in fle2-InsertFind-Unindexed #1286

Merged
merged 1 commit into from Aug 2, 2022

Conversation

kevinAlbs
Copy link
Contributor

@kevinAlbs kevinAlbs commented Jul 30, 2022

Summary

  • Update expected error in fle2-InsertFind-Unindexed.

Background & Motivation

The expected error message changed recently in https://github.com/10gen/mongo-enterprise-modules/commit/212b584d4f7a44bed41c826a180a4aff00923d7a#diff-5f12b55e8d5c52c2f62853ec595dc2c1e2e5cb4fdbf7a32739a8e3acb3c6f818 Driver tests against "latest" will start failing.

Before the message was "cannot query non-indexed fields with the randomized encryption algorithm". Now it is "can only execute encrypted equality queries with an encrypted equality index".

Please complete the following before merging:

  • Bump spec version and last modified date.
  • Update changelog.
  • Make sure there are generated JSON files from the YAML test files.
  • Test changes in at least one language driver. Tested in Go
  • Test these changes against all server versions and topologies (including standalone, replica set, sharded clusters, and serverless).

@kevinAlbs kevinAlbs requested a review from a team as a code owner July 30, 2022 19:42
@kevinAlbs kevinAlbs requested review from comandeo and removed request for a team July 30, 2022 19:42
@kevinAlbs kevinAlbs marked this pull request as draft July 30, 2022 19:46
@kevinAlbs kevinAlbs marked this pull request as ready for review July 30, 2022 19:49
@kevinAlbs kevinAlbs merged commit 1a911b5 into mongodb:master Aug 2, 2022
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