squash! feat: Add slots to add tab links and add mechanism for plugin routes

Refactor to .jsx for redwood

Author: xitij2000

src/course-tabs/CourseTabsNavigation.tsx renamed to src/course-tabs/CourseTabsNavigation.jsx

@@ -1,28 +1,15 @@
 import { useIntl } from '@edx/frontend-platform/i18n';
 import { PluginSlot } from '@openedx/frontend-plugin-framework';
 import classNames from 'classnames';
+import PropTypes from 'prop-types';
 import React from 'react';
 import { CoursewareSearch, CoursewareSearchToggle } from '../course-home/courseware-search';
 import { useCoursewareSearchState } from '../course-home/courseware-search/hooks';
 import Tabs from '../generic/tabs/Tabs';
 
 import messages from './messages';
 
-interface CourseTabsNavigationProps {
-  activeTabSlug?: string;
-  className?: string | null;
-  tabs: Array<{
-    title: string;
-    slug: string;
-    url: string;
-  }>;
-}
-
-const CourseTabsNavigation = ({
-  activeTabSlug = undefined,
-  className = null,
-  tabs,
-}:CourseTabsNavigationProps) => {
+const CourseTabsNavigation = ({ activeTabSlug, className, tabs }) => {
   const { show } = useCoursewareSearchState();
   const intl = useIntl();
 
@@ -33,15 +20,21 @@ const CourseTabsNavigation = ({
           className="nav-underline-tabs"
           aria-label={intl.formatMessage(messages.courseMaterial)}
         >
-          <PluginSlot id="course_tab_links_slot">{tabs.map(({ url, title, slug }) => (
-            <a
-              key={slug}
-              className={classNames('nav-item flex-shrink-0 nav-link', { active: slug === activeTabSlug })}
-              href={url}
-            >
-              {title}
-            </a>
-          ))}</PluginSlot>
+          <PluginSlot id="course_tab_links_slot">
+            {tabs.map(({
+              url,
+              title,
+              slug,
+            }) => (
+              <a
+                key={slug}
+                className={classNames('nav-item flex-shrink-0 nav-link', { active: slug === activeTabSlug })}
+                href={url}
+              >
+                {title}
+              </a>
+            ))}
+          </PluginSlot>
         </Tabs>
       </div>
       <div className="course-tabs-navigation__search-toggle">
@@ -52,4 +45,19 @@ const CourseTabsNavigation = ({
   );
 };
 
+CourseTabsNavigation.propTypes = {
+  activeTabSlug: PropTypes.string,
+  className: PropTypes.string,
+  tabs: PropTypes.arrayOf(PropTypes.shape({
+    title: PropTypes.string.isRequired,
+    slug: PropTypes.string.isRequired,
+    url: PropTypes.string.isRequired,
+  })).isRequired,
+};
+
+CourseTabsNavigation.defaultProps = {
+  activeTabSlug: undefined,
+  className: null,
+};
+
 export default CourseTabsNavigation;

src/setupTest.js

@@ -30,12 +30,6 @@ import { appendBrowserTimezoneToUrl, executeThunk } from './utils';
 import buildSimpleCourseAndSequenceMetadata from './courseware/data/__factories__/sequenceMetadata.factory';
 import { buildOutlineFromBlocks } from './courseware/data/__factories__/learningSequencesOutline.factory';
 
-jest.mock('@openedx/frontend-plugin-framework', () => ({
-  ...jest.requireActual('@openedx/frontend-plugin-framework'),
-  Plugin: () => 'Plugin',
-  PluginSlot: () => 'PluginSlot',
-}));
-
 jest.mock('@src/generic/plugin-store', () => ({
   ...jest.requireActual('@src/generic/plugin-store'),
   usePluginsCallback: jest.fn((_, cb) => cb),