Skip to content

Conversation

@kevinAlbs
Copy link
Contributor

@kevinAlbs kevinAlbs commented Mar 12, 2024

Summary

Skip tests using the rangePreview algorithm for server 8.0+:

  • Skip specification tests with the name fle2v2-Range-*
  • Skip the prose test 22. Range Explicit Encryption.

This PR does not resolve DRIVERS-2767. This PR is intended to prevent failures in driver of latest server builds until range tests are added.

Background & Motivation

This slack thread notes:

Upon this change hitting master, creating collections with the "rangePreview" query type will fail.

I expect driver tests will start to fail once https://github.com/10gen/mongo/pull/19645 is merged.


Please complete the following before merging:

  • [ ] Update changelog. N/A. Test changes only
  • Make sure there are generated JSON files from the YAML test files.
  • Test changes in at least one language driver. Tested in C, though server changes expected to fail are not-yet-available
  • [ ] Test these changes against all server versions and topologies (including standalone, replica set, sharded
    clusters, and serverless).
    C QE tests do not run against sharded or serverless

@kevinAlbs kevinAlbs requested a review from a team as a code owner March 12, 2024 14:58
@kevinAlbs kevinAlbs requested review from katcharov and removed request for a team March 12, 2024 14:58
@kevinAlbs kevinAlbs requested a review from a team as a code owner March 12, 2024 15:19
@kevinAlbs kevinAlbs requested review from dariakp and removed request for a team and dariakp March 12, 2024 15:19
Copy link
Contributor

@katcharov katcharov left a comment

Choose a reason for hiding this comment

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

LGTM - I have assumed the changes were automated (global replace etc.) and have not confirmed that they have been applied to all relevant files.

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