Merge pull request #23270 from edx/revert-23253-msingh/run_migration

Revert "Ran make migration on third_party_auth"

common/djangoapps/third_party_auth/migrations/0025_auto_20200302_1701.py

@@ -1,28 +0,0 @@
-# -*- coding: utf-8 -*-
-# Generated by Django 1.11.28 on 2020-03-02 17:01
-from __future__ import unicode_literals
-
-from django.db import migrations, models
-import openedx.core.lib.hash_utils
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('third_party_auth', '0024_fix_edit_disallowed'),
-    ]
-
-    operations = [
-        migrations.RemoveField(
-            model_name='providerapipermissions',
-            name='client',
-        ),
-        migrations.AlterField(
-            model_name='ltiproviderconfig',
-            name='lti_consumer_secret',
-            field=models.CharField(blank=True, default=openedx.core.lib.hash_utils.create_hash256, help_text='The shared secret that the LTI Tool Consumer will use to authenticate requests. Only this edX instance and this tool consumer instance should know this value. For increased security, you can avoid storing this in your database by leaving this field blank and setting SOCIAL_AUTH_LTI_CONSUMER_SECRETS = {"consumer key": "secret", ...} in your instance\'s Django setttigs (or lms.auth.json)', max_length=255),
-        ),
-        migrations.DeleteModel(
-            name='ProviderApiPermissions',
-        ),
-    ]

common/djangoapps/util/tests/test_db.py

@@ -222,7 +222,7 @@ class MigrationTests(TestCase):
     Tests for migrations.
     """
     @unittest.skip(
-        "Need to skip as part of renaming a field in schedules app."
+        "Need to skip as part of renaming a field in schedules app. This will be unskipped in DE-1825. Also need to skip as part of removal of ProviderApiPermissions.This will be unskipped in BOM-1350"
     )
     @override_settings(MIGRATION_MODULES={})
     def test_migrations_are_in_sync(self):