Fixed #27845 -- Allowed both right and left optimizations of operations.

Thanks Raphael Gaschignard for the suggestion.

1 files changed, 1 insertions, 9 deletions

diff --git a/tests/migrations/test_optimizer.py b/tests/migrations/test_optimizer.py
index 973be67c80..a408db9ef2 100644
--- a/tests/migrations/test_optimizer.py
+++ b/tests/migrations/test_optimizer.py
@@ -270,8 +270,6 @@ class OptimizerTests(SimpleTestCase):
             app_label="testapp",
         )
 
-        # This could be optimized a bit more but it generates a valid set of
-        # operations.
         self.assertOptimizesTo(
             [
                 migrations.CreateModel('Book', [('name', models.CharField(max_length=255))]),
@@ -284,18 +282,12 @@ class OptimizerTests(SimpleTestCase):
                 migrations.DeleteModel('Person'),
             ],
             [
-                migrations.CreateModel('Person', [('name', models.CharField(max_length=255))]),
-                migrations.CreateModel('Book', [
-                    ('name', models.CharField(max_length=255)),
-                    ('author', models.ForeignKey('test_app.Person', models.CASCADE)),
-                ]),
+                migrations.CreateModel('Book', [('name', models.CharField(max_length=255))]),
                 migrations.CreateModel('Reviewer', [('name', models.CharField(max_length=255))]),
                 migrations.CreateModel('Review', [
                     ('book', models.ForeignKey('test_app.Book', models.CASCADE)),
                     ('reviewer', models.ForeignKey('test_app.Reviewer', models.CASCADE)),
                 ]),
-                migrations.RemoveField('book', 'author'),
-                migrations.DeleteModel('Person'),
             ],
         )