test: fix completely broken tests

Author: bradenmacdonald

src/course-unit/CourseUnit.test.jsx

@@ -2291,41 +2291,38 @@ describe('<CourseUnit />', () => {
   });
 
   it('renders and navigates to the new HTML XBlock editor after xblock duplicating', async () => {
-    render(<RootWrapper />);
     const updatedCourseVerticalChildrenMock = JSON.parse(JSON.stringify(courseVerticalChildrenMock));
-    const targetBlockId = updatedCourseVerticalChildrenMock.children[1].block_id;
-
-    updatedCourseVerticalChildrenMock.children = updatedCourseVerticalChildrenMock.children
-      .map((child) => (child.block_id === targetBlockId
-        ? { ...child, block_type: 'html' }
-        : child));
+    // Convert the second child from drag and drop to HTML:
+    const targetChild = updatedCourseVerticalChildrenMock.children[1];
+    targetChild.block_type = 'html';
+    targetChild.name = 'Test HTML Block';
+    targetChild.block_id = 'block-v1:OpenedX+L153+3T2023+type@html+block@test123original';
 
     axiosMock
       .onPost(postXBlockBaseApiUrl({
         parent_locator: blockId,
-        duplicate_source_locator: courseVerticalChildrenMock.children[0].block_id,
+        duplicate_source_locator: targetChild.block_id,
       }))
       .replyOnce(200, { locator: '1234567890' });
 
     axiosMock
       .onGet(getCourseVerticalChildrenApiUrl(blockId))
       .reply(200, updatedCourseVerticalChildrenMock);
 
+    render(<RootWrapper />);
     await executeThunk(fetchCourseVerticalChildrenData(blockId), store.dispatch);
 
     await waitFor(() => {
       const iframe = screen.getByTitle(xblockContainerIframeMessages.xblockIframeTitle.defaultMessage);
       expect(iframe).toBeInTheDocument();
-      simulatePostMessageEvent(messageTypes.currentXBlockId, {
-        id: targetBlockId,
-      });
     });
 
+    // After duplicating, the editor modal will open:
+    expect(screen.queryByRole("dialog")).not.toBeInTheDocument()
+    simulatePostMessageEvent(messageTypes.duplicateXBlock, { usageId: targetChild.block_id });
+    simulatePostMessageEvent(messageTypes.newXBlockEditor, { blockType: 'html', usageId: targetChild.block_id });
     await waitFor(() => {
-      simulatePostMessageEvent(messageTypes.duplicateXBlock, {});
-      simulatePostMessageEvent(messageTypes.newXBlockEditor, {});
-      expect(mockedUsedNavigate)
-        .toHaveBeenCalledWith(`/course/${courseId}/editor/html/${targetBlockId}`, { replace: true });
+      expect(screen.queryByRole("dialog")).toBeInTheDocument()
     });
   });
 

src/schedule-and-details/requirements-section/entrance-exam/EntranceExam.test.jsx

@@ -1,11 +1,10 @@
-import React from 'react';
 import {
-  render, fireEvent, waitFor,
+  render, screen, waitFor,
 } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
 import { IntlProvider } from '@edx/frontend-platform/i18n';
 
 import { courseDetailsMock } from '../../__mocks__';
-import gradeRequirementsMessages from '../grade-requirements/messages';
 import messages from './messages';
 import EntranceExam from '.';
 
@@ -42,18 +41,20 @@ describe('<EntranceExam />', () => {
   });
 
   it('should toggle grade requirements after checkbox click', async () => {
-    const { getByText, queryAllByText, getAllByRole } = render(
-      <RootWrapper {...props} />,
-    );
-    const checkbox = getAllByRole('checkbox')[0];
-    expect(
-      getByText(gradeRequirementsMessages.requirementsEntranceCollapseLabel.defaultMessage),
-    ).toBeInTheDocument();
-    fireEvent.click(checkbox);
+    onChangeMock.mockClear();
+    const ui = render(<RootWrapper {...props} />);
+    const user = userEvent.setup();
+    const checkbox = screen.getByRole('checkbox', { name: 'Require students to pass an exam before beginning the course.' });
+
+    expect(checkbox).toBeChecked();
+    expect(screen.queryByText('Grade requirements')).toBeInTheDocument();
+
+    await user.click(checkbox);
+    expect(onChangeMock).toHaveBeenCalledWith('false', 'entranceExamEnabled');
+    ui.rerender(<RootWrapper {...props} isCheckedString="false" />);
     await waitFor(() => {
-      expect(
-        queryAllByText(gradeRequirementsMessages.requirementsEntranceCollapseLabel.defaultMessage).length,
-      ).toBe(0);
+      expect(checkbox).not.toBeChecked();
+      expect(screen.queryByText('Grade requirements')).not.toBeInTheDocument();
     });
   });
 });