From 9809a7f7d16beec78dce758c654138f262fe95ab Mon Sep 17 00:00:00 2001 From: Christie Rice <8483753+crice100@users.noreply.github.com> Date: Wed, 28 Apr 2021 09:54:10 -0400 Subject: [PATCH] feat!: Add migration to remove allow_certificate field from the user profile as the field is no longer used DEPR-140 MICROBA-985 --- ...0043_remove_userprofile_allow_certificate.py | 17 +++++++++++++++++ common/djangoapps/util/tests/test_db.py | 3 --- 2 files changed, 17 insertions(+), 3 deletions(-) create mode 100644 common/djangoapps/student/migrations/0043_remove_userprofile_allow_certificate.py diff --git a/common/djangoapps/student/migrations/0043_remove_userprofile_allow_certificate.py b/common/djangoapps/student/migrations/0043_remove_userprofile_allow_certificate.py new file mode 100644 index 0000000000..b2eeea2fb9 --- /dev/null +++ b/common/djangoapps/student/migrations/0043_remove_userprofile_allow_certificate.py @@ -0,0 +1,17 @@ +# Generated by Django 2.2.20 on 2021-04-28 13:52 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('student', '0042_allow_certificate_null_20210427_1519'), + ] + + operations = [ + migrations.RemoveField( + model_name='userprofile', + name='allow_certificate', + ), + ] diff --git a/common/djangoapps/util/tests/test_db.py b/common/djangoapps/util/tests/test_db.py index 462c613128..f3e3db4539 100644 --- a/common/djangoapps/util/tests/test_db.py +++ b/common/djangoapps/util/tests/test_db.py @@ -197,9 +197,6 @@ class MigrationTests(TestCase): """ @override_settings(MIGRATION_MODULES={}) - @unittest.skip( - "Temporary skip for DEPR-140 / MICROBA-985 where the allow_certificate column is being removed" - ) def test_migrations_are_in_sync(self): """ Tests that the migration files are in sync with the models.