chore: rebase

openedx · Sep 13, 2024 · 6468b1c · 6468b1c
2 parents 0cde4e7 + 9d06fe6
commit 6468b1c
Show file tree

Hide file tree

Showing 8 changed files with 187 additions and 82 deletions.
diff --git a/src/Configuration/Customers/CustomerDetailView/CustomerIntegrations.jsx b/src/Configuration/Customers/CustomerDetailView/CustomerIntegrations.jsx
@@ -15,6 +15,10 @@ const CustomerIntegrations = ({
     integrationCount++;
   }
 
+  if (!integrationCount) {
+    return null;
+  }
+
   return (
     <div>
       {(integrationCount > 0) && (

diff --git a/src/Configuration/Customers/CustomerDetailView/CustomerPlanContainer.jsx b/src/Configuration/Customers/CustomerDetailView/CustomerPlanContainer.jsx
@@ -1,23 +1,25 @@
-import { useState } from 'react';
-import { useParams } from 'react-router-dom';
+import { useState, useEffect } from 'react';
 import PropTypes from 'prop-types';
 import { Form, Skeleton } from '@openedx/paragon';
-import useAllAssociatedPlans from '../data/hooks/useAllAssociatedPlans';
 import LearnerCreditPlanCard from './LearnerCreditPlanCard';
 import SubscriptionPlanCard from './SubscriptionPlanCard';
 
-const CustomerPlanContainer = ({ slug }) => {
-  const { id } = useParams();
-  const {
-    activeSubsidies,
-    activeSubscriptions,
-    countOfActivePlans,
-    countOfAllPlans,
-    inactiveSubsidies,
-    inactiveSubscriptions,
-    isLoading,
-  } = useAllAssociatedPlans(id);
+const CustomerPlanContainer = ({
+  slug,
+  activeSubsidies,
+  activeSubscriptions,
+  countOfActivePlans,
+  countOfAllPlans,
+  inactiveSubsidies,
+  inactiveSubscriptions,
+  isLoading,
+}) => {
   const [showInactive, setShowInactive] = useState(false);
+  useEffect(() => {
+    if (!countOfActivePlans && countOfAllPlans) {
+      setShowInactive(true);
+    }
+  }, []);
   const renderActiveSubsidiesCard = activeSubsidies.map(subsidy => (
     <LearnerCreditPlanCard key={subsidy.uuid} isActive slug={slug} subsidy={subsidy} />
   ));
@@ -30,23 +32,25 @@ const CustomerPlanContainer = ({ slug }) => {
   const renderInActiveSubscriptions = inactiveSubscriptions.map(subscription => (
     <SubscriptionPlanCard key={subscription.uuid} isActive={false} slug={slug} subscription={subscription} />
   ));
+
   return (
     <div>
       {!isLoading ? (
         <div>
           <div className="d-flex justify-content-between">
             <h2>Associated subsidy plans ({showInactive ? countOfAllPlans : countOfActivePlans})</h2>
-            <Form.Switch
-              className="ml-2.5 mt-2.5"
-              checked={showInactive}
-              disabled={countOfAllPlans === countOfActivePlans}
-              onChange={() => {
-                setShowInactive(prevState => !prevState);
-              }}
-              data-testid="show-removed-toggle"
-            >
-              Show inactive
-            </Form.Switch>
+            {(countOfAllPlans > countOfActivePlans && countOfActivePlans) ? (
+              <Form.Switch
+                className="ml-2.5 mt-2.5"
+                checked={showInactive}
+                onChange={() => {
+                  setShowInactive(prevState => !prevState);
+                }}
+                data-testid="show-removed-toggle"
+              >
+                Show inactive
+              </Form.Switch>
+            ) : null}
           </div>
           <hr />
           {renderActiveSubsidiesCard}
@@ -65,6 +69,33 @@ const CustomerPlanContainer = ({ slug }) => {
 
 CustomerPlanContainer.propTypes = {
   slug: PropTypes.string.isRequired,
+  activeSubsidies: PropTypes.arrayOf(PropTypes.shape({
+    uuid: PropTypes.string.isRequired,
+    activeDatetime: PropTypes.string.isRequired,
+    expirationDatetime: PropTypes.string.isRequired,
+    created: PropTypes.string.isRequired,
+  })).isRequired,
+  activeSubscriptions: PropTypes.arrayOf(PropTypes.shape({
+    uuid: PropTypes.string.isRequired,
+    startDate: PropTypes.string.isRequired,
+    expirationDate: PropTypes.string.isRequired,
+    created: PropTypes.string.isRequired,
+  })).isRequired,
+  countOfActivePlans: PropTypes.number.isRequired,
+  countOfAllPlans: PropTypes.number.isRequired,
+  inactiveSubsidies: PropTypes.arrayOf(PropTypes.shape({
+    uuid: PropTypes.string.isRequired,
+    activeDatetime: PropTypes.string.isRequired,
+    expirationDatetime: PropTypes.string.isRequired,
+    created: PropTypes.string.isRequired,
+  })).isRequired,
+  inactiveSubscriptions: PropTypes.arrayOf(PropTypes.shape({
+    uuid: PropTypes.string.isRequired,
+    startDate: PropTypes.string.isRequired,
+    expirationDate: PropTypes.string.isRequired,
+    created: PropTypes.string.isRequired,
+  })).isRequired,
+  isLoading: PropTypes.bool.isRequired,
 };
 
 export default CustomerPlanContainer;
diff --git a/src/Configuration/Customers/CustomerDetailView/CustomerViewContainer.jsx b/src/Configuration/Customers/CustomerDetailView/CustomerViewContainer.jsx
@@ -13,12 +13,14 @@ import { getEnterpriseCustomer } from '../data/utils';
 import CustomerIntegrations from './CustomerIntegrations';
 import EnterpriseCustomerUsersTable from './EnterpriseCustomerUsersTable';
 import CustomerPlanContainer from './CustomerPlanContainer';
+import useAllAssociatedPlans from '../data/hooks/useAllAssociatedPlans';
 
 const CustomerViewContainer = () => {
   const { id } = useParams();
   const [enterpriseCustomer, setEnterpriseCustomer] = useState({});
   const [isLoading, setIsLoading] = useState(true);
   const intl = useIntl();
+  const associatedPlans = useAllAssociatedPlans(id);
 
   const fetchData = useCallback(
     async () => {
@@ -38,6 +40,23 @@ const CustomerViewContainer = () => {
     fetchData();
   }, []);
 
+  const renderPlanContainer = () => {
+    if (!isLoading && !associatedPlans.isLoading && associatedPlans.countOfAllPlans) {
+      return (
+        <Stack gap={2}>
+          <CustomerPlanContainer slug={enterpriseCustomer.slug} {...associatedPlans} />
+        </Stack>
+      );
+    }
+    if (!associatedPlans.isLoading && !associatedPlans.countOfAllPlans) {
+      return false;
+    }
+    if (associatedPlans.isLoading) {
+      return <Skeleton height={230} />;
+    }
+    return null;
+  };
+
   return (
     <div>
       {!isLoading ? (
@@ -64,11 +83,9 @@ const CustomerViewContainer = () => {
         </Stack>
       </Container>
       <Container className="mt-4">
-        <Stack gap={2}>
-          {!isLoading ? <CustomerPlanContainer slug={enterpriseCustomer.slug} /> : <Skeleton height={230} />}
-        </Stack>
+        {renderPlanContainer()}
       </Container>
-      <Container className="mt-4">
+      <Container className="mt-4 mb-4">
         <Stack gap={2}>
           <CustomerIntegrations
             slug={enterpriseCustomer.slug}

diff --git a/src/Configuration/Customers/CustomerDetailView/EnterpriseCustomerUsersTable.jsx b/src/Configuration/Customers/CustomerDetailView/EnterpriseCustomerUsersTable.jsx
@@ -9,52 +9,60 @@ const EnterpriseCustomerUsersTable = () => {
     isLoading,
     enterpriseUsersTableData,
     fetchEnterpriseUsersData,
+    showTable,
   } = useCustomerUsersTableData(id);
+
   return (
     <div>
-      <h2>Associated users ({enterpriseUsersTableData.itemCount})</h2>
-      <hr />
-      <DataTable
-        isLoading={isLoading}
-        isExpandable
-        isPaginated
-        manualPagination
-        isFilterable
-        manualFilters
-        initialState={{
-          pageSize: 8,
-          pageIndex: 0,
-          sortBy: [],
-          filters: [],
-        }}
-        defaultColumnValues={{ Filter: TextFilter }}
-        fetchData={fetchEnterpriseUsersData}
-        data={enterpriseUsersTableData.results}
-        itemCount={enterpriseUsersTableData.itemCount}
-        pageCount={enterpriseUsersTableData.pageCount}
-        columns={[
-          {
-            id: 'details',
-            Header: 'User details',
-            accessor: 'details',
-            Cell: EnterpriseCustomerUserDetail,
-          },
-          {
-            id: 'administrator',
-            Header: 'Administrator',
-            accessor: 'administrator',
-            disableFilters: true,
-            Cell: AdministratorCell,
-          },
-          {
-            id: 'learner',
-            Header: 'Learner',
-            accessor: 'learner',
-            disableFilters: true,
-            Cell: LearnerCell,
-          },
-        ]}
-      />
+      {showTable ? (
+        <div>
+          <h2>Associated users {enterpriseUsersTableData.itemCount > 0
+            && <span>({enterpriseUsersTableData.itemCount})</span>}
+          </h2>
+          <hr />
+          <DataTable
+            isLoading={isLoading}
+            isExpandable
+            isPaginated
+            manualPagination
+            isFilterable
+            manualFilters
+            initialState={{
+              pageSize: 8,
+              pageIndex: 0,
+              sortBy: [],
+              filters: [],
+            }}
+            defaultColumnValues={{ Filter: TextFilter }}
+            fetchData={fetchEnterpriseUsersData}
+            data={enterpriseUsersTableData.results}
+            itemCount={enterpriseUsersTableData.itemCount}
+            pageCount={enterpriseUsersTableData.pageCount}
+            columns={[
+              {
+                id: 'details',
+                Header: 'User details',
+                accessor: 'details',
+                Cell: EnterpriseCustomerUserDetail,
+              },
+              {
+                id: 'administrator',
+                Header: 'Administrator',
+                accessor: 'administrator',
+                disableFilters: true,
+                Cell: AdministratorCell,
+              },
+              {
+                id: 'learner',
+                Header: 'Learner',
+                accessor: 'learner',
+                disableFilters: true,
+                Cell: LearnerCell,
+              },
+            ]}
+          />
+        </div>
+      ) : null}
     </div>
   );
 };

diff --git a/src/Configuration/Customers/CustomerDetailView/tests/CustomerPlanContainer.test.jsx b/src/Configuration/Customers/CustomerDetailView/tests/CustomerPlanContainer.test.jsx
@@ -5,7 +5,6 @@ import userEvent from '@testing-library/user-event';
 import { getConfig } from '@edx/frontend-platform';
 import { IntlProvider } from '@edx/frontend-platform/i18n';
 
-import useAllAssociatedPlans from '../../data/hooks/useAllAssociatedPlans';
 import { formatDate } from '../../data/utils';
 import CustomerPlanContainer from '../CustomerPlanContainer';
 
@@ -19,11 +18,6 @@ jest.mock('@edx/frontend-platform', () => ({
   getConfig: jest.fn(),
 }));
 
-jest.mock('react-router-dom', () => ({
-  ...jest.requireActual('react-router-dom'),
-  useParams: () => ({ id: 'test-uuid' }),
-}));
-
 const CUSTOMER_SLUG = 'test-slug';
 
 describe('CustomerPlanContainer', () => {
@@ -44,7 +38,7 @@ describe('CustomerPlanContainer', () => {
       SUBSIDY_BASE_URL: 'http:www.enterprise-subsidy.com',
       LICENSE_MANAGER_URL: 'http:www.license-manager.com',
     }));
-    useAllAssociatedPlans.mockReturnValue({
+    const mockProps = {
       isLoading: false,
       activeSubsidies: [{
         activeDatetime: '2024-08-23T20:02:57.651943Z',
@@ -69,10 +63,10 @@ describe('CustomerPlanContainer', () => {
         uuid: 'test-uuid',
         isActive: false,
       }],
-    });
+    };
     render(
       <IntlProvider locale="en">
-        <CustomerPlanContainer slug={CUSTOMER_SLUG} />
+        <CustomerPlanContainer slug={CUSTOMER_SLUG} {...mockProps} />
       </IntlProvider>,
     );
     const djangoLinks = screen.getAllByRole('link', { name: 'Open in Django' });

diff --git a/src/Configuration/Customers/CustomerDetailView/tests/CustomerViewIntegrations.test.jsx b/src/Configuration/Customers/CustomerDetailView/tests/CustomerViewIntegrations.test.jsx
@@ -63,4 +63,20 @@ describe('CustomerViewIntegrations', () => {
       expect(screen.getByText('API')).toBeInTheDocument();
     });
   });
+  it('does not render cards', async () => {
+    formatDate.mockReturnValue('September 15, 2024');
+    render(
+      <IntlProvider locale="en">
+        <CustomerIntegrations
+          slug="marcel-the-shell"
+          activeIntegrations={mockIntegratedChannelData}
+          activeSSO={[]}
+          apiCredentialsEnabled={false}
+        />
+      </IntlProvider>,
+    );
+    await waitFor(() => {
+      expect(screen.queryByText('Associated integrations (0)')).not.toBeInTheDocument();
+    });
+  });
 });
diff --git a/src/Configuration/Customers/CustomerDetailView/tests/EnterpriseCustomerUsersTable.test.jsx b/src/Configuration/Customers/CustomerDetailView/tests/EnterpriseCustomerUsersTable.test.jsx
@@ -33,6 +33,7 @@ const mockData = {
     ],
   },
   fetchEnterpriseUsersData: mockFetchEnterpriseUsersData,
+  showTable: true,
 };
 
 jest.mock('../../data/hooks/useCustomerUsersTableData');
@@ -80,4 +81,24 @@ describe('EnterpriseCustomerUsersTable', () => {
       });
     });
   });
+
+  it('does not render user table section', () => {
+    const emptyResults = {
+      isLoading: false,
+      enterpriseUsersTableData: {
+        itemCount: 0,
+        pageCount: 1,
+        results: [],
+      },
+      fetchEnterpriseUsersData: mockFetchEnterpriseUsersData,
+    };
+    useCustomerUsersTableData.mockReturnValue(emptyResults);
+    render(
+      <IntlProvider locale="en">
+        <EnterpriseCustomerUsersTable />
+      </IntlProvider>,
+    );
+    expect(screen.queryByText('Search user details')).not.toBeInTheDocument();
+    expect(screen.queryByText('Associated users (0)')).not.toBeInTheDocument();
+  });
 });