remove periods from end of error messages

Author: sleepyStick

bson/__init__.py

@@ -1386,7 +1386,7 @@ def is_valid(bson: bytes) -> bool:
     :param bson: the data to be validated
     """
     if not isinstance(bson, bytes):
-        raise TypeError(f"BSON data must be an instance of a subclass of bytes, not {type(bson)}.")
+        raise TypeError(f"BSON data must be an instance of a subclass of bytes, not {type(bson)}")
 
     try:
         _bson_to_dict(bson, DEFAULT_CODEC_OPTIONS)

bson/binary.py

@@ -290,7 +290,7 @@ def __new__(
         subtype: int = BINARY_SUBTYPE,
     ) -> Binary:
         if not isinstance(subtype, int):
-            raise TypeError(f"subtype must be an instance of int, not {type(subtype)}.")
+            raise TypeError(f"subtype must be an instance of int, not {type(subtype)}")
         if subtype >= 256 or subtype < 0:
             raise ValueError("subtype must be contained in [0, 256)")
         # Support any type that implements the buffer protocol.
@@ -321,7 +321,7 @@ def from_uuid(
         .. versionadded:: 3.11
         """
         if not isinstance(uuid, UUID):
-            raise TypeError(f"uuid must be an instance of uuid.UUID, not {type(uuid)}.")
+            raise TypeError(f"uuid must be an instance of uuid.UUID, not {type(uuid)}")
 
         if uuid_representation not in ALL_UUID_REPRESENTATIONS:
             raise ValueError(
@@ -470,7 +470,7 @@ def as_vector(self) -> BinaryVector:
         """
 
         if self.subtype != VECTOR_SUBTYPE:
-            raise ValueError(f"Cannot decode subtype {self.subtype} as a vector.")
+            raise ValueError(f"Cannot decode subtype {self.subtype} as a vector")
 
         position = 0
         dtype, padding = struct.unpack_from("<sB", self, position)

bson/code.py

@@ -56,7 +56,7 @@ def __new__(
         **kwargs: Any,
     ) -> Code:
         if not isinstance(code, str):
-            raise TypeError(f"code must be an instance of str, not {type(code)}.")
+            raise TypeError(f"code must be an instance of str, not {type(code)}")
 
         self = str.__new__(cls, code)
 
@@ -67,7 +67,7 @@ def __new__(
 
         if scope is not None:
             if not isinstance(scope, _Mapping):
-                raise TypeError(f"scope must be an instance of dict, not {type(scope)}.")
+                raise TypeError(f"scope must be an instance of dict, not {type(scope)}")
             if self.__scope is not None:
                 self.__scope.update(scope)  # type: ignore
             else:

bson/codec_options.py

@@ -402,12 +402,12 @@ def __new__(
                 )
             if not isinstance(unicode_decode_error_handler, str):
                 raise ValueError(
-                    f"unicode_decode_error_handler must be a string, not {type(unicode_decode_error_handler)}."
+                    f"unicode_decode_error_handler must be a string, not {type(unicode_decode_error_handler)}"
                 )
             if tzinfo is not None:
                 if not isinstance(tzinfo, datetime.tzinfo):
                     raise TypeError(
-                        f"tzinfo must be an instance of datetime.tzinfo, not {type(tzinfo)}."
+                        f"tzinfo must be an instance of datetime.tzinfo, not {type(tzinfo)}"
                     )
                 if not tz_aware:
                     raise ValueError("cannot specify tzinfo without also setting tz_aware=True")
@@ -416,7 +416,7 @@ def __new__(
 
             if not isinstance(type_registry, TypeRegistry):
                 raise TypeError(
-                    f"type_registry must be an instance of TypeRegistry, not {type(type_registry)}."
+                    f"type_registry must be an instance of TypeRegistry, not {type(type_registry)}"
                 )
 
             return tuple.__new__(

bson/dbref.py

@@ -56,9 +56,9 @@ def __init__(
         .. seealso:: The MongoDB documentation on `dbrefs <https://dochub.mongodb.org/core/dbrefs>`_.
         """
         if not isinstance(collection, str):
-            raise TypeError(f"collection must be an instance of str, not {type(collection)}.")
+            raise TypeError(f"collection must be an instance of str, not {type(collection)}")
         if database is not None and not isinstance(database, str):
-            raise TypeError(f"database must be an instance of str, not {type(database)}.")
+            raise TypeError(f"database must be an instance of str, not {type(database)}")
 
         self.__collection = collection
         self.__id = id

bson/decimal128.py

@@ -277,7 +277,7 @@ def from_bid(cls: Type[Decimal128], value: bytes) -> Decimal128:
             point in Binary Integer Decimal (BID) format).
         """
         if not isinstance(value, bytes):
-            raise TypeError(f"value must be an instance of bytes, not {type(value)}.")
+            raise TypeError(f"value must be an instance of bytes, not {type(value)}")
         if len(value) != 16:
             raise ValueError("value must be exactly 16 bytes")
         return cls((_UNPACK_64(value[8:])[0], _UNPACK_64(value[:8])[0]))  # type: ignore

bson/timestamp.py

@@ -58,9 +58,9 @@ def __init__(self, time: Union[datetime.datetime, int], inc: int) -> None:
                 time = time - offset
             time = int(calendar.timegm(time.timetuple()))
         if not isinstance(time, int):
-            raise TypeError(f"time must be an instance of int, not {type(time)}.")
+            raise TypeError(f"time must be an instance of int, not {type(time)}")
         if not isinstance(inc, int):
-            raise TypeError(f"inc must be an instance of int, not {type(inc)}.")
+            raise TypeError(f"inc must be an instance of int, not {type(inc)}")
         if not 0 <= time < UPPERBOUND:
             raise ValueError("time must be contained in [0, 2**32)")
         if not 0 <= inc < UPPERBOUND:

gridfs/asynchronous/grid_file.py

@@ -100,7 +100,7 @@ def __init__(self, database: AsyncDatabase, collection: str = "fs"):
         .. seealso:: The MongoDB documentation on `gridfs <https://dochub.mongodb.org/core/gridfs>`_.
         """
         if not isinstance(database, AsyncDatabase):
