fix: remove old editorContainer (#2268)

Author: diana-villalvazo-wgu

src/CourseAuthoringRoutes.jsx

@@ -7,7 +7,6 @@ import { PageWrap } from '@edx/frontend-platform/react';
 import { Textbooks } from './textbooks';
 import CourseAuthoringPage from './CourseAuthoringPage';
 import { PagesAndResources } from './pages-and-resources';
-import EditorContainer from './editors/EditorContainer';
 import VideoSelectorContainer from './selectors/VideoSelectorContainer';
 import CustomPages from './custom-pages';
 import { FilesPage, VideosPage } from './files-and-videos';
@@ -37,7 +36,7 @@ import { IframeProvider } from './generic/hooks/context/iFrameContext';
  *
  * /course/:courseId/course-pages
  * /course/:courseId/proctored-exam-settings
- * /course/:courseId/editor/:blockType/:blockId
+ * /course/:courseId/course-videos/:blockId
  *
  * This component and CourseAuthoringPage should maybe be combined once we no longer need to have
  * CourseAuthoringPage split out for use in LegacyProctoringRoute.  Once that route is removed, we
@@ -93,10 +92,6 @@ const CourseAuthoringRoutes = () => {
           path="editor/course-videos/:blockId"
           element={<PageWrap><VideoSelectorContainer courseId={courseId} /></PageWrap>}
         />
-        <Route
-          path="editor/:blockType/:blockId?"
-          element={<PageWrap><EditorContainer learningContextId={courseId} /></PageWrap>}
-        />
         <Route
           path="settings/details"
           element={<PageWrap><ScheduleAndDetails courseId={courseId} /></PageWrap>}

src/CourseAuthoringRoutes.test.jsx

@@ -6,7 +6,6 @@ import {
 
 const courseId = 'course-v1:edX+TestX+Test_Course';
 const pagesAndResourcesMockText = 'Pages And Resources';
-const editorContainerMockText = 'Editor Container';
 const videoSelectorContainerMockText = 'Video Selector Container';
 const customPagesMockText = 'Custom Pages';
 const mockComponentFn = jest.fn();
@@ -33,10 +32,6 @@ jest.mock('./pages-and-resources/PagesAndResources', () => (props) => {
   mockComponentFn(props);
   return pagesAndResourcesMockText;
 });
-jest.mock('./editors/EditorContainer', () => (props) => {
-  mockComponentFn(props);
-  return editorContainerMockText;
-});
 jest.mock('./selectors/VideoSelectorContainer', () => (props) => {
   mockComponentFn(props);
   return videoSelectorContainerMockText;
@@ -69,22 +64,6 @@ describe('<CourseAuthoringRoutes>', () => {
     });
   });
 
-  it('renders the EditorContainer component when the course editor route is active', async () => {
-    render(
-      <CourseAuthoringRoutes />,
-      { routerProps: { initialEntries: ['/editor/video/block-id'] } },
-    );
-    await waitFor(() => {
-      expect(screen.queryByText(editorContainerMockText)).toBeInTheDocument();
-      expect(screen.queryByText(pagesAndResourcesMockText)).not.toBeInTheDocument();
-      expect(mockComponentFn).toHaveBeenCalledWith(
-        expect.objectContaining({
-          learningContextId: courseId,
-        }),
-      );
-    });
-  });
-
   it('renders the VideoSelectorContainer component when the course videos route is active', async () => {
     render(
       <CourseAuthoringRoutes />,