chore: Clean up learnerDashboardHeader files (#614)

src/containers/LearnerDashboardHeader/BrandLogo.jsx

@@ -1,27 +0,0 @@
-import React from 'react';
-
-import { useIntl } from '@edx/frontend-platform/i18n';
-
-import { reduxHooks } from 'hooks';
-
-import { getConfig } from '@edx/frontend-platform';
-import messages from './messages';
-
-export const BrandLogo = () => {
-  const { formatMessage } = useIntl();
-  const dashboard = reduxHooks.useEnterpriseDashboardData();
-
-  return (
-    <a href={dashboard?.url || '/'} className="mx-auto">
-      <img
-        className="logo py-3"
-        src={getConfig().LOGO_URL}
-        alt={formatMessage(messages.logoAltText)}
-      />
-    </a>
-  );
-};
-
-BrandLogo.propTypes = {};
-
-export default BrandLogo;

src/containers/LearnerDashboardHeader/BrandLogo.test.jsx

@@ -1,28 +0,0 @@
-import { shallow } from '@edx/react-unit-test-utils';
-
-import { reduxHooks } from 'hooks';
-import BrandLogo from './BrandLogo';
-
-jest.mock('hooks', () => ({
-  reduxHooks: {
-    useEnterpriseDashboardData: jest.fn(),
-  },
-}));
-
-describe('BrandLogo', () => {
-  test('dashboard defined', () => {
-    reduxHooks.useEnterpriseDashboardData.mockReturnValueOnce({
-      url: 'url',
-    });
-    const wrapper = shallow(<BrandLogo />);
-    expect(wrapper.snapshot).toMatchSnapshot();
-    expect(wrapper.instance.findByType('a')[0].props.href).toEqual('url');
-  });
-
-  test('dashboard undefined', () => {
-    reduxHooks.useEnterpriseDashboardData.mockReturnValueOnce(null);
-    const wrapper = shallow(<BrandLogo />);
-    expect(wrapper.snapshot).toMatchSnapshot();
-    expect(wrapper.instance.findByType('a')[0].props.href).toEqual('/');
-  });
-});

src/containers/LearnerDashboardHeader/__snapshots__/BrandLogo.test.jsx.snap

@@ -1,27 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`BrandLogo dashboard defined 1`] = `
-<a
-  className="mx-auto"
-  href="url"
->
-  <img
-    alt="edX, Inc. Dashboard"
-    className="logo py-3"
-    src="https://edx-cdn.org/v3/default/logo.svg"
-  />
-</a>
-`;
-
-exports[`BrandLogo dashboard undefined 1`] = `
-<a
-  className="mx-auto"
-  href="/"
->
-  <img
-    alt="edX, Inc. Dashboard"
-    className="logo py-3"
-    src="https://edx-cdn.org/v3/default/logo.svg"
-  />
-</a>
-`;

src/containers/LearnerDashboardHeader/hooks.js

@@ -1,5 +1,4 @@
 import React from 'react';
-import { useWindowSize, breakpoints } from '@openedx/paragon';
 import { useIntl } from '@edx/frontend-platform/i18n';
 import track from 'tracking';
 import { StrictDict } from 'utils';
@@ -7,26 +6,14 @@ import { linkNames } from 'tracking/constants';
 
 import getLearnerHeaderMenu from './LearnerDashboardMenu';
 
-import * as module from './hooks';
-
 export const state = StrictDict({
   isOpen: (val) => React.useState(val), // eslint-disable-line
 });
 
-export const useIsCollapsed = () => {
-  const { width } = useWindowSize();
-  const isCollapsed = React.useMemo(() => (width <= breakpoints.large.minWidth), [width]);
-  return isCollapsed;
-};
-
 export const findCoursesNavClicked = (href) => track.findCourses.findCoursesClicked(href, {
   linkName: linkNames.learnerHomeNavExplore,
 });
 
-export const findCoursesNavDropdownClicked = (href) => track.findCourses.findCoursesClicked(href, {
-  linkName: linkNames.learnerHomeNavDropdownExplore,
-});
-
 export const useLearnerDashboardHeaderMenu = ({
   courseSearchUrl, authenticatedUser, exploreCoursesClick,
 }) => {
@@ -34,20 +21,7 @@ export const useLearnerDashboardHeaderMenu = ({
   return getLearnerHeaderMenu(formatMessage, courseSearchUrl, authenticatedUser, exploreCoursesClick);
 };
 
-export const useLearnerDashboardHeaderData = () => {
-  const [isOpen, setIsOpen] = module.state.isOpen(false);
-  const toggleIsOpen = () => setIsOpen(!isOpen);
-
-  return {
-    isOpen,
-    toggleIsOpen,
-  };
-};
-
 export default {
-  useIsCollapsed,
   findCoursesNavClicked,
-  findCoursesNavDropdownClicked,
-  useLearnerDashboardHeaderData,
   useLearnerDashboardHeaderMenu,
 };

src/containers/LearnerDashboardHeader/hooks.test.js

@@ -1,4 +1,3 @@
-import { useWindowSize, breakpoints } from '@openedx/paragon';
 import track from 'tracking';
 import { linkNames } from 'tracking/constants';
 
@@ -9,10 +8,7 @@ import * as hooks from './hooks';
 const state = new MockUseState(hooks);
 
 const {
-  useIsCollapsed,
   findCoursesNavClicked,
-  findCoursesNavDropdownClicked,
-  useLearnerDashboardHeaderData,
   useLearnerDashboardHeaderMenu,
 } = hooks;
 
@@ -29,17 +25,6 @@ describe('LearnerDashboardHeader hooks', () => {
     state.testGetter(state.keys.isOpen);
   });
 
-  describe('useIsCollapsed', () => {
-    test('large screen is not collapsed', () => {
-      useWindowSize.mockReturnValueOnce({ width: breakpoints.large.minWidth + 1 });
-      expect(useIsCollapsed()).toEqual(false);
-    });
-    test('small screen is collapsed', () => {
-      useWindowSize.mockReturnValueOnce({ width: breakpoints.large.minWidth - 1 });
-      expect(useIsCollapsed()).toEqual(true);
-    });
-  });
-
   describe('findCoursesNavClicked', () => {
     test('calls tracking with nav link name', () => {
       findCoursesNavClicked(url);
@@ -59,23 +44,4 @@ describe('LearnerDashboardHeader hooks', () => {
       expect(learnerHomeHeaderMenu.mainMenu.length).toBe(2);
     });
   });
-
-  describe('findCoursesNavDropdownClicked', () => {
-    test('calls tracking with dropdown link name', () => {
-      findCoursesNavDropdownClicked(url);
-      expect(track.findCourses.findCoursesClicked).toHaveBeenCalledWith(url, {
-        linkName: linkNames.learnerHomeNavDropdownExplore,
-      });
-    });
-  });
-
-  describe('useLearnerDashboardHeaderData', () => {
-    test('default state', () => {
-      state.mock();
-      const out = useLearnerDashboardHeaderData();
-      state.expectInitializedWith(state.keys.isOpen, false);
-      out.toggleIsOpen();
-      expect(state.values.isOpen).toEqual(true);
-    });
-  });
 });

src/data/redux/requests/reducer.test.js

@@ -47,5 +47,16 @@ describe('requests reducer', () => {
         });
       });
     });
+    describe('clearRequest', () => {
+      it('cleanup status and error', () => {
+        expect(reducer(
+          testingState,
+          actions.clearRequest({ requestKey: testKey, error: testValue }),
+        )).toEqual({
+          ...testingState,
+          [testKey]: {},
+        });
+      });
+    });
   });
 });