Skip to content

PYTHON-5912 - Skip substringPreview on server 9.0+ - #2910

Merged
NoahStapp merged 1 commit into
mongodb:masterfrom
NoahStapp:PYTHON-5912
Jul 1, 2026
Merged

NoahStapp merged 1 commit into
mongodb:masterfrom
NoahStapp:PYTHON-5912

Conversation

@NoahStapp

@NoahStapp NoahStapp commented Jul 1, 2026 •

Copy link
Copy Markdown
Contributor

PYTHON-5912

Changes in this PR

See ticket.

Test Plan

Test changes. Note that we already skip the substringPreview prose tests on server version 9.0+.

Checklist

Checklist for Author

  • [ ] Did you update the changelog (if necessary)?
  • Is there test coverage?
  • [ ] Is any followup work tracked in a JIRA ticket? If so, add link(s).

Checklist for Reviewer

  • Does the title of the PR reference a JIRA Ticket?
  • Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
  • Is all relevant documentation (README or docstring) updated?

Copilot AI review requested due to automatic review settings July 1, 2026 16:05
@NoahStapp
NoahStapp requested a review from a team as a code owner July 1, 2026 16:05
@NoahStapp
NoahStapp requested a review from sleepyStick July 1, 2026 16:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the unified client-side encryption spec test for QE-Text-substringPreview to avoid running it on MongoDB server versions 9.0+ by constraining the server version in runOnRequirements.

Changes:

  • Add maxServerVersion: "8.99.99" to the test’s runOnRequirements so it only runs on servers < 9.0.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@NoahStapp
NoahStapp merged commit f7611d1 into mongodb:master Jul 1, 2026
88 checks passed
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.

4 participants