chore: Update test for mapToStore enterpriseSlug

openedx · Dec 14, 2022 · d77cd5e · d77cd5e
1 parent edce596
commit d77cd5e
Show file tree

Hide file tree

Showing 8 changed files with 47 additions and 43 deletions.
diff --git a/src/components/ContentHighlights/ContentHighlightSetCardContainer.jsx b/src/components/ContentHighlights/ContentHighlightSetCardContainer.jsx
diff --git a/src/components/ContentHighlights/ContentHighlightsCardItemsContainer.jsx b/src/components/ContentHighlights/ContentHighlightsCardItemsContainer.jsx
@@ -26,8 +26,8 @@ const ContentHighlightsCardItemsContainer = ({ enterpriseSlug, isLoading, highli
           key={uuid}
           cardImageUrl="https://picsum.photos/200/300"
           title={title}
-          hyperlink={generateAboutPageUrl(enterpriseSlug, contentType.toLowerCase(), contentKey)}
-          contentType={contentType.toLowerCase()}
+          hyperlink={generateAboutPageUrl(enterpriseSlug, contentType?.toLowerCase(), contentKey)}
+          contentType={contentType?.toLowerCase()}
           partners={authoringOrganizations}
         />
       ))}

diff --git a/src/components/ContentHighlights/HighlightStepper/ContentConfirmContentCard.jsx b/src/components/ContentHighlights/HighlightStepper/ContentConfirmContentCard.jsx
@@ -38,7 +38,7 @@ const ContentConfirmContentCard = ({ enterpriseSlug, original }) => {
     <React.Fragment key={key}>
       <ContentHighlightCardItem
         title={title}
-        hyperlink={generateAboutPageUrl(enterpriseSlug, contentType.toLowerCase(), aggregationKey.split(':')[1])}
+        hyperlink={generateAboutPageUrl(enterpriseSlug, contentType?.toLowerCase(), aggregationKey?.split(':')[1])}
         contentType={contentType}
         partners={partners}
         cardImageUrl={cardImageUrl || originalImageUrl}
@@ -72,7 +72,7 @@ ContentConfirmContentCard.propTypes = {
   }).isRequired,
 };
 
