refactor: ContentLibraryContainersTest to use update_container_children to add components

Author: ChrisChV

openedx/core/djangoapps/content_libraries/api/containers.py

@@ -60,7 +60,6 @@ class ContainerMetadata(PublishableItem):
     Class that represents the metadata about a Container (e.g. Unit) in a content library.
     """
     container_key: LibraryContainerLocator
-    container_pk: int
     container_type: ContainerType
 
     @classmethod
@@ -89,7 +88,6 @@ def from_container(cls, library_key, container: Container, associated_collection
 
         return cls(
             container_key=container_key,  # LibraryContainerLocator
-            container_pk=container.pk,  # authoring_models.Container.pk
             container_type=container_type,
             display_name=draft.title,
             created=container.created,

openedx/core/djangoapps/content_libraries/tests/test_api.py

@@ -766,38 +766,25 @@ def setUp(self):
 
         # Create XBlocks
         # Create some library blocks in lib1
-        self.problem_block_dict = self._add_block_to_library(
+        self.problem_block = self._add_block_to_library(
             self.lib1.library_key, "problem", "problem1",
         )
-        self.problem_block_usage_key = UsageKey.from_string(self.problem_block_dict["id"])
-        self.html_block_dict = self._add_block_to_library(
+        self.problem_block_usage_key = UsageKey.from_string(self.problem_block["id"])
+        self.html_block = self._add_block_to_library(
             self.lib1.library_key, "html", "html1",
         )
-        self.html_block_usage_key = UsageKey.from_string(self.html_block_dict["id"])
-        now = datetime.now(tz=timezone.utc)
+        self.html_block_usage_key = UsageKey.from_string(self.html_block["id"])
 
         # Add content to units
-        # TODO build API for this
-        self.problem_block_component = api.get_component_from_usage_key(self.problem_block_usage_key)
-        self.html_block_component = api.get_component_from_usage_key(self.html_block_usage_key)
-        authoring_api.create_next_container_version(
-            self.unit1.container_pk,
-            publishable_entities_pks=[
-                self.problem_block_component.publishable_entity.id,
-                self.html_block_component.publishable_entity.id,
-            ],
-            title=None,
-            entity_version_pks=None,
-            created=now,
-            created_by=None,
-        )
-        authoring_api.create_next_container_version(
-            self.unit2.container_pk,
-            publishable_entities_pks=[self.html_block_component.publishable_entity.id],
-            title=None,
-            entity_version_pks=None,
-            created=now,
-            created_by=None,
+        api.update_container_children(
+            self.unit1.container_key,
+            [self.problem_block_usage_key, self.html_block_usage_key],
+            None,
+        )
+        api.update_container_children(
+            self.unit2.container_key,
+            [self.html_block_usage_key],
+            None,
         )
 
     def test_get_containers_contains_component(self):