Apply suggestions from code review

Co-authored-by: Ethan Ho <[email protected]>

Author: CBroz1

datajoint/table.py

@@ -486,7 +486,7 @@ def delete(
         transaction: bool = True,
         safemode: Union[bool, None] = None,
         force_parts: bool = False,
-        force_masters: bool = True,
+        force_masters: bool = False,
     ) -> int:
         """
         Deletes the contents of the table and its dependent tables, recursively.
@@ -499,7 +499,7 @@ def delete(
                 is `dj.config['safemode']`.
             force_parts: Delete from parts even when not deleting from their masters.
             force_masters: If `True`, include part/master pairs in the cascade.
-                Default is `True`.
+                Default is `False`.
 
         Returns:
             Number of deleted rows (excluding those from dependent tables).

tests/test_cascading_delete.py

@@ -127,7 +127,7 @@ def test_delete_parts(schema_simp_pop):
 def test_delete_parts_complex(schema_simp_pop):
     """test issue #151 with complex master/part. PR #1158."""
     prev_len = len(G())
-    (A() & "id_a=1").delete()
+    (A() & "id_a=1").delete(force_masters=True)
     assert prev_len - len(G()) == 16, "Failed to delete parts"