-const mapStateToProps = state => ({
+export const mapStateToProps = state => ({
   enterpriseSlug: state.portalConfiguration.enterpriseSlug,
 });
 

diff --git a/src/components/ContentHighlights/HighlightStepper/ContentSearchResultCard.jsx b/src/components/ContentHighlights/HighlightStepper/ContentSearchResultCard.jsx
@@ -17,7 +17,7 @@ const ContentSearchResultCard = ({ enterpriseSlug, original }) => {
   return (
     <ContentHighlightCardItem
       title={title}
-      hyperlink={generateAboutPageUrl(enterpriseSlug, contentType.toLowerCase(), aggregationKey.split(':')[1])}
+      hyperlink={generateAboutPageUrl(enterpriseSlug, contentType?.toLowerCase(), aggregationKey?.split(':')[1])}
       contentType={contentType}
       partners={partners}
       cardImageUrl={cardImageUrl || originalImageUrl}

diff --git a/src/components/ContentHighlights/HighlightStepper/HighlightStepperSelectContentSearch.jsx b/src/components/ContentHighlights/HighlightStepper/HighlightStepperSelectContentSearch.jsx
@@ -91,7 +91,6 @@ const BaseHighlightStepperSelectContentDataTable = ({
   searchResults,
 }) => {
   const [currentView, setCurrentView] = useState(defaultActiveStateValue);
-
   const tableData = useMemo(() => camelCaseObject(searchResults?.hits || []), [searchResults]);
   const searchResultsItemCount = searchResults?.nbHits || 0;
   const searchResultsPageCount = searchResults?.nbPages || 0;

diff --git a/src/components/ContentHighlights/HighlightStepper/tests/ContentConfirmContentCard.test.jsx b/src/components/ContentHighlights/HighlightStepper/tests/ContentConfirmContentCard.test.jsx
@@ -1,18 +1,32 @@
 import React, { useState } from 'react';
-import { screen, render, fireEvent } from '@testing-library/react';
+import { screen, fireEvent } from '@testing-library/react';
 import '@testing-library/jest-dom/extend-expect';
 import algoliasearch from 'algoliasearch/lite';
+import { Provider } from 'react-redux';
+import configureMockStore from 'redux-mock-store';
+import { renderWithRouter } from '@edx/frontend-enterprise-utils';
 import ContentConfirmContentCard from '../ContentConfirmContentCard';
 import { testCourseData, testCourseAggregation, FOOTER_TEXT_BY_CONTENT_TYPE } from '../../data/constants';
 import { ContentHighlightsContext } from '../../ContentHighlightsContext';
 import { configuration } from '../../../../config';
 
+const mockStore = configureMockStore();
+const initialState = {
+  portalConfiguration:
+    {
+      enterpriseSlug: 'test-enterprise',
+    },
+};
+
 const searchClient = algoliasearch(
   configuration.ALGOLIA.APP_ID,
   configuration.ALGOLIA.SEARCH_API_KEY,
 );
 
-const ContentHighlightContentCardWrapper = () => {
+const ContentHighlightContentCardWrapper = ({
+  // eslint-disable-next-line react/prop-types
+  store = mockStore(initialState),
+}) => {
   const contextValue = useState({
     stepperModal: {
       isOpen: false,
@@ -24,15 +38,18 @@ const ContentHighlightContentCardWrapper = () => {
     searchClient,
   });
   return (
-    <ContentHighlightsContext.Provider value={contextValue}>
-      {testCourseData.map((original) => <ContentConfirmContentCard original={original} />)}
-    </ContentHighlightsContext.Provider>
+    <Provider store={store}>
+      <ContentHighlightsContext.Provider value={contextValue}>
+        {testCourseData.map((original) => <ContentConfirmContentCard original={original} />)}
+      </ContentHighlightsContext.Provider>
+    </Provider>
   );
 };
 
 describe('<ContentConfirmContentCard />', () => {
   it('renders the correct content', () => {
-    render(<ContentHighlightContentCardWrapper />);
+    renderWithRouter(<ContentHighlightContentCardWrapper />);
+
     for (let i = 0; i < testCourseData.length; i++) {
       expect(screen.getByText(testCourseData[i].title)).toBeInTheDocument();
       expect(screen.getByText(testCourseData[i].firstEnrollablePaidSeatPrice, { exact: false })).toBeInTheDocument();
@@ -42,7 +59,8 @@ describe('<ContentConfirmContentCard />', () => {
     }
   });
   it('deletes the correct content', () => {
-    render(<ContentHighlightContentCardWrapper />);
+    renderWithRouter(<ContentHighlightContentCardWrapper />);
+
     const deleteButton = screen.getAllByRole('button', { 'aria-label': 'Delete' });
     fireEvent.click(deleteButton[0]);
     for (let i = 0; i < testCourseData.length; i++) {

diff --git a/src/components/ContentHighlights/HighlightStepper/tests/ContentHighlightStepper.test.jsx b/src/components/ContentHighlights/HighlightStepper/tests/ContentHighlightStepper.test.jsx
@@ -18,6 +18,7 @@ const mockStore = configureMockStore([thunk]);
 const initialState = {
   portalConfiguration: {
     enterpriseSlug: 'test-enterprise',
+    enterpriseId: 'test-enterprise-id',
   },
 };
 

diff --git a/...mponents/ContentHighlights/HighlightStepper/tests/HighlightStepperConfirmContent.test.jsx b/...mponents/ContentHighlights/HighlightStepper/tests/HighlightStepperConfirmContent.test.jsx
@@ -3,13 +3,24 @@ import { screen } from '@testing-library/react';
 import '@testing-library/jest-dom/extend-expect';
 import algoliasearch from 'algoliasearch/lite';
 import { renderWithRouter } from '@edx/frontend-enterprise-utils';
+import configureMockStore from 'redux-mock-store';
+import thunk from 'redux-thunk';
+import { Provider } from 'react-redux';
 import HighlightStepperConfirmContent, { BaseReviewContentSelections, SelectedContent } from '../HighlightStepperConfirmContent';
 import {
   testCourseAggregation,
 } from '../../data/constants';
 import { ContentHighlightsContext } from '../../ContentHighlightsContext';
 import { configuration } from '../../../../config';
 
+const mockStore = configureMockStore([thunk]);
+const initialState = {
+  portalConfiguration:
+    {
+      enterpriseSlug: 'test-enterprise',
+    },
+};
+
 const searchClient = algoliasearch(
   configuration.ALGOLIA.APP_ID,
   configuration.ALGOLIA.SEARCH_API_KEY,
@@ -29,9 +40,11 @@ const HighlightStepperConfirmContentWrapper = ({ children, currentSelectedRowIds
     searchClient,
   });
   return (
-    <ContentHighlightsContext.Provider value={contextValue}>
-      {children}
-    </ContentHighlightsContext.Provider>
+    <Provider store={mockStore(initialState)}>
+      <ContentHighlightsContext.Provider value={contextValue}>
+        {children}
+      </ContentHighlightsContext.Provider>
+    </Provider>
   );
 };