Merge pull request #33576 from thezaeemaanwar/badges_deletion_migrations

feat: Migrations for badges deletion

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 ship not referencing it first")
     @override_settings(MIGRATION_MODULES={})
     def test_migrations_are_in_sync(self):
         """

lms/djangoapps/badges/migrations/0005_delete_all_models.py

@@ -0,0 +1,41 @@
+# Generated by Django 3.2.22 on 2023-10-24 15:01
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('badges', '0004_badgeclass_badgr_server_slug'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='badgeassertion',
+            name='badge_class',
+        ),
+        migrations.RemoveField(
+            model_name='badgeassertion',
+            name='user',
+        ),
+        migrations.AlterUniqueTogether(
+            name='badgeclass',
+            unique_together=None,
+        ),
+        migrations.DeleteModel(
+            name='CourseCompleteImageConfiguration',
+        ),
+        migrations.RemoveField(
+            model_name='courseeventbadgesconfiguration',
+            name='changed_by',
+        ),
+        migrations.DeleteModel(
+            name='BadgeAssertion',
+        ),
+        migrations.DeleteModel(
+            name='BadgeClass',
+        ),
+        migrations.DeleteModel(
+            name='CourseEventBadgesConfiguration',
+        ),
+    ]