feat: refactoring

openedx · Oct 25, 2024 · e9e8ec4 · e9e8ec4
1 parent 470beb9
commit e9e8ec4
Show file tree

Hide file tree

Showing 24 changed files with 3,669 additions and 475 deletions.
diff --git a/src/CourseAuthoringRoutes.jsx b/src/CourseAuthoringRoutes.jsx
@@ -17,7 +17,7 @@ import ScheduleAndDetails from './schedule-and-details';
 import { GradingSettings } from './grading-settings';
 import CourseTeam from './course-team/CourseTeam';
 import { CourseUpdates } from './course-updates';
-import { CourseUnit } from './course-unit';
+import { CourseUnit, IframeProvider } from './course-unit';
 import { Certificates } from './certificates';
 import CourseExportPage from './export-page/CourseExportPage';
 import CourseImportPage from './import-page/CourseImportPage';
@@ -79,7 +79,7 @@ const CourseAuthoringRoutes = () => {
           <Route
             key={path}
             path={path}
-            element={<PageWrap><CourseUnit courseId={courseId} /></PageWrap>}
+            element={<PageWrap><IframeProvider><CourseUnit courseId={courseId} /></IframeProvider></PageWrap>}
           />
         ))}
         <Route

diff --git a/src/course-unit/CourseUnit.jsx b/src/course-unit/CourseUnit.jsx
@@ -110,10 +110,16 @@ const CourseUnit = ({ courseId }) => {
                 aria-hidden={movedXBlockParams.isSuccess}
                 dismissible
                 actions={movedXBlockParams.isUndo ? null : [
-                  <Button onClick={handleRollbackMovedXBlock}>
+                  <Button
+                    onClick={handleRollbackMovedXBlock}
+                    key="xblock-moved-alert-undo-move-button"
+                  >
                     {intl.formatMessage(messages.undoMoveButton)}
                   </Button>,
-                  <Button onClick={handleNavigateToTargetUnit}>
+                  <Button
+                    onClick={handleNavigateToTargetUnit}
+                    key="xblock-moved-alert-new-location-button"
+                  >
                     {intl.formatMessage(messages.newLocationButton)}
                   </Button>,
                 ]}
@@ -141,18 +147,6 @@ const CourseUnit = ({ courseId }) => {
               />
             )}
           />
-
-          <hr />
-          <hr />
-          <Button onClick={openMoveModal}>MOVE MODAL TRIGGER</Button>
-          <MoveModal
-            isOpen={isMoveModalOpen}
-            close={closeMoveModal}
-            displayName="Choosing a Compute Option"
-          />
-          <hr />
-          <hr />
-
           <Sequence
             courseId={courseId}
             sequenceId={sequenceId}
@@ -194,6 +188,12 @@ const CourseUnit = ({ courseId }) => {
                   text={intl.formatMessage(messages.pasteButtonText)}
                 />
               )}
+              <MoveModal
+                isOpenModal={isMoveModalOpen}
+                openModal={openMoveModal}
+                closeModal={closeMoveModal}
+                courseId={courseId}
+              />
             </Layout.Element>
             <Layout.Element>
               <Stack gap={3}>

diff --git a/src/course-unit/CourseUnit.test.jsx b/src/course-unit/CourseUnit.test.jsx
@@ -53,6 +53,7 @@ import addComponentMessages from './add-component/messages';
 import { PUBLISH_TYPES, UNIT_VISIBILITY_STATES } from './constants';
 import messages from './messages';
 import { getContentTaxonomyTagsApiUrl, getContentTaxonomyTagsCountApiUrl } from '../content-tags-drawer/data/api';
+import { IframeProvider } from './context/iFrameContext';
 
 let axiosMock;
 let store;
@@ -108,7 +109,9 @@ global.BroadcastChannel = jest.fn(() => clipboardBroadcastChannelMock);
 const RootWrapper = () => (
   <AppProvider store={store}>
     <IntlProvider locale="en">
-      <CourseUnit courseId={courseId} />
+      <IframeProvider>
+        <CourseUnit courseId={courseId} />
+      </IframeProvider>
     </IntlProvider>
   </AppProvider>
 );