From 25d3f831a2f77bf981e48f9fc23248a355d611a6 Mon Sep 17 00:00:00 2001
From: Peter Kulko <peter.kulko@raccoongang.com>
Date: Thu, 14 Nov 2024 19:57:03 +0200
Subject: [PATCH] fix: removed program tab in the header

---
 .../CollapsedHeader/CollapseMenuBody.jsx      | 12 ++--
 .../CollapsedHeader/CollapseMenuBody.test.jsx | 12 ++++
 .../CollapseMenuBody.test.jsx.snap            | 59 +++++++++++++++++++
 .../__snapshots__/index.test.jsx.snap         | 41 +++++++++++++
 .../ExpandedHeader/index.jsx                  | 22 ++++---
 .../ExpandedHeader/index.test.jsx             | 12 ++++
 src/data/services/lms/api.js                  |  3 +
 src/data/services/lms/urls.js                 |  2 +
 src/hooks/api.js                              | 20 +++++++
 9 files changed, 170 insertions(+), 13 deletions(-)

diff --git a/src/containers/LearnerDashboardHeader/CollapsedHeader/CollapseMenuBody.jsx b/src/containers/LearnerDashboardHeader/CollapsedHeader/CollapseMenuBody.jsx
index 82de5aa..4769328 100644
--- a/src/containers/LearnerDashboardHeader/CollapsedHeader/CollapseMenuBody.jsx
+++ b/src/containers/LearnerDashboardHeader/CollapsedHeader/CollapseMenuBody.jsx
@@ -7,7 +7,7 @@ import { AppContext } from '@edx/frontend-platform/react';
 import { Button, Badge } from '@openedx/paragon';
 
 import urls from 'data/services/lms/urls';
-import { reduxHooks } from 'hooks';
+import { reduxHooks, apiHooks } from 'hooks';
 
 import { findCoursesNavDropdownClicked } from '../hooks';
 import messages from '../messages';
