fix: Fix rows counter in the Edit Grade modal window

Merge pull request #386 from DmytroAlipov/fix-second-opening-grade-quince
fix: "An unexpected error occurred" after second opening any grade
2025-01-26 21:34:11 +05:30 · 2024-04-09 14:11:57 +03:00 · 2024-04-03 15:27:59 -03:00 · 2024-03-16 18:26:24 +02:00 · 2023-12-10 20:47:54 -03:00 · 2023-12-08 16:27:28 -03:00
28 changed files with 4232 additions and 22669 deletions
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -25,7 +25,7 @@ jobs:
        node-version: ${{ env.NODE_VER }}

    - name: Install dependencies
-      run: npm ci --legacy-peer-deps
+      run: npm ci

    - name: Unit Tests
      run: npm run test
--- a/package-lock.json
+++ b/package-lock.json
--- a/package.json
+++ b/package.json
@@ -29,9 +29,9 @@
  ],
  "dependencies": {
    "@edx/brand": "npm:@edx/brand-openedx@^1.2.0",
-    "@edx/frontend-component-footer": "12.1.0",
-    "@edx/frontend-component-header": "4.3.0",
-    "@edx/frontend-platform": "4.6.0",
+    "@edx/frontend-component-footer": "12.2.0",
+    "@edx/frontend-component-header": "4.6.0",
+    "@edx/frontend-platform": "5.5.4",
    "@edx/paragon": "20.45.0",
    "@edx/react-unit-test-utils": "1.7.0",
    "@edx/reactifex": "^2.1.1",
@@ -54,8 +54,8 @@
    "react-dom": "17.0.2",
    "react-helmet": "^6.1.0",
    "react-redux": "^7.2.9",
-    "react-router": "5.2.0",
-    "react-router-dom": "5.2.0",
+    "react-router": "6.15.0",
+    "react-router-dom": "6.15.0",
    "react-router-redux": "^5.0.0-alpha.9",
    "redux": "4.0.5",
    "redux-beacon": "^2.1.0",
@@ -68,9 +68,9 @@
  },
  "devDependencies": {
    "@edx/browserslist-config": "^1.1.1",
+    "@edx/frontend-build": "12.9.3",
    "@testing-library/react": "12.1.5",
    "@wojtekmaj/enzyme-adapter-react-17": "0.8.0",
-    "@edx/frontend-build": "12.9.0-alpha.6",
    "axios": "0.21.2",
    "axios-mock-adapter": "^1.17.0",
    "es-check": "^2.3.0",
--- a/src/App.jsx
+++ b/src/App.jsx
@@ -1,12 +1,11 @@
 import React from 'react';
-import { BrowserRouter as Router, Route, Switch } from 'react-router-dom';
+import { Route, Routes } from 'react-router-dom';

 import { AppProvider } from '@edx/frontend-platform/react';

 import Footer from '@edx/frontend-component-footer';
 import Header from '@edx/frontend-component-header';

-import { routePath } from 'data/constants/app';
 import store from 'data/store';
 import GradebookPage from 'containers/GradebookPage';
 import './App.scss';
@@ -15,21 +14,18 @@ import Head from './head/Head';
 const App = () => (
  <AppProvider store={store}>
    <Head />
-    <Router>
-      <div>
-        <Header />
-        <main>
-          <Switch>
-            <Route
-              exact
-              path={routePath}
-              component={GradebookPage}
-            />
-          </Switch>
-        </main>
-        <Footer logo={process.env.LOGO_POWERED_BY_OPEN_EDX_URL_SVG} />
-      </div>
-    </Router>
+    <div>
+      <Header />
+      <main>
+        <Routes>
+          <Route
+            path="/:courseId"
+            element={<GradebookPage />}
+          />
+        </Routes>
+      </main>
+      <Footer logo={process.env.LOGO_POWERED_BY_OPEN_EDX_URL_SVG} />
+    </div>
  </AppProvider>
 );

--- a/src/App.test.jsx
+++ b/src/App.test.jsx
@@ -1,13 +1,12 @@
 import React from 'react';
 import { shallow } from 'enzyme';

-import { BrowserRouter as Router, Route, Switch } from 'react-router-dom';
+import { Route, Routes } from 'react-router-dom';
 import { AppProvider } from '@edx/frontend-platform/react';

 import Footer from '@edx/frontend-component-footer';
 import Header from '@edx/frontend-component-header';

-import { routePath } from 'data/constants/app';
 import store from 'data/store';
 import GradebookPage from 'containers/GradebookPage';

@@ -17,14 +16,10 @@ import Head from './head/Head';
 jest.mock('react-router-dom', () => ({
  BrowserRouter: () => 'BrowserRouter',
  Route: () => 'Route',
-  Switch: () => 'Switch',
 }));
 jest.mock('@edx/frontend-platform/react', () => ({
  AppProvider: () => 'AppProvider',
 }));
-jest.mock('data/constants/app', () => ({
-  routePath: '/:courseId',
-}));
 jest.mock('@edx/frontend-component-footer', () => 'Footer');
 jest.mock('data/store', () => 'testStore');
 jest.mock('containers/GradebookPage', () => 'GradebookPage');
@@ -32,7 +27,7 @@ jest.mock('@edx/frontend-component-header', () => 'Header');

 const logo = 'fakeLogo.png';
 let el;
-let router;
+let secondChild;

 describe('App router component', () => {
  test('snapshot', () => {
@@ -42,7 +37,7 @@ describe('App router component', () => {
    beforeEach(() => {
      process.env.LOGO_POWERED_BY_OPEN_EDX_URL_SVG = logo;
      el = shallow(<App />);
-      router = el.childAt(1);
+      secondChild = el.childAt(1);
    });
    describe('AppProvider', () => {
      test('AppProvider is the parent component, passed the redux store props', () => {
@@ -57,24 +52,24 @@ describe('App router component', () => {
    });
    describe('Router', () => {
      test('second child of AppProvider', () => {
-        expect(router.type()).toBe(Router);
+        expect(secondChild.type()).toBe('div');
      });
      test('Header is above/outside-of the routing', () => {
-        expect(router.childAt(0).childAt(0).type()).toBe(Header);
-        expect(router.childAt(0).childAt(1).type()).toBe('main');
+        expect(secondChild.childAt(0).type()).toBe(Header);
+        expect(secondChild.childAt(1).type()).toBe('main');
      });
      test('Routing - GradebookPage is only route', () => {
-        expect(router.find('main')).toEqual(shallow(
+        expect(secondChild.find('main')).toEqual(shallow(
          <main>
-            <Switch>
-              <Route exact path={routePath} component={GradebookPage} />
-            </Switch>
+            <Routes>
+              <Route path="/:courseId" element={<GradebookPage />} />
+            </Routes>
          </main>,
        ));
      });
    });
    test('Footer logo drawn from env variable', () => {
-      expect(router.find(Footer).props().logo).toEqual(logo);
+      expect(secondChild.find(Footer).props().logo).toEqual(logo);
    });
  });
 });
--- a/src/snapshots/App.test.jsx.snap
+++ b/src/snapshots/App.test.jsx.snap
@@ -5,20 +5,17 @@ exports[`App router component snapshot 1`] = `
  store="testStore"
 >
  <Head />
-  <BrowserRouter>
-    <div>
-      <Header />
-      <main>
-        <Switch>
-          <Route
-            component="GradebookPage"
-            exact={true}
-            path="/:courseId"
-          />
-        </Switch>
-      </main>
-      <Footer />
-    </div>
-  </BrowserRouter>
+  <div>
+    <Header />
+    <main>
+      <Component>
+        <Route
+          element={<GradebookPage />}
+          path="/:courseId"
+        />
+      </Component>
+    </main>
+    <Footer />
+  </div>
 </AppProvider>
 `;
--- a/src/components/GradebookHeader/snapshots/index.test.jsx.snap
+++ b/src/components/GradebookHeader/snapshots/index.test.jsx.snap
@@ -21,7 +21,9 @@ exports[`GradebookHeader component render default view shapshot 1`] = `
  <div
    className="subtitle-row d-flex justify-content-between align-items-center"
  >
-    <h2>
+    <h2
+      className="text-break"
+    >
      test-course-id
    </h2>
  </div>
@@ -49,7 +51,9 @@ exports[`GradebookHeader component render frozen grades snapshot: show frozen wa
  <div
    className="subtitle-row d-flex justify-content-between align-items-center"
  >
-    <h2>
+    <h2
+      className="text-break"
+    >
      test-course-id
    </h2>
  </div>
@@ -83,7 +87,9 @@ exports[`GradebookHeader component render show bulk management snapshot: show to
  <div
    className="subtitle-row d-flex justify-content-between align-items-center"
  >
-    <h2>
+    <h2
+      className="text-break"
+    >
      test-course-id
    </h2>
    <Button
@@ -117,7 +123,9 @@ exports[`GradebookHeader component render user cannot view gradebook snapshot: s
  <div
    className="subtitle-row d-flex justify-content-between align-items-center"
  >
-    <h2>
+    <h2
+      className="text-break"
+    >
      test-course-id
    </h2>
  </div>
--- a/src/components/GradebookHeader/index.jsx
+++ b/src/components/GradebookHeader/index.jsx
@@ -26,7 +26,7 @@ export const GradebookHeader = () => {
      </a>
      <h1>{formatMessage(messages.gradebook)}</h1>
      <div className="subtitle-row d-flex justify-content-between align-items-center">
-        <h2>{courseId}</h2>
+        <h2 className="text-break">{courseId}</h2>
        {showBulkManagement && (
          <Button variant="tertiary" onClick={handleToggleViewClick}>
            {formatMessage(toggleViewMessage)}
--- a/src/components/GradesView/EditModal/OverrideTable/snapshots/index.test.jsx.snap
+++ b/src/components/GradesView/EditModal/OverrideTable/snapshots/index.test.jsx.snap
@@ -20,6 +20,6 @@ exports[`OverrideTable component render snapshot 1`] = `
      },
    ]
  }
-  itemCount={2}
+  itemCount={3}
 />
 `;
--- a/src/components/GradesView/EditModal/OverrideTable/hooks.js
+++ b/src/components/GradesView/EditModal/OverrideTable/hooks.js
@@ -9,7 +9,7 @@ const useOverrideTableData = () => {
  const { formatMessage } = useIntl();

  const hide = selectors.grades.useHasOverrideErrors();
-  const gradeOverrides = selectors.grades.useGradeData().gradeOverrideHistoryResults;
+  const gradeOverrides = selectors.grades.useGradeData().gradeOverrideHistoryResults || [];
  const tableProps = {};
  if (!hide) {
    tableProps.columns = [
--- a/src/components/GradesView/EditModal/OverrideTable/index.jsx
+++ b/src/components/GradesView/EditModal/OverrideTable/index.jsx
@@ -20,18 +20,20 @@ export const OverrideTable = () => {

  if (hide) { return null; }

+  const tableData = [
+    ...data,
+    {
+      adjustedGrade: <AdjustedGradeInput />,
+      date: formatDateForDisplay(new Date()),
+      reason: <ReasonInput />,
+    },
+  ];
+
  return (
    <DataTable
      columns={columns}
-      data={[
-        ...data,
-        {
-          adjustedGrade: <AdjustedGradeInput />,
-          date: formatDateForDisplay(new Date()),
-          reason: <ReasonInput />,
-        },
-      ]}
-      itemCount={data.length}
+      data={tableData}
+      itemCount={tableData.length}
    />
  );
 };
--- a/src/components/GradesView/FilterMenuToggle/snapshots/index.test.jsx.snap
+++ b/src/components/GradesView/FilterMenuToggle/snapshots/index.test.jsx.snap
@@ -7,6 +7,7 @@ exports[`FilterMenuToggle component render snapshot 1`] = `
  onClick={[MockFunction hooks.toggleFilterMenu]}
 >
  <Icon
+    className="mr-1"
    src="FilterAlt"
  />
   
--- a/src/components/GradesView/FilterMenuToggle/index.jsx
+++ b/src/components/GradesView/FilterMenuToggle/index.jsx
@@ -21,7 +21,7 @@ export const FilterMenuToggle = () => {
      className="btn-primary align-self-start"
      onClick={toggleFilterMenu}
    >
-      <Icon src={FilterAlt} /> {formatMessage(messages.editFilters)}
+      <Icon src={FilterAlt} className="mr-1" /> {formatMessage(messages.editFilters)}
    </Button>
  );
 };
--- a/src/components/GradesView/GradebookTable/GradeButton.jsx
+++ b/src/components/GradesView/GradebookTable/GradeButton.jsx
@@ -11,7 +11,7 @@ export const useGradeButtonData = ({ entry, subsection }) => {
  const areGradesFrozen = selectors.assignmentTypes.useAreGradesFrozen();
  const { gradeFormat } = selectors.grades.useGradeData();
  const setModalState = thunkActions.app.useSetModalStateFromTable();
-  const label = transforms.grades.subsectionGrade({ gradeFormat, subsection });
+  const label = transforms.grades.subsectionGrade({ gradeFormat, subsection })();

  const onClick = () => {
    setModalState({
--- a/src/components/GradesView/GradebookTable/GradeButton.test.jsx
+++ b/src/components/GradesView/GradebookTable/GradeButton.test.jsx
@@ -41,7 +41,7 @@ const props = {
 };
 const gradeFormat = 'percent';
 const setModalState = jest.fn();
-const subsectionGrade = 'test-subsection-grade';
+const subsectionGrade = () => 'test-subsection-grade';
 selectors.assignmentTypes.useAreGradesFrozen.mockReturnValue(false);
 selectors.grades.useGradeData.mockReturnValue({ gradeFormat });
 thunkActions.app.useSetModalStateFromTable.mockReturnValue(setModalState);
@@ -73,7 +73,7 @@ describe('GradeButton', () => {
        expect(out.areGradesFrozen).toEqual(false);
      });
      test('label passed from subsection grade redux hook', () => {
-        expect(out.label).toEqual(subsectionGrade);
+        expect(out.label).toEqual(subsectionGrade());
      });
      test('onClick sets modal state with user entry and subsection', () => {
        out.onClick();
--- a/src/components/GradesView/GradebookTable/hooks.jsx
+++ b/src/components/GradesView/GradebookTable/hooks.jsx
@@ -39,7 +39,7 @@ export const useGradebookTableData = () => {
    [Headings.username]: (
      <Fields.Username username={entry.username} userKey={entry.external_user_key} />
    ),
-    [Headings.email]: (<Fields.Email email={entry.email} />),
+    [Headings.email]: (<Fields.Text value={entry.email} />),
    [Headings.totalGrade]: `${roundGrade(entry.percent * 100)}${getLocalizedPercentSign()}`,
    ...entry.section_breakdown.reduce((acc, subsection) => ({
      ...acc,
--- a/src/components/GradesView/GradebookTable/hooks.test.jsx
+++ b/src/components/GradesView/GradebookTable/hooks.test.jsx
@@ -22,7 +22,7 @@ jest.mock('i18n/utils', () => ({
 jest.mock('./GradeButton', () => 'GradeButton');
 jest.mock('./Fields', () => jest.requireActual('testUtils').mockNestedComponents({
  Username: 'Fields.Username',
-  Email: 'Fields.Email',
+  Text: 'Fields.Text',
 }));
 jest.mock('./LabelReplacements', () => jest.requireActual('testUtils').mockNestedComponents({
  TotalGradeLabelReplacement: 'LabelReplacements.TotalGradeLabelReplacement',
@@ -158,7 +158,7 @@ describe('useGradebookTableData', () => {
      test('email field', () => {
        allGrades.forEach((entry, index) => {
          expect(out.data[index][Headings.email]).toMatchObject(
-            <Fields.Email email={entry.email} />,
+            <Fields.Text value={entry.email} />,
          );
        });
      });
--- a/src/components/GradesView/GradesView.scss
+++ b/src/components/GradesView/GradesView.scss
@@ -46,6 +46,7 @@
 }
 .grade-history-header{
    float: left;
+    min-width: 170px;
 }

 .grade-history-assignment{
@@ -65,7 +66,7 @@
 .gradebook-container {
    width: 100%;
    overflow-x: auto;
-    height: 600px;
+    max-height: 600px;
    overflow-y: auto;
    position: relative;
 }
@@ -122,3 +123,34 @@ select#ScoreView.form-control {
        border-right-color: $black;
    }
 }
+
+#edit-filters-btn {
+    @include media-breakpoint-down(xs) {
+        width: 100%;
+        margin-bottom: 1rem;
+    }
+}
+
+.search-container {
+    @include media-breakpoint-down(xs) {
+        width: 100%;
+    }
+}
+
+.pgn__modal-body-content .pgn__data-table-layout-wrapper {
+    @include media-breakpoint-down(sm) {
+        clear: both;
+        padding: 1rem 0;
+    }
+}
+
+.page-gradebook {
+    position: relative;
+
+    .sidebar-container {
+        position: relative;
+    }
+    aside.sidebar {
+        overflow: auto;
+    }
+}
--- a/src/components/GradesView/SearchControls/snapshots/index.test.jsx.snap
+++ b/src/components/GradesView/SearchControls/snapshots/index.test.jsx.snap
@@ -1,7 +1,9 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP

 exports[`SearchControls component render snapshot 1`] = `
-<div>
+<div
+  className="search-container"
+>
  <SearchField
    inputLabel="test-input-label"
    onBlur={[MockFunction hooks.onBlur]}
--- a/src/components/GradesView/SearchControls/index.jsx
+++ b/src/components/GradesView/SearchControls/index.jsx
@@ -18,7 +18,7 @@ export const SearchControls = () => {
  } = useSearchControlsData();

  return (
-    <div>
+    <div className="search-container">
      <SearchField
        onSubmit={onSubmit}
        inputLabel={inputLabel}
--- a/src/components/GradesView/snapshots/index.test.jsx.snap
+++ b/src/components/GradesView/snapshots/index.test.jsx.snap
@@ -10,7 +10,7 @@ exports[`GradesView component render snapshot 1`] = `
    filter-step-heading
  </h3>
  <div
-    className="d-flex justify-content-between"
+    className="d-flex justify-content-between flex-wrap"
  >
    <FilterMenuToggle />
    <SearchControls />
--- a/src/components/GradesView/index.jsx
+++ b/src/components/GradesView/index.jsx
@@ -34,7 +34,7 @@ export const GradesView = ({ updateQueryParams }) => {
        {stepHeadings.filter}
      </h3>

-      <div className="d-flex justify-content-between">
+      <div className="d-flex justify-content-between flex-wrap">
        <FilterMenuToggle />
        <SearchControls />
      </div>
--- a/src/containers/GradebookPage/index.jsx
+++ b/src/containers/GradebookPage/index.jsx
@@ -14,6 +14,8 @@ import GradesView from 'components/GradesView';
 import GradebookFilters from 'components/GradebookFilters';
 import BulkManagementHistoryView from 'components/BulkManagementHistoryView';

+import { withParams, withNavigate, withLocation } from '../../utils/hoc';
+
 /**
 * <GradebookPage />
 * Top-level view for the Gradebook MFE.
@@ -28,10 +30,11 @@ export class GradebookPage extends React.Component {

  componentDidMount() {
    const urlQuery = queryString.parse(this.props.location.search);
-    this.props.initializeApp(this.props.match.params.courseId, urlQuery);
+    this.props.initializeApp(this.props.courseId, urlQuery);
  }

  updateQueryParams(queryParams) {
+    const { pathname } = this.props.location;
    const parsed = queryString.parse(this.props.location.search);
    Object.keys(queryParams).forEach((key) => {
      if (queryParams[key]) {
@@ -40,7 +43,7 @@ export class GradebookPage extends React.Component {
        delete parsed[key];
      }
    });
-    this.props.history.push(`?${queryString.stringify(parsed)}`);
+    this.props.navigate({ pathname, search: `?${queryString.stringify(parsed)}` });
  }

  render() {
@@ -60,18 +63,12 @@ export class GradebookPage extends React.Component {
  }
 }
 GradebookPage.defaultProps = {
-  location: { search: '' },
+  location: { pathname: '/', search: '' },
 };
 GradebookPage.propTypes = {
-  history: PropTypes.shape({
-    push: PropTypes.func,
-  }).isRequired,
-  location: PropTypes.shape({ search: PropTypes.string }),
-  match: PropTypes.shape({
-    params: PropTypes.shape({
-      courseId: PropTypes.string,
-    }),
-  }).isRequired,
+  navigate: PropTypes.func.isRequired,
+  location: PropTypes.shape({ pathname: PropTypes.string, search: PropTypes.string }),
+  courseId: PropTypes.string.isRequired,
  // redux
  activeView: PropTypes.string.isRequired,
  initializeApp: PropTypes.func.isRequired,
@@ -85,4 +82,4 @@ export const mapDispatchToProps = {
  initializeApp: thunkActions.app.initialize,
 };

-export default connect(mapStateToProps, mapDispatchToProps)(GradebookPage);
+export default connect(mapStateToProps, mapDispatchToProps)(withParams(withNavigate(withLocation(GradebookPage))));
--- a/src/containers/GradebookPage/test.jsx
+++ b/src/containers/GradebookPage/test.jsx
@@ -50,14 +50,15 @@ describe('GradebookPage', () => {
    let el;
    const props = {
      location: {
+        pathname: '/',
        search: 'searchString',
      },
-      match: { params: { courseId } },
+      courseId,
      activeView: views.grades,
    };
    beforeEach(() => {
      props.initializeApp = jest.fn();
-      props.history = { push: jest.fn() };
+      props.navigate = jest.fn();
    });
    test('snapshot - shows BulkManagementHistoryView if activeView === views.bulkManagementHistory', () => {
      el = shallow(<GradebookPage {...props} activeView={views.bulkManagementHistory} />);
@@ -130,7 +131,7 @@ describe('GradebookPage', () => {
          const val2 = 'VALTWO!!';
          const args = { [newKey]: val1, [props.location.search]: val2 };
          el.instance().updateQueryParams(args);
-          expect(props.history.push).toHaveBeenCalledWith(`?${queryString.stringify(args)}`);
+          expect(props.navigate).toHaveBeenCalledWith({ pathname: '/', search: `?${queryString.stringify(args)}` });
        });
        it('clears values for non-truthy values', () => {
          queryString.parse.mockImplementation(key => ({ [key]: key }));
@@ -139,8 +140,8 @@ describe('GradebookPage', () => {
          const val2 = false;
          const args = { [newKey]: val1, [props.location.search]: val2 };
          el.instance().updateQueryParams(args);
-          expect(props.history.push).toHaveBeenCalledWith(
-            `?${queryString.stringify({ [newKey]: val1 })}`,
+          expect(props.navigate).toHaveBeenCalledWith(
+            { pathname: '/', search: `?${queryString.stringify({ [newKey]: val1 })}` },
          );
        });
      });
--- a/src/data/constants/app.js
+++ b/src/data/constants/app.js
@@ -1,7 +1,4 @@
 import { StrictDict } from 'utils';
-import { getConfig } from '@edx/frontend-platform';
-
-export const routePath = `${getConfig().PUBLIC_PATH}:courseId`;

 export const views = StrictDict({
  grades: 'grades',
--- a/src/data/thunkActions/roles.js
+++ b/src/data/thunkActions/roles.js
@@ -10,7 +10,7 @@ import { fetchGrades } from './grades';
 import { fetchTracks } from './tracks';
 import { fetchAssignmentTypes } from './assignmentTypes';

-export const allowedRoles = ['staff', 'instructor', 'support'];
+export const allowedRoles = ['staff', 'limited_staff', 'instructor', 'support'];

 export const fetchRoles = () => (
  (dispatch, getState) => {
--- a/src/utils/hoc.jsx
+++ b/src/utils/hoc.jsx
@@ -0,0 +1,24 @@
+import React from 'react';
+
+import { useLocation, useNavigate, useParams } from 'react-router-dom';
+
+export const withParams = WrappedComponent => {
+  const WithParamsComponent = props => <WrappedComponent {...useParams()} {...props} />;
+  return WithParamsComponent;
+};
+
+export const withNavigate = WrappedComponent => {
+  const WithNavigateComponent = props => {
+    const navigate = useNavigate();
+    return <WrappedComponent navigate={navigate} {...props} />;
+  };
+  return WithNavigateComponent;
+};
+
+export const withLocation = WrappedComponent => {
+  const WithLocationComponent = props => {
+    const location = useLocation();
+    return <WrappedComponent location={location} {...props} />;
+  };
+  return WithLocationComponent;
+};
--- a/src/utils/hoc.test.jsx
+++ b/src/utils/hoc.test.jsx
@@ -0,0 +1,38 @@
+import React from 'react';
+import { mount } from 'enzyme';
+
+import { withLocation, withNavigate } from './hoc';
+
+const mockedNavigator = jest.fn();
+
+jest.mock('react-router-dom', () => ({
+  useNavigate: () => mockedNavigator,
+  useLocation: () => ({
+    pathname: '/current-location',
+  }),
+}));
+
+// eslint-disable-next-line react/prop-types
+const MockComponent = ({ navigate, location }) => (
+  // eslint-disable-next-line react/button-has-type, react/prop-types
+  <button id="btn" onClick={() => navigate('/some-route')}>{location.pathname}</button>
+);
+const WrappedComponent = withNavigate(withLocation(MockComponent));
+
+test('Provide Navigation to Component', () => {
+  const wrapper = mount(
+    <WrappedComponent />,
+  );
+  const btn = wrapper.find('#btn');
+  btn.simulate('click');
+
+  expect(mockedNavigator).toHaveBeenCalledWith('/some-route');
+});
+
+test('Provide Location object to Component', () => {
+  const wrapper = mount(
+    <WrappedComponent />,
+  );
+
+  expect(wrapper.find('#btn').text()).toContain('/current-location');
+});
Author	SHA1	Message	Date
Stanislav Lunyachek	706d0ba36b	fix: Fix rows counter in the Edit Grade modal window	2025-01-26 21:34:11 +05:30
Max Sokolski	aa2b1a56bc	Merge pull request #386 from DmytroAlipov/fix-second-opening-grade-quince fix: "An unexpected error occurred" after second opening any grade	2024-04-09 14:11:57 +03:00
Adolfo R. Brandes	e795163aa6	Merge pull request #370 from raccoongang/fix/fix-responsive-issues-quince fix: fix sidebar scrolling and adaptation for mobile	2024-04-03 15:27:59 -03:00
Dima Alipov	bd25b6b4d0	fix: "An unexpected error occurred" after second opening any grade	2024-03-16 18:26:24 +02:00
Adolfo R. Brandes	acdad5af00	Merge pull request #376 from arbrandes/fix-route-quince [Quince backport] fix: Route with PUBLIC_PATH	2023-12-10 20:47:54 -03:00
Adolfo R. Brandes	ba9fb375d9	fix: Route with PUBLIC_PATH Prior to this change, the MFE would fail to render with any PUBLIC_PATH set. Because that is now handled entirely by `frontend-platform`, we can avoid referring to PUBLIC_PATH entirely, here.	2023-12-08 16:27:28 -03:00
ihor-romaniuk	f303712830	fix: fix sidebar scrolling and adaptation for mobile - fix text overload in heading - fix appearance for sidebar with a short main content - fix transformation of search and filter button on mobile - fix extra empty space below main content table - fix adaptation modal content to mobile view	2023-10-27 10:34:00 +03:00
Syed Ali Abbas Zaidi	b536180a3e	chore: bump frontend-platform (#356 )	2023-10-18 11:22:51 +05:00
0x29a	295048b4e9	fix: add limited_staff to allowedRoles	2023-10-10 10:29:21 -04:00
Diana Catalina Olarte	1c70458590	fix: replace Field.Email reference for Field.Text	2023-09-25 11:10:13 -03:00
Syed Ali Abbas Zaidi	247e9f3668	feat: upgrade react router to v6 (#319 ) * feat: upgrade react router to v6 * refactor: removed remaining router v5 code * refactor: improve code coverage	2023-09-18 11:27:40 -04:00