Refs #36143 -- Tested bulk_batch_size limit for bulk_update and bulk_create.

laymonage · sarahboyce · commit 38660a612cd9 · 2025-05-08T11:37:11.000+01:00
diff --git a/tests/bulk_create/tests.py b/tests/bulk_create/tests.py
@@ -293,6 +293,14 @@ def test_explicit_batch_size_respects_max_batch_size(self):
         with self.assertNumQueries(ceil(len(objs) / max_batch_size)):
             Country.objects.bulk_create(objs, batch_size=max_batch_size + 1)
 
+    @skipUnlessDBFeature("has_bulk_insert")
+    def test_max_batch_size(self):
+        objs = [Country(name=f"Country {i}") for i in range(1000)]
+        fields = ["name", "iso_two_letter", "description"]
+        max_batch_size = connection.ops.bulk_batch_size(fields, objs)
+        with self.assertNumQueries(ceil(len(objs) / max_batch_size)):
+            Country.objects.bulk_create(objs)
+
     @skipUnlessDBFeature("has_bulk_insert")
     def test_bulk_insert_expressions(self):
         Restaurant.objects.bulk_create(
diff --git a/tests/queries/test_bulk_update.py b/tests/queries/test_bulk_update.py
@@ -1,6 +1,8 @@
 import datetime
+from math import ceil
 
 from django.core.exceptions import FieldDoesNotExist
+from django.db import connection
 from django.db.models import F
 from django.db.models.functions import Lower
 from django.db.utils import IntegrityError
@@ -69,6 +71,15 @@ def test_batch_size(self):
         with self.assertNumQueries(len(self.notes)):
             Note.objects.bulk_update(self.notes, fields=["note"], batch_size=1)
 
+    def test_max_batch_size(self):
+        max_batch_size = connection.ops.bulk_batch_size(
+            # PK is used twice, see comment in bulk_update().
+            [Note._meta.pk, Note._meta.pk, Note._meta.get_field("note")],
+            self.notes,
+        )
+        with self.assertNumQueries(ceil(len(self.notes) / max_batch_size)):
+            Note.objects.bulk_update(self.notes, fields=["note"])
+
     def test_unsaved_models(self):
         objs = self.notes + [Note(note="test", misc="test")]
         msg = "All bulk_update() objects must have a primary key set."
