By default don't create parent for factory record

test/api/test_collection.py

@@ -429,7 +429,7 @@ def test_delete_collection(api, collection_batch, scopes, collection_auth, has_r
     del modified_collection_batch[2]
 
     if has_record:
-        RecordFactory(collection=collection_batch[2], is_child_record=False)
+        RecordFactory(collection=collection_batch[2])
 
     r = api(scopes, api_client_collections).delete(f'/collection/{collection_batch[2].id}')
 

test/api/test_provider.py

@@ -237,7 +237,7 @@ def test_delete_provider(api, provider_batch, scopes, has_record):
 
     if has_record:
         if collection := next((c for c in provider_batch[2].collections), None):
-            RecordFactory(collection=collection, is_child_record=False)
+            RecordFactory(collection=collection)
         else:
             has_record = False
 

test/factories.py

@@ -238,10 +238,10 @@ class Meta:
                                              SchemaFactory(id=r.schema_id, type='metadata'))
     timestamp = factory.LazyFunction(lambda: datetime.now(timezone.utc))
 
-    is_child_record = factory.LazyFunction(lambda: randint(0, 1))
+    is_child_record = False
     parent = factory.Maybe(
         'is_child_record',
-        yes_declaration=factory.SubFactory('test.factories.RecordFactory', is_child_record=False),
+        yes_declaration=factory.SubFactory('test.factories.RecordFactory'),
         no_declaration=None,
     )
 

test/test_db.py

@@ -1,3 +1,5 @@
+from random import randint
+
 from sqlalchemy import select
 
 import migrate.systemdata
@@ -90,7 +92,7 @@ def test_create_provider():
 
 
 def test_create_record():
-    record = RecordFactory()
+    record = RecordFactory(is_child_record=randint(0, 1))
     result = Session.execute(
         select(Record).where(Record.id == record.id)
     ).scalar_one()