diff options
| author | Ed Morley <edmorley@users.noreply.github.com> | 2017-01-25 18:52:44 +0000 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2017-01-25 13:52:44 -0500 |
| commit | bfd42392ab8db3942b2cbb36a94af3b48813de13 (patch) | |
| tree | bd3ce4597b7d84bc716793f13b015f1518417ef8 | |
| parent | 90db4bb0d72e4731052bd33500840fff00834768 (diff) | |
Simplified tests with assertDoesNotOptimize().
| -rw-r--r-- | tests/migrations/test_optimizer.py | 30 |
1 files changed, 4 insertions, 26 deletions
diff --git a/tests/migrations/test_optimizer.py b/tests/migrations/test_optimizer.py index be2db97ba4..40cc50b33b 100644 --- a/tests/migrations/test_optimizer.py +++ b/tests/migrations/test_optimizer.py @@ -209,12 +209,7 @@ class OptimizerTests(SimpleTestCase): [], ) # This should not work - FK should block it - self.assertOptimizesTo( - [ - migrations.CreateModel("Foo", [("name", models.CharField(max_length=255))]), - migrations.CreateModel("Bar", [("other", models.ForeignKey("testapp.Foo", models.CASCADE))]), - migrations.DeleteModel("Foo"), - ], + self.assertDoesNotOptimize( [ migrations.CreateModel("Foo", [("name", models.CharField(max_length=255))]), migrations.CreateModel("Bar", [("other", models.ForeignKey("testapp.Foo", models.CASCADE))]), @@ -245,12 +240,7 @@ class OptimizerTests(SimpleTestCase): app_label="testapp", ) # This should not work - bases should block it - self.assertOptimizesTo( - [ - migrations.CreateModel("Foo", [("name", models.CharField(max_length=255))]), - migrations.CreateModel("Bar", [("size", models.IntegerField())], bases=("testapp.Foo", )), - migrations.DeleteModel("Foo"), - ], + self.assertDoesNotOptimize( [ migrations.CreateModel("Foo", [("name", models.CharField(max_length=255))]), migrations.CreateModel("Bar", [("size", models.IntegerField())], bases=("testapp.Foo", )), @@ -315,12 +305,7 @@ class OptimizerTests(SimpleTestCase): AddField should NOT optimize into CreateModel if it's an FK to a model that's between them. """ - self.assertOptimizesTo( - [ - migrations.CreateModel("Foo", [("name", models.CharField(max_length=255))]), - migrations.CreateModel("Link", [("url", models.TextField())]), - migrations.AddField("Foo", "link", models.ForeignKey("migrations.Link", models.CASCADE)), - ], + self.assertDoesNotOptimize( [ migrations.CreateModel("Foo", [("name", models.CharField(max_length=255))]), migrations.CreateModel("Link", [("url", models.TextField())]), @@ -335,14 +320,7 @@ class OptimizerTests(SimpleTestCase): """ # Note: The middle model is not actually a valid through model, # but that doesn't matter, as we never render it. - self.assertOptimizesTo( - [ - migrations.CreateModel("Foo", [("name", models.CharField(max_length=255))]), - migrations.CreateModel("LinkThrough", []), - migrations.AddField( - "Foo", "link", models.ManyToManyField("migrations.Link", through="migrations.LinkThrough") - ), - ], + self.assertDoesNotOptimize( [ migrations.CreateModel("Foo", [("name", models.CharField(max_length=255))]), migrations.CreateModel("LinkThrough", []), |