switch! rm unused args

Author: Thomas Reggi

lib/sdam/topology.js

@@ -131,12 +131,11 @@ class Topology extends EventEmitter {
       if (options[optionName]) {
         emitDeprecationWarning(
           `The option \`${optionName}\` is incompatible with the unified topology, please read more by visiting http://bit.ly/2D8WfT6`,
-          'DeprecationWarning'
         );
       }
     });
 
-    const topologyType = topologyTypeFromSeedlist(seedlist, options);
+    const topologyType = topologyTypeFromSeedlist(options);
     const topologyId = globalTopologyCounter++;
     const serverDescriptions = seedlist.reduce((result, seed) => {
       if (seed.domain_socket) seed.host = seed.domain_socket;
@@ -797,7 +796,7 @@ function parseStringSeedlist(seedlist) {
   }));
 }
 
-function topologyTypeFromSeedlist(seedlist, options) {
+function topologyTypeFromSeedlist(options) {
   if (options.directConnection) {
     return TopologyType.Single;
   }

lib/sdam/topology_description.js

@@ -277,23 +277,19 @@ class TopologyDescription {
 }
 
 function topologyTypeForServerType(serverType) {
-  if (serverType === ServerType.Standalone) {
-    return TopologyType.Single;
-  }
-
-  if (serverType === ServerType.Mongos) {
-    return TopologyType.Sharded;
-  }
-
-  if (serverType === ServerType.RSPrimary) {
-    return TopologyType.ReplicaSetWithPrimary;
-  }
-
-  if (serverType === ServerType.RSGhost || serverType === ServerType.Unknown) {
-    return TopologyType.Unknown;
+  switch (serverType) {
+    case ServerType.Standalone:
+      return TopologyType.Single;
+    case ServerType.Mongos:
+      return TopologyType.Sharded;
+    case ServerType.RSPrimary:
+      return TopologyType.ReplicaSetWithPrimary;
+    case ServerType.RSGhost:
+    case ServerType.Unknown:
+      return TopologyType.Unknown;
+    default:
+      return TopologyType.ReplicaSetNoPrimary;
   }
-
-  return TopologyType.ReplicaSetNoPrimary;
 }
 
 function compareObjectId(oid1, oid2) {