From 7d5446125dfea46ad69120e2c195364673bea620 Mon Sep 17 00:00:00 2001 From: Waheed Ahmed Date: Tue, 29 Aug 2017 12:58:41 +0500 Subject: [PATCH] Create migration to remove 'sailthru_activation_template' from DB. Follow up PR for PR-15863 to remove the field from database table. LEARNER-2201 --- common/djangoapps/util/tests/test_db.py | 9 +-------- ...nfiguration_sailthru_activation_template.py | 18 ++++++++++++++++++ 2 files changed, 19 insertions(+), 8 deletions(-) create mode 100644 lms/djangoapps/email_marketing/migrations/0009_remove_emailmarketingconfiguration_sailthru_activation_template.py diff --git a/common/djangoapps/util/tests/test_db.py b/common/djangoapps/util/tests/test_db.py index e9d7245f8a..a3c7801490 100644 --- a/common/djangoapps/util/tests/test_db.py +++ b/common/djangoapps/util/tests/test_db.py @@ -226,11 +226,4 @@ class MigrationTests(TestCase): out = StringIO() call_command('makemigrations', dry_run=True, verbosity=3, stdout=out) output = out.getvalue() - - # Temporary check, remove it once migration is created and use the else part. - if settings.ROOT_URLCONF == 'lms.urls': - migrations_count = output.count('Migrations for') - self.assertIn('Remove field sailthru_activation_template', output) - self.assertEqual(migrations_count, 1) - else: - self.assertIn('No changes detected', output) + self.assertIn('No changes detected', output) diff --git a/lms/djangoapps/email_marketing/migrations/0009_remove_emailmarketingconfiguration_sailthru_activation_template.py b/lms/djangoapps/email_marketing/migrations/0009_remove_emailmarketingconfiguration_sailthru_activation_template.py new file mode 100644 index 0000000000..59fc491bce --- /dev/null +++ b/lms/djangoapps/email_marketing/migrations/0009_remove_emailmarketingconfiguration_sailthru_activation_template.py @@ -0,0 +1,18 @@ +# -*- coding: utf-8 -*- +from __future__ import unicode_literals + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('email_marketing', '0008_auto_20170809_0539'), + ] + + operations = [ + migrations.RemoveField( + model_name='emailmarketingconfiguration', + name='sailthru_activation_template', + ), + ]