Skip to content

test(NODE-7656): Skip time-sensitive CSOT tests opening change streams on 9.0+ sharded clusters#5002

Open
PavelSafronov wants to merge 1 commit into
mainfrom
node-7656
Open

test(NODE-7656): Skip time-sensitive CSOT tests opening change streams on 9.0+ sharded clusters#5002
PavelSafronov wants to merge 1 commit into
mainfrom
node-7656

Conversation

@PavelSafronov

Copy link
Copy Markdown
Contributor

Description

Summary of Changes

Skip CSOT tests against 9.0+ server.

Double check the following

  • Lint is passing (npm run check:lint)
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@PavelSafronov
PavelSafronov marked this pull request as ready for review July 13, 2026 19:54
@PavelSafronov
PavelSafronov requested a review from a team as a code owner July 13, 2026 19:54
Copilot AI review requested due to automatic review settings July 13, 2026 19:54

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 adjusts the Client-Side Operations Timeout (CSOT) unified spec fixtures to skip specific change stream-opening tests on sharded clusters running MongoDB 9.0+ by tightening runOnRequirements for those test cases.

Changes:

  • Added per-test runOnRequirements to allow these change stream CSOT tests on replica sets, while restricting sharded execution to maxServerVersion: 8.99.
  • Updated both YAML and JSON representations of the affected CSOT spec suites to keep them in sync.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
test/spec/client-side-operations-timeout/retryability-timeoutMS.yml Adds topology/version gating to skip sharded 9.0+ for retryability change stream CSOT tests
test/spec/client-side-operations-timeout/retryability-timeoutMS.json Mirrors the same runOnRequirements changes in the JSON fixture consumed by the spec runner
test/spec/client-side-operations-timeout/override-operation-timeoutMS.yml Adds topology/version gating to skip sharded 9.0+ for override-timeout change stream CSOT tests
test/spec/client-side-operations-timeout/override-operation-timeoutMS.json Mirrors the same runOnRequirements changes in the JSON fixture consumed by the spec runner

Comment thread test/spec/client-side-operations-timeout/retryability-timeoutMS.yml
@tadjik1 tadjik1 self-assigned this Jul 14, 2026
@tadjik1 tadjik1 added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Jul 14, 2026

@tadjik1 tadjik1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @PavelSafronov! Please check the configuration for these tests, we already skip them.

@PavelSafronov

Copy link
Copy Markdown
Contributor Author

Thanks @PavelSafronov! Please check the configuration for these tests, we already skip them.

Excellent point. I removed the code skips and found that we need some more tests skipped in our specifications. PR opened to add these: mongodb/specifications#1964
Will update this PR after the specifications update is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Primary Review In Review with primary reviewer, not yet ready for team's eyes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants