feat: show published children when showing only published Unit content

Author: pomegranited

src/library-authoring/components/ContainerCard.test.tsx

@@ -183,4 +183,25 @@ describe('<ContainerCard />', () => {
     expect((await screen.findAllByTitle(/lb:org1:Demo_course:html:text-*/)).length).toBe(4);
     expect(screen.queryByText('+2')).toBeInTheDocument();
   });
+
+  it('should render published child blocks when rendering a published card preview', async () => {
+    const containerWithPublishedChildren = {
+      ...containerHitSample,
+      content: {
+        childUsageKeys: Array(6).fill('').map((_child, idx) => `lb:org1:Demo_course:html:text-${idx}`),
+      },
+      published: {
+        content: {
+          childUsageKeys: Array(2).fill('').map((_child, idx) => `lb:org1:Demo_course:html:text-${idx}`),
+        },
+      },
+    } satisfies ContainerHit;
+    render(
+      <ContainerCard hit={containerWithPublishedChildren} />,
+      true,
+    );
+
+    expect((await screen.findAllByTitle(/lb:org1:Demo_course:html:text-*/)).length).toBe(2);
+    expect(screen.queryByText('+2')).not.toBeInTheDocument();
+  });
 });

src/library-authoring/components/ContainerCard.tsx

@@ -108,13 +108,11 @@ const ContainerMenu = ({ hit } : ContainerMenuProps) => {
 };
 
 type ContainerCardPreviewProps = {
-  hit: ContainerHit,
+  childUsageKeys: Array<string>;
   showMaxChildren?: number;
 };
 
-const ContainerCardPreview = ({ hit, showMaxChildren = 5 }: ContainerCardPreviewProps) => {
-  const { content } = hit;
-  const { childUsageKeys } = content ?? { childUsageKeys: [] };
+const ContainerCardPreview = ({ childUsageKeys, showMaxChildren = 5 }: ContainerCardPreviewProps) => {
   const hiddenChildren = childUsageKeys.length - showMaxChildren;
   return (
     <Stack direction="horizontal" gap={2}>
@@ -172,9 +170,10 @@ const ContainerCard = ({ hit } : ContainerCardProps) => {
     formatted,
     tags,
     numChildren,
-    published,
+    published = {},
     publishStatus,
     usageKey: unitId,
+    content = {},
   } = hit;
 
   const numChildrenCount = showOnlyPublished ? (
@@ -185,6 +184,11 @@ const ContainerCard = ({ hit } : ContainerCardProps) => {
     showOnlyPublished ? formatted.published?.displayName : formatted.displayName
   ) ?? '';
 
+  published.content = published?.content ?? {};
+  const childUsageKeys: Array<string> = (
+    showOnlyPublished ? published.content?.childUsageKeys : content?.childUsageKeys
+  ) ?? [];
+
   const { navigateTo } = useLibraryRoutes();
 
   const openContainer = useCallback(() => {
@@ -199,7 +203,7 @@ const ContainerCard = ({ hit } : ContainerCardProps) => {
     <BaseCard
       itemType={itemType}
       displayName={displayName}
-      preview={<ContainerCardPreview hit={hit} />}
+      preview={<ContainerCardPreview childUsageKeys={childUsageKeys} />}
       tags={tags}
       numChildren={numChildrenCount}
       actions={(

src/search-manager/data/api.ts

@@ -50,6 +50,7 @@ export const getContentSearchConfig = async (): Promise<{ url: string, indexName
 export interface ContentDetails {
   htmlContent?: string;
   capaContent?: string;
+  childUsageKeys?: Array<string>;
   [k: string]: any;
 }
 
@@ -151,9 +152,10 @@ export interface ContentHit extends BaseContentHit {
  * Defined in edx-platform/openedx/core/djangoapps/content/search/documents.py
  */
 export interface ContentPublishedData {
-  description?: string,
-  displayName?: string,
-  numChildren?: number,
+  description?: string;
+  displayName?: string;
+  numChildren?: number;
+  content?: ContentDetails;
 }
 
 /**
@@ -172,7 +174,7 @@ export interface CollectionHit extends BaseContentHit {
  * Defined in edx-platform/openedx/core/djangoapps/content/search/documents.py
  */
 interface ContainerHitContent {
-  childUsageKeys: string[],
+  childUsageKeys?: string[],
 }
 export interface ContainerHit extends BaseContentHit {
   type: 'library_container';