bulk_insert should work with no rows

A fix was previously applied only in `bulk_uspert`.

Author: Photonios

psqlextra/query.py

@@ -139,6 +139,12 @@ def bulk_insert(
             the models of the rows inserted with defaults for any fields not specified
         """
 
+        def is_empty(r):
+            return all([False for _ in r])
+
+        if not rows or is_empty(rows):
+            return []
+
         if not self.conflict_target and not self.conflict_action:
             # no special action required, use the standard Django bulk_create(..)
             return super().bulk_create(
@@ -375,12 +381,6 @@ def bulk_upsert(
             the models of the rows upserted
         """
 
-        def is_empty(r):
-            return all([False for _ in r])
-
-        if not rows or is_empty(rows):
-            return []
-
         self.on_conflict(
             conflict_target,
             ConflictAction.UPDATE,

tests/test_upsert.py

@@ -7,6 +7,7 @@
 
 from psqlextra.expressions import ExcludedCol
 from psqlextra.fields import HStoreField
+from psqlextra.query import ConflictAction
 
 from .fake_model import get_fake_model
 
@@ -245,10 +246,18 @@ def test_upsert_bulk_no_rows():
         {"name": models.CharField(max_length=255, null=True, unique=True)}
     )
 
+    model.objects.on_conflict(ConflictAction.UPDATE, ["name"]).bulk_insert(
+        rows=[]
+    )
+
     model.objects.bulk_upsert(conflict_target=["name"], rows=[])
 
     model.objects.bulk_upsert(conflict_target=["name"], rows=None)
 
+    model.objects.on_conflict(ConflictAction.UPDATE, ["name"]).bulk_insert(
+        rows=None
+    )
+
 
 def test_bulk_upsert_return_models():
     """Tests whether models are returned instead of dictionaries when