Don't use Course.copied_from_id for LMSCourse equivalent

We need the FK on the LMSCourse table itself, not the old table. However we can't yet get that value until we backfill the table complete to get a reference for all row in the new table.
hypothesis · Aug 26, 2024 · 8f009ee · 8f009ee
1 parent 555b869
commit 8f009ee
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/lms/services/course.py b/lms/services/course.py
@@ -297,7 +297,6 @@ def _upsert_lms_course(self, course: Course) -> LMSCourse:
                     "tool_consumer_instance_guid": course.application_instance.tool_consumer_instance_guid,
                     "lti_context_id": course.lms_id,
                     "h_authority_provided_id": course.authority_provided_id,
-                    "copied_from_id": course.copied_from_id,
                     "name": course.lms_name,
                 }
             ],

diff --git a/tests/unit/lms/services/course_test.py b/tests/unit/lms/services/course_test.py
@@ -223,7 +223,6 @@ def test_upsert_course(self, svc, grouping_service, bulk_upsert, db_session):
                             "tool_consumer_instance_guid": course.application_instance.tool_consumer_instance_guid,
                             "lti_context_id": course.lms_id,
                             "h_authority_provided_id": course.authority_provided_id,
-                            "copied_from_id": course.copied_from_id,
                             "name": course.lms_name,
                         }
                     ],