mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-11-03 05:13:23 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Fixed #27044 -- Included already applied migration changes in the post-migrate state when the execution plan is empty.

Browse source 
Refs #24100.

Thanks tkhyn for the report and Tim for the review.
This commit is contained in:
Simon Charette 2016-08-20 16:34:06 -04:00
parent ad25fe7350
commit d1757d8df4
5 changed files with 36 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

10
tests/migrate_signals/tests.py
View file

@ -113,3 +113,13 @@ class MigrateSignalTests(TransactionTestCase):
[model._meta.label for model in post_migrate_receiver.call_args['apps'].get_models()],
['migrate_signals.Signal']
)
# Migrating with an empty plan.
post_migrate_receiver = Receiver(signals.post_migrate)
management.call_command(
'migrate', database=MIGRATE_DATABASE, verbosity=MIGRATE_VERBOSITY,
interactive=MIGRATE_INTERACTIVE, stdout=stdout,
)
self.assertEqual(
[model._meta.label for model in post_migrate_receiver.call_args['apps'].get_models()],
['migrate_signals.Signal']
)

8
tests/migrations/test_executor.py
View file

@ -168,6 +168,14 @@ class ExecutorTests(MigrationTestBase):
("migrations2", "0001_initial"),
])
self.assertEqual(plan, [])
# The resulting state should include applied migrations.
state = executor.migrate([
("migrations", "0002_second"),
("migrations2", "0001_initial"),
])
self.assertIn(('migrations', 'book'), state.models)
self.assertIn(('migrations', 'author'), state.models)
self.assertIn(('migrations2', 'otherauthor'), state.models)
# Erase all the fake records
executor.recorder.record_unapplied("migrations2", "0001_initial")
executor.recorder.record_unapplied("migrations", "0002_second")