From 49be65cc58f0eedac40d66237954f76effbc63d0 Mon Sep 17 00:00:00 2001
From: Manjinder Singh <49171515+jinder1s@users.noreply.github.com>
Date: Mon, 2 Mar 2020 16:32:40 -0500
Subject: [PATCH] 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
---
 .../migrations/0025_auto_20200302_1701.py     | 28 +++++++++++++++++++
 common/djangoapps/util/tests/test_db.py       |  2 +-
 2 files changed, 29 insertions(+), 1 deletion(-)
 create 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
new file mode 100644
index 0000000000..0aa4ffb242
--- /dev/null
+++ b/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',
+        ),
+    ]
diff --git a/common/djangoapps/util/tests/test_db.py b/common/djangoapps/util/tests/test_db.py
index a20896fa61..159e81c6d9 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. 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):