Omit size=None from ArrayField.deconstruct()

Author: timgraham

django_mongodb_backend/fields/array.py

@@ -124,12 +124,9 @@ def deconstruct(self):
         name, path, args, kwargs = super().deconstruct()
         if path == "django_mongodb_backend.fields.array.ArrayField":
             path = "django_mongodb_backend.fields.ArrayField"
-        kwargs.update(
-            {
-                "base_field": self.base_field.clone(),
-                "size": self.size,
-            }
-        )
+        kwargs["base_field"] = self.base_field.clone()
+        if self.size is not None:
+            kwargs["size"] = self.size
         return name, path, args, kwargs
 
     def to_python(self, value):

tests/model_fields_/array_default_migrations/0001_initial.py

@@ -21,7 +21,7 @@ class Migration(migrations.Migration):
                 ),
                 (
                     "field",
-                    django_mongodb_backend.fields.ArrayField(models.IntegerField(), size=None),
+                    django_mongodb_backend.fields.ArrayField(models.IntegerField()),
                 ),
             ],
             options={},

tests/model_fields_/array_default_migrations/0002_integerarraymodel_field_2.py

@@ -12,9 +12,7 @@ class Migration(migrations.Migration):
         migrations.AddField(
             model_name="integerarraydefaultmodel",
             name="field_2",
-            field=django_mongodb_backend.fields.ArrayField(
-                models.IntegerField(), default=[], size=None
-            ),
+            field=django_mongodb_backend.fields.ArrayField(models.IntegerField(), default=[]),
             preserve_default=False,
         ),
     ]

tests/model_fields_/test_arrayfield.py

@@ -85,6 +85,7 @@ class MyModel(models.Model):
     def test_deconstruct(self):
         field = ArrayField(models.IntegerField())
         name, path, args, kwargs = field.deconstruct()
+        self.assertEqual(kwargs.keys(), {"base_field"})
         new = ArrayField(*args, **kwargs)
         self.assertEqual(type(new.base_field), type(field.base_field))
         self.assertIsNot(new.base_field, field.base_field)
@@ -722,7 +723,7 @@ class MigrationsTests(TransactionTestCase):
     )
     def test_adding_field_with_default(self):
         class IntegerArrayDefaultModel(models.Model):
-            field = ArrayField(models.IntegerField(), size=None)
+            field = ArrayField(models.IntegerField())
 
         table_name = "model_fields__integerarraydefaultmodel"
         self.assertNotIn(table_name, connection.introspection.table_names(None))
@@ -734,8 +735,8 @@ class IntegerArrayDefaultModel(models.Model):
         call_command("migrate", "model_fields_", "0002", verbosity=0)
 
         class UpdatedIntegerArrayDefaultModel(models.Model):
-            field = ArrayField(models.IntegerField(), size=None)
-            field_2 = ArrayField(models.IntegerField(), default=[], size=None)
+            field = ArrayField(models.IntegerField())
+            field_2 = ArrayField(models.IntegerField(), default=[])
 
             class Meta:
                 db_table = "model_fields__integerarraydefaultmodel"