feat: Unit page - make Video button functional

openedx · Feb 8, 2024 · a4959b2 · a4959b2
1 parent f07a7a9
commit a4959b2
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 2 deletions.
diff --git a/src/course-unit/CourseUnit.test.jsx b/src/course-unit/CourseUnit.test.jsx
@@ -176,4 +176,22 @@ describe('<CourseUnit />', () => {
       expect(mockedUsedNavigate).toHaveBeenCalledWith(`/course/${courseKey}/editor/problem/${locator}`);
     });
   });
+
+  it('handles creating Video xblock and navigates to editor page', async () => {
+    const { courseKey, locator } = courseCreateXblockMock;
+    axiosMock
+      .onPost(postXBlockBaseApiUrl({ type: 'video', category: 'video', parentLocator: blockId }))
+      .reply(200, courseCreateXblockMock);
+    const { getByRole } = render(<RootWrapper />);
+
+    await waitFor(() => {
+      const discussionButton = getByRole('button', {
+        name: new RegExp(`${messages.buttonText.defaultMessage} Video`, 'i'),
+      });
+
+      userEvent.click(discussionButton);
+      expect(mockedUsedNavigate).toHaveBeenCalled();
+      expect(mockedUsedNavigate).toHaveBeenCalledWith(`/course/${courseKey}/editor/video/${locator}`);
+    });
+  });
 });
diff --git a/src/course-unit/add-component/AddComponent.jsx b/src/course-unit/add-component/AddComponent.jsx
@@ -21,8 +21,9 @@ const AddComponent = ({ blockId, handleCreateNewCourseXblock }) => {
       handleCreateNewCourseXblock({ type, parentLocator: blockId });
       break;
     case COMPONENT_ICON_TYPES.problem:
+    case COMPONENT_ICON_TYPES.video:
       handleCreateNewCourseXblock({ type, parentLocator: blockId }, ({ courseKey, locator }) => {
-        navigate(`/course/${courseKey}/editor/problem/${locator}`);
+        navigate(`/course/${courseKey}/editor/${type}/${locator}`);
       });
       break;
     default:

diff --git a/src/course-unit/add-component/AddComponent.test.jsx b/src/course-unit/add-component/AddComponent.test.jsx
@@ -93,7 +93,7 @@ describe('<AddComponent />', () => {
     });
   });
 
-  it('create new "Problem" xblock on click', () => {
+  it('calls handleCreateNewCourseXblock with correct parameters when Problem xblock create button is clicked', () => {
     const { getByRole } = renderComponent();
 
     const discussionButton = getByRole('button', {
@@ -107,4 +107,19 @@ describe('<AddComponent />', () => {
       type: 'problem',
     }, expect.any(Function));
   });
+
+  it('calls handleCreateNewCourseXblock with correct parameters when Video xblock create button is clicked', () => {
+    const { getByRole } = renderComponent();
+
+    const discussionButton = getByRole('button', {
+      name: new RegExp(`${messages.buttonText.defaultMessage} Video`, 'i'),
+    });
+
+    userEvent.click(discussionButton);
+    expect(handleCreateNewCourseXblockMock).toHaveBeenCalled();
+    expect(handleCreateNewCourseXblockMock).toHaveBeenCalledWith({
+      parentLocator: '123',
+      type: 'video',
+    }, expect.any(Function));
+  });
 });