Merge pull request #21232 from edx/feanil/remove_parsepy_leftovers

Feanil/remove parsepy leftovers

cms/djangoapps/contentstore/migrations/0004_remove_push_notification_configmodel_table.py

@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.22 on 2019-07-26 20:12
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('contentstore', '0003_remove_assets_page_flag'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='pushnotificationconfig',
+            name='changed_by',
+        ),
+        migrations.DeleteModel(
+            name='PushNotificationConfig',
+        ),
+    ]

cms/djangoapps/contentstore/tasks.py

@@ -550,15 +550,6 @@ def update_library_index(library_id, triggered_time_isoformat):
         LOGGER.debug(u'Search indexing successful for library %s', library_id)
 
 
-@task()
-def push_course_update_task(course_key_string, course_subscription_id, course_display_name):
-    """
-    Sends a push notification for a course update.
-    """
-    # TODO Delete once we've done a deploy where nothing is using this. DEPR-41
-    pass
-
-
 class CourseExportTask(UserTask):  # pylint: disable=abstract-method
     """
     Base class for course and library export tasks.

common/djangoapps/util/tests/test_db.py

@@ -222,7 +222,6 @@ class MigrationTests(TestCase):
     """
     Tests for migrations.
     """
-    @unittest.skip("Migration will delete a model. Need to ship not referencing it first. DEPR-41.")
     @override_settings(MIGRATION_MODULES={})
     def test_migrations_are_in_sync(self):
         """