fix: assert collection doc have unique id

openedx · Sep 11, 2024 · 1738447 · 1738447
1 parent e6b469d
commit 1738447
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/openedx/core/djangoapps/content/search/documents.py b/openedx/core/djangoapps/content/search/documents.py
@@ -288,7 +288,7 @@ def searchable_doc_for_collection(collection) -> dict:
     found using faceted search.
     """
     doc = {
-        Fields.id: collection.key,
+        Fields.id: collection.id,
         Fields.type: DocType.collection,
         Fields.display_name: collection.title,
         Fields.description: collection.description,

diff --git a/openedx/core/djangoapps/content/search/tests/test_api.py b/openedx/core/djangoapps/content/search/tests/test_api.py
@@ -186,7 +186,7 @@ def setUp(self):
                 description="my collection description"
             )
         self.collection_dict = {
-            'id': 'MYCOL',
+            'id': self.collection.id,
             'type': 'collection',
             'display_name': 'my_collection',
             'description': 'my collection description',

diff --git a/openedx/core/djangoapps/content/search/tests/test_documents.py b/openedx/core/djangoapps/content/search/tests/test_documents.py
@@ -222,7 +222,7 @@ def test_collection_with_no_library(self):
             )
         doc = searchable_doc_for_collection(collection)
         assert doc == {
-            "id": "MYCOL",
+            "id": collection.id,
             "type": "collection",
             "display_name": "my_collection",
             "description": "my collection description",