@@ -16,6 +16,8 @@ export const CollapseMenuBody = ({ isOpen }) => {
   const { formatMessage } = useIntl();
   const { authenticatedUser } = React.useContext(AppContext);
 
+  const { enabled: programsEnabled } = apiHooks.useProgramsConfig();
+
   const dashboard = reduxHooks.useEnterpriseDashboardData();
   const { courseSearchUrl } = reduxHooks.usePlatformSettingsData();
 
@@ -32,9 +34,11 @@ export const CollapseMenuBody = ({ isOpen }) => {
       <Button as="a" href="/" variant="inverse-primary">
         {formatMessage(messages.course)}
       </Button>
-      <Button as="a" href={urls.programsUrl()} variant="inverse-primary">
-        {formatMessage(messages.program)}
-      </Button>
+      {programsEnabled && (
+        <Button as="a" href={urls.programsUrl()} variant="inverse-primary">
+          {formatMessage(messages.program)}
+        </Button>
+      )}
       <Button
         as="a"
         href={urls.baseAppUrl(courseSearchUrl)}
diff --git a/src/containers/LearnerDashboardHeader/CollapsedHeader/CollapseMenuBody.test.jsx b/src/containers/LearnerDashboardHeader/CollapsedHeader/CollapseMenuBody.test.jsx
index 2af8ae7..ac9f450 100644
--- a/src/containers/LearnerDashboardHeader/CollapsedHeader/CollapseMenuBody.test.jsx
+++ b/src/containers/LearnerDashboardHeader/CollapsedHeader/CollapseMenuBody.test.jsx
@@ -1,6 +1,7 @@
 import { shallow } from '@edx/react-unit-test-utils';
 import { AppContext } from '@edx/frontend-platform/react';
 
+import { apiHooks } from 'hooks';
 import CollapseMenuBody from './CollapseMenuBody';
 
 jest.mock('@edx/frontend-platform/react', () => ({
@@ -20,6 +21,11 @@ jest.mock('hooks', () => ({
       courseSearchUrl: '/courseSearchUrl',
     }),
   },
+  apiHooks: {
+    useProgramsConfig: () => ({
+      enabled: true,
+    }),
+  },
 }));
 
 jest.mock('../hooks', () => ({
@@ -45,4 +51,10 @@ describe('CollapseMenuBody', () => {
     expect(wrapper.snapshot).toMatchSnapshot();
     AppContext.authenticatedUser = authenticatedUser;
   });
+
+  test('render with disabled programs', () => {
+    apiHooks.useProgramsConfig = () => ({ enabled: false });
+    const wrapper = shallow(<CollapseMenuBody isOpen />);
+    expect(wrapper.snapshot).toMatchSnapshot();
+  });
 });
diff --git a/src/containers/LearnerDashboardHeader/CollapsedHeader/__snapshots__/CollapseMenuBody.test.jsx.snap b/src/containers/LearnerDashboardHeader/CollapsedHeader/__snapshots__/CollapseMenuBody.test.jsx.snap
index c8cc4de..7347a05 100644
--- a/src/containers/LearnerDashboardHeader/CollapsedHeader/__snapshots__/CollapseMenuBody.test.jsx.snap
+++ b/src/containers/LearnerDashboardHeader/CollapsedHeader/__snapshots__/CollapseMenuBody.test.jsx.snap
@@ -103,3 +103,62 @@ exports[`CollapseMenuBody render unauthenticated 1`] = `
   </Button>
 </div>
 `;
+
+exports[`CollapseMenuBody render with disabled programs 1`] = `
+<div
+  className="d-flex flex-column shadow-sm nav-small-menu"
+>
+  <Button
+    as="a"
+    href="/"
+    variant="inverse-primary"
+  >
+    Courses
+  </Button>
+  <Button
+    as="a"
+    href="http://localhost:18000/courseSearchUrl"
+    onClick={[MockFunction findCoursesNavDropdownClicked("http://localhost:18000/courseSearchUrl")]}
+    variant="inverse-primary"
+  >
+    Discover New
+  </Button>
+  <Button
+    as="a"
+    href="http://localhost:18000/support"
+    variant="inverse-primary"
+  >
+    Help
+  </Button>
+  <Fragment>
+    <Button
+      as="a"
+      href="url"
+      variant="inverse-primary"
+    >
+      Dashboard
+    </Button>
+    <Button
+      as="a"
+      href="http://localhost:18000/u/username"
+      variant="inverse-primary"
+    >
+      Profile
+    </Button>
+    <Button
+      as="a"
+      href="http://localhost:18000/account/settings"
+      variant="inverse-primary"
+    >
+      Account
+    </Button>
+    <Button
+      as="a"
+      href="http://localhost:18000/logout"
+      variant="inverse-primary"
+    >
+      Sign Out
+    </Button>
+  </Fragment>
+</div>
+`;
diff --git a/src/containers/LearnerDashboardHeader/ExpandedHeader/__snapshots__/index.test.jsx.snap b/src/containers/LearnerDashboardHeader/ExpandedHeader/__snapshots__/index.test.jsx.snap
index 44e08f1..5215cf5 100644
--- a/src/containers/LearnerDashboardHeader/ExpandedHeader/__snapshots__/index.test.jsx.snap
+++ b/src/containers/LearnerDashboardHeader/ExpandedHeader/__snapshots__/index.test.jsx.snap
@@ -50,3 +50,44 @@ exports[`ExpandedHeader render 1`] = `
 `;
 
 exports[`ExpandedHeader render empty if collapsed 1`] = `null`;
+
+exports[`ExpandedHeader render with disabled programs 1`] = `
+<header
+  className="d-flex shadow-sm align-items-center learner-variant-header pl-4"
+>
+  <div
+    className="flex-grow-1 d-flex align-items-center"
+  >
+    <BrandLogo />
+    <Button
+      as="a"
+      className="p-4 course-link"
+      href="/"
+      variant="inverse-primary"
+    >
+      Courses
+    </Button>
+    <Button
+      as="a"
+      className="p-4"
+      href="http://localhost:18000/courseSearchUrl"
+      onClick={[MockFunction findCoursesNavClicked("http://localhost:18000/courseSearchUrl")]}
+      variant="inverse-primary"
+    >
+      Discover New
+    </Button>
+    <span
+      className="flex-grow-1"
+    />
+    <Button
+      as="a"
+      className="p-4"
+      href="http://localhost:18000/support"
+      variant="inverse-primary"
+    >
+      Help
+    </Button>
+  </div>
+  <AuthenticatedUserDropdown />
+</header>
+`;
diff --git a/src/containers/LearnerDashboardHeader/ExpandedHeader/index.jsx b/src/containers/LearnerDashboardHeader/ExpandedHeader/index.jsx
index f257cbc..035a79e 100644
--- a/src/containers/LearnerDashboardHeader/ExpandedHeader/index.jsx
+++ b/src/containers/LearnerDashboardHeader/ExpandedHeader/index.jsx
@@ -5,7 +5,7 @@ import { useIntl } from '@edx/frontend-platform/i18n';
 import { Button } from '@openedx/paragon';
 
 import urls from 'data/services/lms/urls';
-import { reduxHooks } from 'hooks';
+import { reduxHooks, apiHooks } from 'hooks';
 
 import AuthenticatedUserDropdown from './AuthenticatedUserDropdown';
 import { useIsCollapsed, findCoursesNavClicked } from '../hooks';
@@ -17,6 +17,8 @@ export const ExpandedHeader = () => {
   const { courseSearchUrl } = reduxHooks.usePlatformSettingsData();
   const isCollapsed = useIsCollapsed();
 
+  const { enabled: programsEnabled } = apiHooks.useProgramsConfig();
+
   const exploreCoursesClick = findCoursesNavClicked(
     urls.baseAppUrl(courseSearchUrl),
   );
@@ -38,14 +40,16 @@ export const ExpandedHeader = () => {
         >
           {formatMessage(messages.course)}
         </Button>
-        <Button
-          as="a"
-          href={urls.programsUrl()}
-          variant="inverse-primary"
-          className="p-4"
-        >
-          {formatMessage(messages.program)}
-        </Button>
+        {programsEnabled && (
+          <Button
+            as="a"
+            href={urls.programsUrl()}
+            variant="inverse-primary"
+            className="p-4"
+          >
+            {formatMessage(messages.program)}
+          </Button>
+        )}
         <Button
           as="a"
           href={urls.baseAppUrl(courseSearchUrl)}
diff --git a/src/containers/LearnerDashboardHeader/ExpandedHeader/index.test.jsx b/src/containers/LearnerDashboardHeader/ExpandedHeader/index.test.jsx
index 3b96a1d..b4f73c8 100644
--- a/src/containers/LearnerDashboardHeader/ExpandedHeader/index.test.jsx
+++ b/src/containers/LearnerDashboardHeader/ExpandedHeader/index.test.jsx
@@ -1,5 +1,6 @@
 import { shallow } from '@edx/react-unit-test-utils';
 
+import { apiHooks } from 'hooks';
 import ExpandedHeader from '.';
 
 import { useIsCollapsed } from '../hooks';
@@ -15,6 +16,11 @@ jest.mock('hooks', () => ({
       courseSearchUrl: '/courseSearchUrl',
     }),
   },
+  apiHooks: {
+    useProgramsConfig: () => ({
+      enabled: true,
+    }),
+  },
 }));
 
 jest.mock('../hooks', () => ({
@@ -38,4 +44,10 @@ describe('ExpandedHeader', () => {
     expect(wrapper.snapshot).toMatchSnapshot();
     expect(wrapper.isEmptyRender()).toBe(true);
   });
+
+  test('render with disabled programs', () => {
+    apiHooks.useProgramsConfig = () => ({ enabled: false });
+    const wrapper = shallow(<ExpandedHeader />);
+    expect(wrapper.snapshot).toMatchSnapshot();
+  });
 });
diff --git a/src/data/services/lms/api.js b/src/data/services/lms/api.js
index 6f9acd5..21933c0 100644
--- a/src/data/services/lms/api.js
+++ b/src/data/services/lms/api.js
@@ -20,6 +20,8 @@ export const initializeList = ({ user } = {}) => get(
   stringifyUrl(urls.getInitApiUrl(), { [apiKeys.user]: user }),
 );
 
+export const getProgramsConfig = () => get(urls.programsConfigUrl());
+
 export const updateEntitlementEnrollment = ({ uuid, courseId }) => post(
   urls.entitlementEnrollment(uuid),
   { [apiKeys.courseRunId]: courseId },
@@ -75,6 +77,7 @@ export default {
   initializeList,
   unenrollFromCourse,
   updateEmailSettings,
+  getProgramsConfig,
   updateEntitlementEnrollment,
   deleteEntitlementEnrollment,
   logEvent,
diff --git a/src/data/services/lms/urls.js b/src/data/services/lms/urls.js
index 30c2bbf..0e6a452 100644
--- a/src/data/services/lms/urls.js
+++ b/src/data/services/lms/urls.js
@@ -22,6 +22,7 @@ export const baseAppUrl = (url) => updateUrl(getBaseUrl(), url);
 export const learningMfeUrl = (url) => updateUrl(getConfig().LEARNING_BASE_URL, url);
 
 // static view url
+const programsConfigUrl = () => baseAppUrl('/config/programs');
 const programsUrl = () => baseAppUrl('/dashboard/programs');
 
 export const creditPurchaseUrl = (courseId) => `${getEcommerceUrl()}/credit/checkout/${courseId}/`;
@@ -38,5 +39,6 @@ export default StrictDict({
   getInitApiUrl,
   learningMfeUrl,
   programsUrl,
+  programsConfigUrl,
   updateEmailSettings,
 });
diff --git a/src/hooks/api.js b/src/hooks/api.js
index d64a118..94726ae 100644
--- a/src/hooks/api.js
+++ b/src/hooks/api.js
@@ -1,6 +1,7 @@
 import React from 'react';
 
 import { AppContext } from '@edx/frontend-platform/react';
+import { logError } from '@edx/frontend-platform/logging';
 
 import { RequestKeys } from 'data/constants/requests';
 import { post } from 'data/services/lms/utils';
@@ -31,6 +32,25 @@ export const useInitializeApp = () => {
   });
 };
 
+export const useProgramsConfig = () => {
+  const [config, setConfig] = React.useState({});
+
+  const fetchProgramsConfig = React.useCallback(async () => {
+    try {
+      const { data } = await api.getProgramsConfig();
+      setConfig(data);
+    } catch (error) {
+      logError(`Error accessing programs configuration: ${error.message}`);
+    }
+  }, []);
+
+  React.useEffect(() => {
+    fetchProgramsConfig();
+  }, [fetchProgramsConfig]);
+
+  return config;
+};
+
 export const useNewEntitlementEnrollment = (cardId) => {
   const { uuid } = reduxHooks.useCardEntitlementData(cardId);
   const onSuccess = module.useInitializeApp();