-            raise TypeError(f"database must be an instance of Database, not {type(database)}.")
+            raise TypeError(f"database must be an instance of Database, not {type(database)}")
 
         database = _clear_entity_type_registry(database)
 
@@ -503,7 +503,7 @@ def __init__(
         .. seealso:: The MongoDB documentation on `gridfs <https://dochub.mongodb.org/core/gridfs>`_.
         """
         if not isinstance(db, AsyncDatabase):
-            raise TypeError(f"database must be an instance of AsyncDatabase, not {type(db)}.")
+            raise TypeError(f"database must be an instance of AsyncDatabase, not {type(db)}")
 
         db = _clear_entity_type_registry(db)
 
@@ -1083,7 +1083,7 @@ def __init__(
         """
         if not isinstance(root_collection, AsyncCollection):
             raise TypeError(
-                f"root_collection must be an instance of AsyncCollection, not {type(root_collection)}."
+                f"root_collection must be an instance of AsyncCollection, not {type(root_collection)}"
             )
 
         if not root_collection.write_concern.acknowledged:
@@ -1439,7 +1439,7 @@ def __init__(
         """
         if not isinstance(root_collection, AsyncCollection):
             raise TypeError(
-                f"root_collection must be an instance of AsyncCollection, not {type(root_collection)}."
+                f"root_collection must be an instance of AsyncCollection, not {type(root_collection)}"
             )
         _disallow_transactions(session)
 

gridfs/synchronous/grid_file.py

@@ -100,7 +100,7 @@ def __init__(self, database: Database, collection: str = "fs"):
         .. seealso:: The MongoDB documentation on `gridfs <https://dochub.mongodb.org/core/gridfs>`_.
         """
         if not isinstance(database, Database):
-            raise TypeError(f"database must be an instance of Database, not {type(database)}.")
+            raise TypeError(f"database must be an instance of Database, not {type(database)}")
 
         database = _clear_entity_type_registry(database)
 
@@ -501,7 +501,7 @@ def __init__(
         .. seealso:: The MongoDB documentation on `gridfs <https://dochub.mongodb.org/core/gridfs>`_.
         """
         if not isinstance(db, Database):
-            raise TypeError(f"database must be an instance of Database, not {type(db)}.")
+            raise TypeError(f"database must be an instance of Database, not {type(db)}")
 
         db = _clear_entity_type_registry(db)
 
@@ -1077,7 +1077,7 @@ def __init__(
         """
         if not isinstance(root_collection, Collection):
             raise TypeError(
-                f"root_collection must be an instance of Collection, not {type(root_collection)}."
+                f"root_collection must be an instance of Collection, not {type(root_collection)}"
             )
 
         if not root_collection.write_concern.acknowledged:
@@ -1429,7 +1429,7 @@ def __init__(
         """
         if not isinstance(root_collection, Collection):
             raise TypeError(
-                f"root_collection must be an instance of Collection, not {type(root_collection)}."
+                f"root_collection must be an instance of Collection, not {type(root_collection)}"
             )
         _disallow_transactions(session)
 

pymongo/__init__.py

@@ -160,7 +160,7 @@ def timeout(seconds: Optional[float]) -> ContextManager[None]:
     .. versionadded:: 4.2
     """
     if not isinstance(seconds, (int, float, type(None))):
-        raise TypeError(f"timeout must be None, an int, or a float, not {type(seconds)}.")
+        raise TypeError(f"timeout must be None, an int, or a float, not {type(seconds)}")
     if seconds and seconds < 0:
         raise ValueError("timeout cannot be negative")
     if seconds is not None: