TOOLS-4148 Add a new sharded test type for sharded cluster integration tests by autarch · Pull Request #916 · mongodb/mongo-tools

autarch · 2026-03-13T20:48:28Z

This adds a new TOOLS_TESTING_SHARDED_INTEGRATION test type, wires it up in the build script via -topology=sharded, adds a "create sharded cluster" Evergreen function (mirroring "create repl_set" but using ShardingTest), and adds integration-X.Y-sharded tasks for all server versions that have cluster tasks (4.4 through latest).

This will be used as I convert JS tests to Go. Some of those tests are specific to sharded clusters, so we need a way to write Go tests against sharded clusters too.

This PR also includes the plan that Claude created for the conversion of all JS tests.

autarch · 2026-03-13T20:48:42Z

TOOLS-4148 Add a new sharded test type for sharded cluster integration tests #916 : 2 dependent PRs (#922 , #956 ) 👈 (View in Graphite)
Fix linting ignore rule for bson types like bson.D #926
Add an AGENTS.md file for this repo #924
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

FGasper

AFAICT this is fine. It does contextualize a bit why (in the next PR) AI wants to skip the SIGPIPE tests. I still hesitate to remove coverage entirely, though, and wonder if we can’t preserve an OS-level test that builds the tool in question & runs it from Go.

FGasper · 2026-03-24T13:36:30Z

.ai-plans/2026-03-13/js-to-go-test-migration/plan.md

+    defer client.Disconnect(context.Background())
+
+    db := client.Database("mongofoo_test_db")
+    defer db.Drop(context.Background()) // clean up after the test


Should use t.Context() instead.

FGasper · 2026-03-24T13:37:47Z

.ai-plans/2026-03-13/js-to-go-test-migration/plan.md

