fix: Broken tests

Author: ChrisChV

plugins/course-apps/proctoring/Settings.test.jsx

@@ -74,6 +74,8 @@ describe('ProctoredExamSettings', () => {
       provider: null,
     });
 
+    axiosMock.onGet(/course_index/).reply(200, { sections: [] });
+
     axiosMock.onGet(
       StudioApiService.getProctoredExamSettingsUrl(defaultProps.courseId),
     ).reply(200, {
@@ -466,7 +468,8 @@ describe('ProctoredExamSettings', () => {
       // (1) for studio settings
       // (2) waffle flags
       // (3) for course details
-      expect(axiosMock.history.get.length).toBe(3);
+      // (4) for course outline index (from CourseAuthoringProvider)
+      expect(axiosMock.history.get.length).toBe(4);
       expect(axiosMock.history.get[0].url.includes('proctored_exam_settings')).toEqual(true);
     });
 

src/course-outline/outline-sidebar/OutlineSidebar.test.tsx

@@ -15,6 +15,8 @@ jest.mock('@src/course-outline/data/apiHooks', () => ({
   useCourseDetails: jest.fn().mockReturnValue({ isPending: false, data: { title: 'Test Course' } }),
   useCreateCourseBlock: jest.fn(),
   useCourseItemData: jest.fn().mockReturnValue({ data: {} }),
+  useDuplicateItem: jest.fn().mockReturnValue({ duplicateItem: jest.fn() }),
+  useDeleteCourseItem: jest.fn().mockReturnValue({ mutateAsync: jest.fn() }),
 }));
 
 const courseId = '123';

src/course-outline/outline-sidebar/info-sidebar/InfoSection.tsx

@@ -54,6 +54,10 @@ export const InfoSection = ({ itemId }: Props) => {
     }
   }, [dispatch, selectedContainerState, queryClient, courseId]);
 
+  if (!itemData) {
+    return null;
+  }
+
   return (
     <>
       <LibraryReferenceCard

src/course-outline/outline-sidebar/info-sidebar/InfoSidebar.test.tsx

@@ -54,6 +54,7 @@ describe('InfoSidebar component', () => {
     const user = userEvent.setup();
     selectedContainerState = {
       currentId: 'block-v1:UNIX+UX1+2025_T3+type@chapter+block@123',
+      sectionId: 'block-v1:UNIX+UX1+2025_T3+type@chapter+block@123',
     };
     const data = {
       id: selectedContainerState.currentId,
@@ -80,6 +81,7 @@ describe('InfoSidebar component', () => {
     const user = userEvent.setup();
     selectedContainerState = {
       currentId: 'block-v1:UNIX+UX1+2025_T3+type@sequential+block@123',
+      subsectionId: 'block-v1:UNIX+UX1+2025_T3+type@sequential+block@123',
       sectionId: 'block-v1:UNIX+UX1+2025_T3+type@chapter+block@123',
     };
     const data = {

src/course-unit/CourseUnit.test.tsx

@@ -68,6 +68,7 @@ import CourseUnit from './CourseUnit';
 import tagsDrawerMessages from '../content-tags-drawer/messages';
 import configureModalMessages from '../generic/configure-modal/messages';
 import { getContentTaxonomyTagsApiUrl, getContentTaxonomyTagsCountApiUrl } from '../content-tags-drawer/data/api';
+import { getXBlockApiUrl } from '../course-outline/data/api';
 import addComponentMessages from './add-component/messages';
 import { messageTypes, PUBLISH_TYPES, UNIT_VISIBILITY_STATES } from './constants';
 import moveModalMessages from './move-modal/messages';
@@ -3250,6 +3251,10 @@ describe('<CourseUnit />', () => {
       ENABLE_UNIT_PAGE_NEW_DESIGN: 'true',
     });
 
+    axiosMock
+      .onGet(getXBlockApiUrl(mockContentData.textXBlock))
+      .reply(200, mockContentData.textXBlockData);
+
     render(<RootWrapper />);
 
     await screen.findByTitle(xblockContainerIframeMessages.xblockIframeTitle.defaultMessage);

src/generic/sidebar/InfoSidebarMenu.tsx

@@ -62,7 +62,7 @@ export const InfoSidebarMenu = ({
         alt={intl.formatMessage(messages.itemMenuAlt)}
       />
       <Dropdown.Menu>
-        {actions.duplicable && onClickDuplicate && (
+        {actions?.duplicable && onClickDuplicate && (
           <Dropdown.Item
             onClick={onClickDuplicate}
           >
@@ -96,7 +96,7 @@ export const InfoSidebarMenu = ({
             </Stack>
           </Dropdown.Item>
         )}
-        {actions.draggable && onClickMoveUp && onClickMoveDown && canMoveItem && (
+        {actions?.draggable && onClickMoveUp && onClickMoveDown && canMoveItem && (
           <>
             <Dropdown.Item
               onClick={onClickMoveUp}
@@ -128,20 +128,20 @@ export const InfoSidebarMenu = ({
             </Stack>
           </Dropdown.Item>
         )}
-        {((actions.unlinkable ?? null) !== null || actions.deletable) && <Dropdown.Divider />}
-        {(actions.unlinkable ?? null) !== null && (
+        {((actions?.unlinkable ?? null) !== null || actions?.deletable) && <Dropdown.Divider />}
+        {(actions?.unlinkable ?? null) !== null && (
           <Dropdown.Item
             onClick={onClickUnlink}
-            disabled={!actions.unlinkable}
-            title={!actions.unlinkable ? intl.formatMessage(messages.menuUnlinkDisabledTooltip) : undefined}
+            disabled={!actions?.unlinkable}
+            title={!actions?.unlinkable ? intl.formatMessage(messages.menuUnlinkDisabledTooltip) : undefined}
           >
             <Stack direction="horizontal" gap={2}>
               <Icon src={LinkOff} />
               {intl.formatMessage(messages.menuUnlink)}
             </Stack>
           </Dropdown.Item>
         )}
-        {actions.deletable && (
+        {actions?.deletable && (
           <Dropdown.Item
             onClick={onClickDelete}
             className="text-danger-700"