CSHARP-5939: Remove duplicated test cases (#1922)

sanych-sun · web-flow · commit 490be9ceb364 · 2026-03-23T09:09:30.000-07:00
diff --git a/tests/MongoDB.Driver.Tests/ClientSessionHandleTests.cs b/tests/MongoDB.Driver.Tests/ClientSessionHandleTests.cs
@@ -487,8 +487,6 @@ public void WithTransaction_callback_with_a_UnknownTransactionCommitResult_shoul
         [InlineData(new[] { false, false }, new[] { WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.NoError }, false /*Should exception be thrown*/, 1, false)]
         [InlineData(new[] { false, true }, new[] { WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.UnknownTransactionCommitResult }, true /*Should exception be thrown*/, 1, false)]
 
-        [InlineData(new[] { false, false }, new[] { WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.NoError }, false /*Should exception be thrown*/, 1, false)]
-
         // async
         [InlineData(null, new[] { WithTransactionErrorState.NoError }, false /*Should exception be thrown*/, 1, true)]
         [InlineData(null, new[] { WithTransactionErrorState.TransientTransactionError, WithTransactionErrorState.NoError }, false /*Should exception be thrown*/, 2, true)]
@@ -500,8 +498,6 @@ public void WithTransaction_callback_with_a_UnknownTransactionCommitResult_shoul
 
         [InlineData(new[] { false, false }, new[] { WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.NoError }, false /*Should exception be thrown*/, 1, true)]
         [InlineData(new[] { false, true }, new[] { WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.UnknownTransactionCommitResult }, true /*Should exception be thrown*/, 1, true)]
-
-        [InlineData(new[] { false, false }, new[] { WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.UnknownTransactionCommitResult, WithTransactionErrorState.NoError }, false /*Should exception be thrown*/, 1, true)]
         public void WithTransaction_commit_after_callback_processing_should_be_processed_with_expected_result(
             bool[] isRetryAttemptsWithTimeout, // the array length should be the same with a number of failed attempts from `commitTransactionErrorStates`
             WithTransactionErrorState[] commitTransactionErrorStates,
diff --git a/tests/MongoDB.Driver.Tests/Core/Clusters/ClusterFactoryTests.cs b/tests/MongoDB.Driver.Tests/Core/Clusters/ClusterFactoryTests.cs
@@ -47,7 +47,6 @@ public void ClusterFactory_should_create_cluster_when_loggerfactory_is_not_set()
 
         [Theory]
         [InlineData("mongodb://a.MONGO.COSMOS.AZURE.COM:19555", ExpectedCosmosDBMessage)]
-        [InlineData("mongodb://a.MONGO.COSMOS.AZURE.COM:19555", ExpectedCosmosDBMessage)]
         [InlineData("mongodb://a.mongo.cosmos.azure.com:19555", ExpectedCosmosDBMessage)]
         [InlineData("mongodb://a.DOCDB-ELASTIC.AMAZONAWS.COM:27017/", ExpectedDocumentDBMessage)]
         [InlineData("mongodb://a.docdb-elastic.amazonaws.com:27017/", ExpectedDocumentDBMessage)]
diff --git a/tests/MongoDB.Driver.Tests/Core/Clusters/MultiServerClusterTests.cs b/tests/MongoDB.Driver.Tests/Core/Clusters/MultiServerClusterTests.cs
@@ -776,14 +776,6 @@ public void Should_not_add_a_new_server_from_a_secondary_when_a_primary_exists()
         [InlineData(ServerType.ShardRouter, ServerType.ReplicaSetPrimary)]
         [InlineData(ServerType.ShardRouter, ServerType.ReplicaSetSecondary)]
         [InlineData(ServerType.ShardRouter, ServerType.Standalone)]
-        [InlineData(ServerType.ReplicaSetPrimary, ServerType.ShardRouter)]
-        [InlineData(ServerType.ReplicaSetPrimary, ServerType.Standalone)]
-        [InlineData(ServerType.ShardRouter, ServerType.ReplicaSetArbiter)]
-        [InlineData(ServerType.ShardRouter, ServerType.ReplicaSetGhost)]
-        [InlineData(ServerType.ShardRouter, ServerType.ReplicaSetOther)]
-        [InlineData(ServerType.ShardRouter, ServerType.ReplicaSetPrimary)]
-        [InlineData(ServerType.ShardRouter, ServerType.ReplicaSetSecondary)]
-        [InlineData(ServerType.ShardRouter, ServerType.Standalone)]
         public void Should_hide_a_seedlist_server_of_the_wrong_type(ServerType initialType, ServerType wrongType)
         {
             _settings = _settings.With(
diff --git a/tests/MongoDB.Driver.Tests/Core/Clusters/SingleServerClusterTests.cs b/tests/MongoDB.Driver.Tests/Core/Clusters/SingleServerClusterTests.cs
@@ -98,7 +98,6 @@ public void Initialize_should_create_and_initialize_the_server()
         [InlineData(ServerType.ReplicaSetOther)]
         [InlineData(ServerType.ReplicaSetPrimary)]
         [InlineData(ServerType.ReplicaSetSecondary)]
-        [InlineData(ServerType.ShardRouter)]
         public void Description_should_contain_any_new_server(ServerType serverType)
         {
             var subject = CreateSubject();
diff --git a/tests/MongoDB.Driver.Tests/Core/Connections/BinaryConnection_CommandEventTests.cs b/tests/MongoDB.Driver.Tests/Core/Connections/BinaryConnection_CommandEventTests.cs
@@ -62,7 +62,6 @@ public static IEnumerable<object[]> GetPotentiallyRedactedCommandTestCases()
                 new object[] { "{ updateUser : 1 }", true },
                 new object[] { "{ copydbsaslstart : 1 }", true },
                 new object[] { "{ copydb : 1 }", true },
-                new object[] { "{ authenticate : 1 }", true },
                 new object[] { "{ hello : 1, helloOk : true }", false },
                 new object[] { "{ hello : 1, helloOk : true, speculativeAuthenticate : { } }", true },
                 new object[] { $"{{ {OppressiveLanguageConstants.LegacyHelloCommandName} : 1, helloOk : true }}", false },
diff --git a/tests/MongoDB.Driver.Tests/Core/Connections/HelloResultTests.cs b/tests/MongoDB.Driver.Tests/Core/Connections/HelloResultTests.cs
@@ -250,7 +250,6 @@ public void SaslSupportedMechs_should_parse_document_correctly(string json, stri
         [InlineData("{ ok: 1, isreplicaset: false }", ServerType.Standalone)]
         [InlineData("{ ok: 1, isreplicaset: 0 }", ServerType.Standalone)]
         [InlineData("{ ok: 1, msg: \"isdbgrid\" }", ServerType.ShardRouter)]
-        [InlineData("{ ok: 1, msg: \"isdbgrid\" }", ServerType.ShardRouter)]
         [InlineData("{ ok: 1, serviceId: ObjectId('111111111111111111111111') }", ServerType.LoadBalanced)]
         [InlineData("{ ok: 1 }", ServerType.Standalone)]
         [InlineData("{ ok: 0 }", ServerType.Unknown)]
diff --git a/tests/MongoDB.Driver.Tests/Core/Misc/RangeTests.cs b/tests/MongoDB.Driver.Tests/Core/Misc/RangeTests.cs
@@ -37,7 +37,6 @@ public void Constructor_should_throw_when_min_is_greater_than_max()
         [Theory]
         [InlineData(0, 0, 0, 0, true)]
         [InlineData(0, 1, 0, 1, true)]
-        [InlineData(0, 0, 0, 0, true)]
         [InlineData(0, 1, 0, 0, false)]
         [InlineData(0, 0, 0, 1, false)]
         public void Equals_should_return_correct_value(int a1, int a2, int b1, int b2, bool expected)
@@ -51,7 +50,6 @@ public void Equals_should_return_correct_value(int a1, int a2, int b1, int b2, b
         [Theory]
         [InlineData(0, 0, 0, 0, true)]
         [InlineData(0, 1, 0, 0, true)]
-        [InlineData(0, 1, 0, 0, true)]
         [InlineData(0, 1, 1, 1, true)]
         [InlineData(0, 0, 0, 1, true)]
         [InlineData(1, 1, 0, 1, true)]
diff --git a/tests/MongoDB.Driver.Tests/Core/MongoCommandExceptionTests.cs b/tests/MongoDB.Driver.Tests/Core/MongoCommandExceptionTests.cs
@@ -79,7 +79,6 @@ public void constructor_with_message_command_result_should_initialize_subject()
         [Theory]
         [InlineData(null, new string[0])]
         [InlineData("{ }", new string[0])]
-        [InlineData("{ }", new string[0])]
         [InlineData("{ errorLabels : \"not an array\" }", new string[0])]
         [InlineData("{ errorLabels : [ ] }", new string[0])]
         [InlineData("{ errorLabels : [ 1 ] }", new string[0])]
diff --git a/tests/MongoDB.Driver.Tests/Core/Operations/RetryableWriteOperationExecutorTests.cs b/tests/MongoDB.Driver.Tests/Core/Operations/RetryableWriteOperationExecutorTests.cs
@@ -56,7 +56,6 @@ public void AreRetryableWritesSupportedTest()
         [InlineData(true, true, false, false, false)]
         [InlineData(true, true, false, true, false)]
         [InlineData(true, true, true, false, true)]
-        [InlineData(true, true, true, false, true)]
         public void DoesContextAllowRetries_should_return_expected_result(
             bool retryRequested,
             bool areRetryableWritesSupported,
diff --git a/tests/MongoDB.Driver.Tests/MongoCredentialTests.cs b/tests/MongoDB.Driver.Tests/MongoCredentialTests.cs
@@ -20,7 +20,6 @@
 
 namespace MongoDB.Driver.Tests
 {
-    [Trait("Category", "Integration")]
     public class MongoCredentialTests
     {
         [Fact]
diff --git a/tests/MongoDB.Driver.Tests/MongoUrlBuilderTests.cs b/tests/MongoDB.Driver.Tests/MongoUrlBuilderTests.cs
@@ -1216,7 +1216,6 @@ public void TestUseTls(bool? useTls, string formatString, string[] values)
 
         [Theory]
         [InlineData(false, false, 0, "mongodb://localhost/?w=0")]
-        [InlineData(false, false, 0, "mongodb://localhost/?w=0")]
         [InlineData(false, true, 1, "mongodb://localhost/?w=1")]
         [InlineData(false, true, 2, "mongodb://localhost/?w=2")]
         [InlineData(false, true, "mode", "mongodb://localhost/?w=mode")]
diff --git a/tests/MongoDB.Driver.Tests/MongocryptdFactoryTests.cs b/tests/MongoDB.Driver.Tests/MongocryptdFactoryTests.cs
@@ -68,7 +68,6 @@ public void ShouldMongocryptdBeSpawned_should_correctly_handle_bypassQueryAnalys
         [InlineData("{ mongocryptdBypassSpawn : false }", "mongocryptd#extension#", "--idleShutdownTimeoutSecs 60 --logpath #logpath# --logappend", true)]
         [InlineData("{ mongocryptdBypassSpawn : false, mongocryptdSpawnPath : 'c:/mongocryptd.exe' }", "c:/mongocryptd.exe", "--idleShutdownTimeoutSecs 60 --logpath #logpath# --logappend", true)]
         [InlineData("{ mongocryptdBypassSpawn : false, mongocryptdSpawnPath : 'c:/mgcr.exe' }", "c:/mgcr.exe", "--idleShutdownTimeoutSecs 60 --logpath #logpath# --logappend", true)]
-        [InlineData("{ mongocryptdBypassSpawn : false, mongocryptdSpawnPath : 'c:/mgcr.exe' }", "c:/mgcr.exe", "--idleShutdownTimeoutSecs 60 --logpath #logpath# --logappend", true)]
         // args string
         [InlineData("{ mongocryptdSpawnArgs : '--arg1 A --arg2 B' }", "mongocryptd#extension#", "--arg1 A --arg2 B --idleShutdownTimeoutSecs 60 --logpath #logpath# --logappend", true)]
         [InlineData("{ mongocryptdSpawnArgs : '--arg1 A --arg2 B --idleShutdownTimeoutSecs 50' }", "mongocryptd#extension#", "--arg1 A --arg2 B --idleShutdownTimeoutSecs 50 --logpath #logpath# --logappend", true)]
diff --git a/tests/MongoDB.Driver.Tests/Search/SearchRangeBuilderTests.cs b/tests/MongoDB.Driver.Tests/Search/SearchRangeBuilderTests.cs
@@ -38,7 +38,7 @@ public void SearchRange_ctor_should_throw_on_invalid_arguments(int min, int max,
         [InlineData(0, 1, true, true)]
         [InlineData(0, 1, false, true)]
         [InlineData(0, 1, true, false)]
-        [InlineData(0, 1, true, true)]
+        [InlineData(0, 1, false, false)]
         public void SearchRange_ctor_should_construct_valid_instance(int? min, int? max, bool minInclusive, bool maxInclusive)
         {
             var subject = new SearchRange<int>(min, max, minInclusive, maxInclusive);
@@ -66,7 +66,7 @@ public void SearchRangeBuilder_should_throw_on_invalid_arguments(int min, int ma
         [InlineData(0, 1, true, true)]
         [InlineData(0, 1, false, true)]
         [InlineData(0, 1, true, false)]
-        [InlineData(0, 1, true, true)]
+        [InlineData(0, 1, false, false)]
         public void SearchRangeBuilder_should_return_valid_instance(int? min, int? max, bool minInclusive, bool maxInclusive)
         {
             SearchRange<int>? subject = null;
diff --git a/tests/MongoDB.Driver.Tests/Search/SearchRangeV2BuilderTests.cs b/tests/MongoDB.Driver.Tests/Search/SearchRangeV2BuilderTests.cs
@@ -28,7 +28,7 @@ public class SearchRangeV2BuilderTests
         [InlineData(0, 1, true, true)]
         [InlineData(0, 1, false, true)]
         [InlineData(0, 1, true, false)]
-        [InlineData(0, 1, true, true)]
+        [InlineData(0, 1, false, false)]
         public void SearchRangeV2Builder_should_return_valid_instance(int? min, int? max, bool minInclusive, bool maxInclusive)
         {
             SearchRangeV2<int>? subject = null;
@@ -56,4 +56,4 @@ public void SearchRangeV2Builder_should_return_valid_instance(int? min, int? max
             subject.Value.Max?.Inclusive.Should().Be(maxInclusive);
         }
     }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,6 @@ public void Initialize_should_create_and_initialize_the_server()`
`98`	`98`	`[InlineData(ServerType.ReplicaSetOther)]`
`99`	`99`	`[InlineData(ServerType.ReplicaSetPrimary)]`
`100`	`100`	`[InlineData(ServerType.ReplicaSetSecondary)]`
`101`		`- [InlineData(ServerType.ShardRouter)]`
`102`	`101`	`public void Description_should_contain_any_new_server(ServerType serverType)`
`103`	`102`	`{`
`104`	`103`	`var subject = CreateSubject();`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,6 @@`
`20`	`20`
`21`	`21`	`namespace MongoDB.Driver.Tests`
`22`	`22`	`{`
`23`		`- [Trait("Category", "Integration")]`
`24`	`23`	`public class MongoCredentialTests`
`25`	`24`	`{`
`26`	`25`	`[Fact]`