Ran make migration on third_party_auth (#23253)

* Ran make migration on third_party_auth and removing skip comment from test_migrations_are_in_sync

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

@@ -0,0 +1,28 @@
+# -*- 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. This will be unskipped in DE-1825. Also need to skip as part of removal of ProviderApiPermissions.This will be unskipped in BOM-1350"
+        "Need to skip as part of renaming a field in schedules app."
     )
     @override_settings(MIGRATION_MODULES={})
     def test_migrations_are_in_sync(self):