address review

Author: blink1073

test/__init__.py

@@ -678,7 +678,6 @@ def is_topology_type(self, topologies):
             "single",
             "replicaset",
             "sharded",
-            "sharded-replicaset",
             "load-balanced",
         }
         if unknown:
@@ -693,16 +692,6 @@ def is_topology_type(self, topologies):
             return True
         if "sharded" in topologies and self.is_mongos:
             return True
-        if "sharded-replicaset" in topologies and self.is_mongos:
-            shards = client_context.client.config.shards.find().to_list()
-            for shard in shards:
-                # For a 3-member RS-backed sharded cluster, shard['host']
-                # will be 'replicaName/ip1:port1,ip2:port2,ip3:port3'
-                # Otherwise it will be 'ip1:port1'
-                host_spec = shard["host"]
-                if not len(host_spec.split("/")) > 1:
-                    return False
-            return True
         return False
 
     def require_cluster_type(self, topologies=None):

test/asynchronous/__init__.py

@@ -680,7 +680,6 @@ async def is_topology_type(self, topologies):
             "single",
             "replicaset",
             "sharded",
-            "sharded-replicaset",
             "load-balanced",
         }
         if unknown:
@@ -695,16 +694,6 @@ async def is_topology_type(self, topologies):
             return True
         if "sharded" in topologies and self.is_mongos:
             return True
-        if "sharded-replicaset" in topologies and self.is_mongos:
-            shards = await async_client_context.client.config.shards.find().to_list()
-            for shard in shards:
-                # For a 3-member RS-backed sharded cluster, shard['host']
-                # will be 'replicaName/ip1:port1,ip2:port2,ip3:port3'
-                # Otherwise it will be 'ip1:port1'
-                host_spec = shard["host"]
-                if not len(host_spec.split("/")) > 1:
-                    return False
-            return True
         return False
 
     def require_cluster_type(self, topologies=None):

test/discovery_and_monitoring/unified/serverMonitoringMode.json

@@ -5,8 +5,7 @@
     {
       "topologies": [
         "single",
-        "sharded",
-        "sharded-replicaset"
+        "sharded"
       ],
       "serverless": "forbid"
     }

test/run_command/unified/runCommand.json

@@ -229,7 +229,6 @@
         {
           "topologies": [
             "replicaset",
-            "sharded-replicaset",
             "load-balanced",
             "sharded"
           ]
@@ -493,7 +492,7 @@
         {
           "minServerVersion": "4.2",
           "topologies": [
-            "sharded-replicaset",
+            "sharded",
             "load-balanced"
           ]
         }