Skip to content

Commit ffd0d65

Browse files
authored
Skip snapshot sessions tests that require the new getSnapshotTime op. (#2266)
1 parent ef6677f commit ffd0d65

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

internal/spectest/skip.go

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -241,12 +241,6 @@ var skipTests = map[string][]string{
241241
"TestUnifiedSpec/server-discovery-and-monitoring/tests/unified/interruptInUse-pool-clear.json/Error_returned_from_connection_pool_clear_with_interruptInUseConnections=true_is_retryable_for_write",
242242
},
243243

244-
// TODO(GODRIVER-2843): Fix and unskip these test cases.
245-
"Test fails frequently, needs fixing (GODRIVER-2843)": {
246-
"TestUnifiedSpec/sessions/snapshot-sessions.json/Find_operation_with_snapshot",
247-
"TestUnifiedSpec/sessions/snapshot-sessions.json/Write_commands_with_snapshot_session_do_not_affect_snapshot_reads",
248-
},
249-
250244
// TODO(GODRIVER-3043): Avoid Appending Write/Read Concern in Atlas Search
251245
// Index Helper Commands.
252246
"Sync tests but avoid write/read concern bug (GODRIVER-3043)": {
@@ -858,6 +852,15 @@ var skipTests = map[string][]string{
858852
"TestSDAMSpec/errors/pre-42-PrimarySteppedDown.json",
859853
"TestSDAMSpec/errors/pre-42-ShutdownInProgress.json",
860854
},
855+
856+
// TODO(GODRIVER-3663): Expose atClusterTime parameter in snapshot sessions
857+
"Expose atClusterTime parameter in snapshot sessions (GODRIVER-3663)": {
858+
"TestUnifiedSpec/sessions/tests/snapshot-sessions.json/Find_operation_with_snapshot_and_snapshot_time",
859+
"TestUnifiedSpec/sessions/tests/snapshot-sessions.json/Distinct_operation_with_snapshot_and_snapshot_time",
860+
"TestUnifiedSpec/sessions/tests/snapshot-sessions.json/Aggregate_operation_with_snapshot_and_snapshot_time",
861+
"TestUnifiedSpec/sessions/tests/snapshot-sessions.json/countDocuments_operation_with_snapshot_and_snapshot_time",
862+
"TestUnifiedSpec/sessions/tests/snapshot-sessions.json/Mixed_operation_with_snapshot_and_snapshotTime",
863+
},
861864
}
862865

863866
// CheckSkip checks if the fully-qualified test name matches a list of skipped test names for a given reason.

0 commit comments

Comments
 (0)