Address review

Author: NoahStapp

pymongo/asynchronous/bulk.py

@@ -87,7 +87,7 @@ def __init__(
         self,
         collection: AsyncCollection[_DocumentType],
         ordered: bool,
-        bypass_document_validation: Optional[bool] = None,
+        bypass_document_validation: Optional[bool],
         comment: Optional[str] = None,
         let: Optional[Any] = None,
     ) -> None:

pymongo/asynchronous/collection.py

@@ -800,8 +800,8 @@ async def _insert_one(
         ordered: bool,
         write_concern: WriteConcern,
         op_id: Optional[int],
+        bypass_doc_val: Optional[bool],
         session: Optional[AsyncClientSession],
-        bypass_doc_val: Optional[bool] = None,
         comment: Optional[Any] = None,
     ) -> Any:
         """Internal helper for inserting a single document."""
@@ -3357,13 +3357,7 @@ async def find_one_and_delete(
         if comment is not None:
             kwargs["comment"] = comment
         return await self._find_and_modify(
-            filter,
-            projection,
-            sort,
-            let=let,
-            hint=hint,
-            session=session,
-            **kwargs,
+            filter, projection, sort, let=let, hint=hint, session=session, **kwargs
         )
 
     async def find_one_and_replace(

pymongo/synchronous/bulk.py

@@ -87,7 +87,7 @@ def __init__(
         self,
         collection: Collection[_DocumentType],
         ordered: bool,
-        bypass_document_validation: Optional[bool] = None,
+        bypass_document_validation: Optional[bool],
         comment: Optional[str] = None,
         let: Optional[Any] = None,
     ) -> None:

pymongo/synchronous/collection.py

@@ -799,8 +799,8 @@ def _insert_one(
         ordered: bool,
         write_concern: WriteConcern,
         op_id: Optional[int],
+        bypass_doc_val: Optional[bool],
         session: Optional[ClientSession],
-        bypass_doc_val: Optional[bool] = None,
         comment: Optional[Any] = None,
     ) -> Any:
         """Internal helper for inserting a single document."""
@@ -3350,13 +3350,7 @@ def find_one_and_delete(
         if comment is not None:
             kwargs["comment"] = comment
         return self._find_and_modify(
-            filter,
-            projection,
-            sort,
-            let=let,
-            hint=hint,
-            session=session,
-            **kwargs,
+            filter, projection, sort, let=let, hint=hint, session=session, **kwargs
         )
 
     def find_one_and_replace(

test/utils_shared.py

@@ -615,10 +615,8 @@ def prepare_spec_arguments(spec, arguments, opname, entity_map, with_txn_callbac
         # Aggregate uses "batchSize", while find uses batch_size.
         elif (arg_name == "batchSize" or arg_name == "allowDiskUse") and opname == "aggregate":
             continue
-        elif (
-            arg_name == "bypassDocumentValidation"
-            and opname == "aggregate"
-            or "find_one_and" in opname
+        elif arg_name == "bypassDocumentValidation" and (
+            opname == "aggregate" or "find_one_and" in opname
         ):
             continue
         elif arg_name == "timeoutMode":