mirrors/django
mirrors/django
0
0
Fork 0
mirror of https://github.com/django/django.git synced 2024-11-21 19:09:18 +01:00
Code Issues Releases Wiki Activity

Implemented reduce for optimizemigration.

Browse Source
This commit is contained in:
Salvo Polizzi 2024-10-30 09:09:28 +01:00
parent 93e88c1fc5
commit b67602c74b
3 changed files with 33 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
django/db/migrations/operations/models.py
View File

@ -1313,3 +1313,8 @@ class AlterConstraint(IndexOperation):
@property
def migration_name_fragment(self):
return f"alter_{self.model_name_lower}_{self.constraint.name.lower()}"
def reduce(self, operation, app_label):
if isinstance(operation, AlterConstraint):
return [operation]
return super().reduce(operation, app_label)

2
docs/releases/5.2.txt
View File

@ -253,7 +253,7 @@ Management Commands
Migrations
~~~~~~~~~~
* The new operation `AlterConstraint` is a noop operation that alters
* The new operation ``AlterConstraint`` is a no-op operation that alters
constraints without dropping and recreating indexes in the database.
Models

27
tests/migrations/test_optimizer.py
View File

@ -1232,6 +1232,33 @@ class OptimizerTests(OptimizerTestBase):
],
)
def test_alter_constraint(self):
gt_constraint = models.CheckConstraint(
condition=models.Q(pink__gt=2), name="constraint_pony_pink_gt_2"
)
gt_constraint_violation_msg = models.CheckConstraint(
condition=models.Q(pink__gt=2),
name="constraint_pony_pink_gt_2",
violation_error_message="error",
)
self.assertOptimizesTo(
[
migrations.AlterConstraint("Pony", gt_constraint.name, gt_constraint),
migrations.AlterConstraint(
"Pony",
gt_constraint_violation_msg.name,
gt_constraint_violation_msg,
),
],
[
migrations.AlterConstraint(
"Pony",
gt_constraint_violation_msg.name,
gt_constraint_violation_msg,
)
],
)
def test_create_model_add_index(self):
self.assertOptimizesTo(
[