Skip to content

RUBY-3821 Remove pre-4.2 spec test fixtures - #3117

Merged
jamis merged 1 commit into
mongodb:masterfrom
jamis:3821-remove-pre-4.2-fixtures
Oct 22, 2026
Merged

jamis merged 1 commit into
mongodb:masterfrom
jamis:3821-remove-pre-4.2-fixtures

Conversation

@jamis

@jamis jamis commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Syncs spec/spec_tests/data with the specifications repo's removal of pre-MongoDB 4.2 tests.

Deletes spec files that are gated behind server version 4.2 and earlier (28 files). Also removes individual tests from 11 more files which were gated behind server version 4.2 and earlier.

Sync spec/spec_tests/data with the specifications repo's removal of
pre-MongoDB 4.2 tests (DRIVERS-3439). Since the driver no longer
supports 4.2 or older, fixtures gated to servers below 4.2 are dead.

Delete 28 wholly-dead fixtures: the 14 crud_unified hint, arrayFilters
and allowDiskUse clientError/serverError files, the 8 pre-42 SDAM
error-handling files, the 4 legacy crud/write *-pre_2.6 files, the
pre-42 command-monitoring fixture, and the CSFLE maxWireVersion
fixture.

Remove pre-4.2-gated tests from 11 mixed files: the dots_and_dollars
and hint-unacknowledged CRUD files, and the copydb cases in
redacted-commands. Where a removed test defined the &filter, &update or
&replacement anchors that surviving tests referenced, the definitions
move to the first surviving test.

The unified and legacy CRUD suites lose only skipped examples: those
tests were already being skipped on a supported (4.4+) server. The SDAM
suite drops 160 executing examples, all from the pre-42 mock fixtures
that simulate a pre-4.2 (wire version 7) server.
@jamis
jamis requested a review from a team as a code owner October 16, 2026 17:40
@jamis
jamis requested a review from comandeo-mongo October 16, 2026 17:40
@jamis
jamis merged commit f865ed9 into mongodb:master Oct 22, 2026
241 checks passed
@jamis
jamis deleted the 3821-remove-pre-4.2-fixtures branch October 22, 2026 14:30
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