address comments

Author: sleepyStick

pymongo/asynchronous/mongo_client.py

@@ -1234,20 +1234,14 @@ def eq_props(self) -> tuple[tuple[_Address, ...], Optional[str], Optional[str],
 
     def __eq__(self, other: Any) -> bool:
         if isinstance(other, self.__class__):
-            if self._topology is None:
-                return self.eq_props() == other.eq_props()
-            else:
-                return self._topology == other._topology
+            return self.eq_props() == other.eq_props()
         return NotImplemented
 
     def __ne__(self, other: Any) -> bool:
         return not self == other
 
     def __hash__(self) -> int:
-        if self._topology is None:
-            return hash(self.eq_props())
-        else:
-            return hash(self._topology)
+        return hash(self.eq_props())
 
     def _repr_helper(self) -> str:
         def option_repr(option: str, value: Any) -> str:
@@ -1264,7 +1258,7 @@ def option_repr(option: str, value: Any) -> str:
 
         # Host first...
         if self._topology is None:
-            options = ["host={self._host}", "port={self._port}"]
+            options = self._resolve_srv_info["seeds"]
         else:
             options = [
                 "host=%r"

pymongo/asynchronous/uri_parser.py

@@ -1,3 +1,19 @@
+# Copyright 2011-present MongoDB, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License"); you
+# may not use this file except in compliance with the License.  You
+# may obtain a copy of the License at
+#
+# https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+# implied.  See the License for the specific language governing
+# permissions and limitations under the License.
+
+
+"""Tools to parse and validate a MongoDB URI."""
 from __future__ import annotations
 
 from typing import Any, Optional

pymongo/srv_resolver.py

@@ -1232,20 +1232,14 @@ def eq_props(self) -> tuple[tuple[_Address, ...], Optional[str], Optional[str],
 
     def __eq__(self, other: Any) -> bool:
         if isinstance(other, self.__class__):
-            if self._topology is None:
-                return self.eq_props() == other.eq_props()
-            else:
-                return self._topology == other._topology
+            return self.eq_props() == other.eq_props()
         return NotImplemented
 
     def __ne__(self, other: Any) -> bool:
         return not self == other
 
     def __hash__(self) -> int:
-        if self._topology is None:
-            return hash(self.eq_props())
-        else:
-            return hash(self._topology)
+        return hash(self.eq_props())
 
     def _repr_helper(self) -> str:
         def option_repr(option: str, value: Any) -> str:
@@ -1262,7 +1256,7 @@ def option_repr(option: str, value: Any) -> str:
 
         # Host first...
         if self._topology is None:
-            options = ["host={self._host}", "port={self._port}"]
+            options = self._resolve_srv_info["seeds"]
         else:
             options = [
                 "host=%r"

pymongo/synchronous/mongo_client.py

@@ -1,3 +1,19 @@
+# Copyright 2011-present MongoDB, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License"); you
+# may not use this file except in compliance with the License.  You
+# may obtain a copy of the License at
+#
+# https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+# implied.  See the License for the specific language governing
+# permissions and limitations under the License.
+
+
+"""Tools to parse and validate a MongoDB URI."""
 from __future__ import annotations
 
 from typing import Any, Optional