Skip to content

Commit

Permalink
Fix a regression with migration definitions on Django 1.6.
Browse files Browse the repository at this point in the history
Some of the new migrations weren't being defined conditionally based on
the presence of Django migrations support. This broke unit tests on
Django 1.6.

This is a trivial change to make these definitions conditional.
  • Loading branch information
chipx86 committed Oct 31, 2020
1 parent 0daf8e4 commit 9127ce5
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 20 deletions.
21 changes: 11 additions & 10 deletions django_evolution/tests/migrations_app/migrations/0002_add_field.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,15 @@
migrations = None


class Migration(migrations.Migration):
dependencies = [
('migrations_app', '0001_initial'),
]
if migrations:
class Migration(migrations.Migration):
dependencies = [
('migrations_app', '0001_initial'),
]

operations = [
migrations.AddField(
model_name='MigrationsAppTestModel',
name='added_field',
field=models.IntegerField(default=42)),
]
operations = [
migrations.AddField(
model_name='MigrationsAppTestModel',
name='added_field',
field=models.IntegerField(default=42)),
]
21 changes: 11 additions & 10 deletions django_evolution/tests/migrations_app2/migrations/0002_add_field.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,15 @@
migrations = None


class Migration(migrations.Migration):
dependencies = [
('migrations_app2', '0001_initial'),
]
if migrations:
class Migration(migrations.Migration):
dependencies = [
('migrations_app2', '0001_initial'),
]

operations = [
migrations.AddField(
model_name='MigrationsApp2TestModel',
name='added_field',
field=models.BooleanField(default=False)),
]
operations = [
migrations.AddField(
model_name='MigrationsApp2TestModel',
name='added_field',
field=models.BooleanField(default=False)),
]

0 comments on commit 9127ce5

Please sign in to comment.