From 6562fad7df0ee5701663748e292ae63aadc6cbc8 Mon Sep 17 00:00:00 2001 From: Mubbshar Anwar <78487564+mubbsharanwar@users.noreply.github.com> Date: Tue, 18 Jul 2023 17:07:28 +0500 Subject: [PATCH] revert: Removing save_far_later models (#32712) This is the second phase of removing save_for_later app in which migrations would be run to drop the DB tables. VAN-1451 --- common/djangoapps/util/tests/test_db.py | 2 -- .../migrations/0003_auto_20230711_1431.py | 19 +++++++++++++++++++ 2 files changed, 19 insertions(+), 2 deletions(-) create mode 100644 lms/djangoapps/save_for_later/migrations/0003_auto_20230711_1431.py diff --git a/common/djangoapps/util/tests/test_db.py b/common/djangoapps/util/tests/test_db.py index 5550bf3a0f..4a16c2a20a 100644 --- a/common/djangoapps/util/tests/test_db.py +++ b/common/djangoapps/util/tests/test_db.py @@ -3,7 +3,6 @@ from io import StringIO import ddt -import unittest from django.core.management import call_command from django.db.transaction import TransactionManagementError, atomic from django.test import TestCase, TransactionTestCase @@ -121,7 +120,6 @@ class MigrationTests(TestCase): Tests for migrations. """ - @unittest.skip("Migration will delete several models. Need to skip not referencing it first.") @override_settings(MIGRATION_MODULES={}) def test_migrations_are_in_sync(self): """ diff --git a/lms/djangoapps/save_for_later/migrations/0003_auto_20230711_1431.py b/lms/djangoapps/save_for_later/migrations/0003_auto_20230711_1431.py new file mode 100644 index 0000000000..41e076ca85 --- /dev/null +++ b/lms/djangoapps/save_for_later/migrations/0003_auto_20230711_1431.py @@ -0,0 +1,19 @@ +# Generated by Django 3.2.20 on 2023-07-11 14:31 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('save_for_later', '0002_auto_20220322_1621'), + ] + + operations = [ + migrations.DeleteModel( + name='SavedCourse', + ), + migrations.DeleteModel( + name='SavedProgram', + ), + ]