+
+    t.Run("does the thing", func(t *testing.T) {
+        // Insert test data
+        _, err := db.Collection("coll").InsertOne(context.Background(), bson.D{{"k", "v"}})


t.Context()

FGasper · 2026-03-24T13:38:30Z

.ai-plans/2026-03-13/js-to-go-test-migration/plan.md

+
+        // Assert outcomes
+        var result bson.D
+        err = db.Collection("coll").FindOne(context.Background(), bson.D{}).Decode(&result)


autarch · 2026-03-25T20:31:59Z

Merge activity

Mar 25, 8:31 PM UTC: A user started a stack merge that includes this pull request via Graphite.
Mar 25, 8:35 PM UTC: The Graphite merge of this pull request was cancelled.
Mar 25, 8:54 PM UTC: A user started a stack merge that includes this pull request via Graphite.
Mar 25, 8:55 PM UTC: Graphite couldn't merge this pull request because a downstack PR Add an AGENTS.md file for this repo #924 failed to merge.
Mar 25, 9:08 PM UTC: A user started a stack merge that includes this pull request via Graphite.
Mar 25, 9:08 PM UTC: Graphite couldn't merge this pull request because a downstack PR Add an AGENTS.md file for this repo #924 failed to merge.
Mar 25, 9:40 PM UTC: A user started a stack merge that includes this pull request via Graphite.
Mar 26, 12:39 AM UTC: Graphite rebased this pull request as part of a merge.
Mar 26, 1:10 AM UTC: @autarch merged this pull request with Graphite.

This adds anew `TOOLS_TESTING_SHARDED_INTEGRATION` test type, wires it up in the build script via -topology=sharded, adds a "create sharded cluster" Evergreen function (mirroring "create repl_set" but using ShardingTest), and adds integration-X.Y-sharded tasks for all server versions that have cluster tasks (4.4 through latest). Co-Authored-By: Claude Code <noreply@anthropic.com>

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch 3 times, most recently from ad10d8f to f8d80ef Compare March 19, 2026 17:09

This was referenced Mar 19, 2026

TOOLS-4148 Annotate JS tests with SKIP, EXTEND, or NEW after auditing to figure out which ones need to be converted to Go #922

Closed

TOOLS-4148 Rewrite all the bsondump tests in Go and delete the JS tests for this #923

Open

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from f8d80ef to b8c834c Compare March 20, 2026 16:53

autarch changed the base branch from master to graphite-base/916 March 20, 2026 18:14

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from b8c834c to e16e09e Compare March 20, 2026 18:14

autarch changed the base branch from graphite-base/916 to 03-20-add_an_agents.md_file_for_this_repo March 20, 2026 18:14

autarch mentioned this pull request Mar 20, 2026

Add an AGENTS.md file for this repo #924

Merged

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch 2 times, most recently from 162b6ad to a3170ff Compare March 20, 2026 18:31

autarch force-pushed the 03-20-add_an_agents.md_file_for_this_repo branch from 55feed5 to 057c6bb Compare March 20, 2026 18:31

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from a3170ff to e787652 Compare March 20, 2026 20:34

autarch force-pushed the 03-20-add_an_agents.md_file_for_this_repo branch from 057c6bb to 22b912b Compare March 20, 2026 20:34

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from e787652 to 7a9eeeb Compare March 20, 2026 20:35

This was referenced Mar 20, 2026

TOOLS-4148 Convert test/qa-tests/jstests/export/basic_data.js to Go #925

Open

Fix linting ignore rule for bson types like bson.D #926

Merged

autarch changed the base branch from 03-20-add_an_agents.md_file_for_this_repo to graphite-base/916 March 20, 2026 21:26

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from 7a9eeeb to fbdcd63 Compare March 20, 2026 21:26

autarch changed the base branch from graphite-base/916 to 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ March 20, 2026 21:26

autarch mentioned this pull request Mar 20, 2026

TOOLS-4148 Convert test/qa-tests/jstests/export/data_types.js to Go #927

Open

autarch marked this pull request as ready for review March 20, 2026 21:27

autarch requested a review from a team as a code owner March 20, 2026 21:27

autarch requested review from FGasper and nickweinberger and removed request for a team and nickweinberger March 20, 2026 21:27

autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from 4e37557 to c1284b7 Compare March 23, 2026 15:07

FGasper approved these changes Mar 24, 2026

View reviewed changes

autarch changed the title ~~Add a new sharded test type for sharded cluster integration tests~~ TOOLS-4148 Add a new sharded test type for sharded cluster integration tests Mar 24, 2026

autarch mentioned this pull request Mar 24, 2026

TOOLS-4148 Convert test/qa-tests/jstests/import/import_document_validation.js to Go #953

Open

autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from 09a82fc to 88df719 Compare March 25, 2026 16:50

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from 493873b to 21e4728 Compare March 25, 2026 16:50

autarch mentioned this pull request Mar 25, 2026

TOOLS-4148 Add tests for SIGPIPE handling, remove equivalent JS tests, and fix handling for other signals #956

Open

autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from 88df719 to c004698 Compare March 25, 2026 16:59

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from 21e4728 to 1085693 Compare March 25, 2026 16:59

autarch force-pushed the 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ branch from c004698 to b7e4b4c Compare March 25, 2026 17:15

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from 1085693 to 766992b Compare March 25, 2026 17:15

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from 766992b to aede66b Compare March 25, 2026 20:47

autarch changed the base branch from 03-20-fix_linting_ignore_rule_for_bson_types_like_bson.d_ to graphite-base/916 March 25, 2026 23:49

autarch changed the base branch from graphite-base/916 to master March 26, 2026 00:38

autarch force-pushed the 03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests branch from aede66b to 032769f Compare March 26, 2026 00:39

autarch merged commit 48c54ef into master Mar 26, 2026
40 checks passed

This was referenced Mar 26, 2026

TOOLS-4148 Convert test/qa-tests/jstests/import/mongoimport mode.js to Go #960

Open

TOOLS-4148 Convert mode_upsert_id_subdoc.js to Go #964

Draft

Add a Claude skill for converting JS tests to Go #965

Draft

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

TOOLS-4148 Add a new sharded test type for sharded cluster integration tests#916

TOOLS-4148 Add a new sharded test type for sharded cluster integration tests#916
autarch merged 1 commit intomasterfrom
03-13-add_a_new_sharded_test_type_for_sharded_cluster_integration_tests

autarch commented Mar 13, 2026 •

edited

Loading

Uh oh!

autarch commented Mar 13, 2026 •

edited

Loading

Uh oh!

FGasper left a comment

Uh oh!

FGasper Mar 24, 2026

Uh oh!

FGasper Mar 24, 2026

Uh oh!

FGasper Mar 24, 2026

Uh oh!

autarch commented Mar 25, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

autarch commented Mar 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

autarch commented Mar 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

FGasper left a comment

Choose a reason for hiding this comment

Uh oh!

FGasper Mar 24, 2026

Choose a reason for hiding this comment

Uh oh!

FGasper Mar 24, 2026

Choose a reason for hiding this comment

Uh oh!

FGasper Mar 24, 2026

Choose a reason for hiding this comment

Uh oh!

autarch commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

autarch commented Mar 13, 2026 •

edited

Loading

autarch commented Mar 13, 2026 •

edited

Loading

autarch commented Mar 25, 2026 •

edited

Loading