From fa2140cd1618ab52f72325b1f71e3b08be07941a Mon Sep 17 00:00:00 2001
From: Manjinder Singh <49171515+jinder1s@users.noreply.github.com>
Date: Tue, 3 Mar 2020 09:17:26 -0500
Subject: [PATCH] Revert "Ran make migration on third_party_auth (#23253)"

This reverts commit 49be65cc58f0eedac40d66237954f76effbc63d0.
---
 .../migrations/0025_auto_20200302_1701.py     | 28 -------------------
 common/djangoapps/util/tests/test_db.py       |  2 +-
 2 files changed, 1 insertion(+), 29 deletions(-)
 delete mode 100644 common/djangoapps/third_party_auth/migrations/0025_auto_20200302_1701.py

diff --git a/common/djangoapps/third_party_auth/migrations/0025_auto_20200302_1701.py b/common/djangoapps/third_party_auth/migrations/0025_auto_20200302_1701.py
deleted file mode 100644
index 0aa4ffb242..0000000000
--- a/common/djangoapps/third_party_auth/migrations/0025_auto_20200302_1701.py
+++ /dev/null
@@ -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',
-        ),
-    ]
diff --git a/common/djangoapps/util/tests/test_db.py b/common/djangoapps/util/tests/test_db.py
index 159e81c6d9..a20896fa61 100644
--- a/common/djangoapps/util/tests/test_db.py
+++ b/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):