chore: Migrate deprecated Table to DataTable for CompletedLearnersTable

src/components/Admin/CompletedLearnersTable.jsx

@@ -0,0 +1,64 @@
+import React from 'react';
+import { useIntl } from '@edx/frontend-platform/i18n';
+import { DataTable } from '@openedx/paragon';
+import { useGenericTableData } from './data/hooks';
+import EnterpriseDataApiService from '../../data/services/EnterpriseDataApiService';
+import Email from './Email';
+
+const CompletedLearnersTable = (enterpriseId) => {
+  const intl = useIntl();
+  const {
+    isLoading,
+    tableData,
+    fetchTableData,
+  } = useGenericTableData(
+    enterpriseId,
+    'completed-learners',
+    EnterpriseDataApiService.fetchCompletedLearners,
+    {
+      userEmail: { key: 'user_email' },
+      completedCourses: { key: 'completed_courses' },
+    },
+  );
+
+  return (
+    <DataTable
+      isSortable
+      manualSortBy
+      isPaginated
+      manualPagination
+      isLoading={isLoading}
+      columns={[
+        {
+          Header: intl.formatMessage({
+            id: 'admin.portal.lpr.completed.learners.table.user_email.column.heading',
+            defaultMessage: 'Email',
+            description: 'Column heading for the user email column in the completed learners table',
+          }),
+          accessor: 'userEmail',
+          Cell: Email,
+        },
+        {
+          Header: intl.formatMessage({
+            id: 'admin.portal.lpr.completed.learners.table.completed_courses.column.heading',
+            defaultMessage: 'Total Course Completed Count',
+            description: 'Column heading for the completed courses column in the completed learners table',
+          }),
+          accessor: 'completedCourses',
+        },
+      ]}
+      initialState={{
+        pageIndex: 20,
+        pageSize: 0,
+        sortBy: [{ id: 'completedCourses', desc: true }],
+        selectedRowsOrdered: [],
+      }}
+      fetchData={fetchTableData}
+      data={tableData.results}
+      itemCount={tableData.itemCount}
+      pageCount={tableData.pageCount}
+    />
+  );
+};
+
+export default CompletedLearnersTable;

src/components/CompletedLearnersTable/CompletedLearnersTable.test.jsx → src/components/Admin/CompletedLearnersTable.test.jsx

@@ -6,26 +6,20 @@ import configureMockStore from 'redux-mock-store';
 import thunk from 'redux-thunk';
 import { Provider } from 'react-redux';
 
-import CompletedLearnersTable from '.';
+import CompletedLearnersTable from './CompletedLearnersTable';
+import { useGenericTableData } from './data/hooks';
 
 const mockStore = configureMockStore([thunk]);
 const enterpriseId = 'test-enterprise';
+
+jest.mock('./data/hooks/useGenericTableData', () => (
+  jest.fn().mockReturnValue({})
+));
+
 const store = mockStore({
   portalConfiguration: {
     enterpriseId,
   },
-  table: {
-    'completed-learners': {
-      data: {
-        results: [],
-        current_page: 1,
-        num_pages: 1,
-      },
-      ordering: null,
-      loading: false,
-      error: null,
-    },
-  },
 });
 
 const CompletedLearnersWrapper = props => (
@@ -42,6 +36,15 @@ const CompletedLearnersWrapper = props => (
 
 describe('CompletedLearnersTable', () => {
   it('renders empty state correctly', () => {
+    useGenericTableData.mockReturnValue({
+      isLoading: false,
+      tableData: {
+        results: [],
+        itemCount: 0,
+        pageCount: 0,
+      },
+      fetchTableData: jest.fn(),
+    });
     const tree = renderer
       .create((
         <CompletedLearnersWrapper />

src/components/Admin/Email.jsx

@@ -0,0 +1,16 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+
+const Email = ({ row }) => (
+  <span data-hj-suppress>{row.original.userEmail}</span>
+);
+
+Email.propTypes = {
+  row: PropTypes.shape({
+    original: PropTypes.shape({
+      userEmail: PropTypes.string.isRequired,
+    }).isRequired,
+  }).isRequired,
+};
+
+export default Email;

src/components/Admin/Email.test.jsx

@@ -0,0 +1,42 @@
+import React from 'react';
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
+import Email from './Email';
+
+describe('Email Component', () => {
+  it('should display the user email and suppress highlighting', () => {
+    const mockRow = {
+      original: {
+        userEmail: '[email protected]',
+      },
+    };
+
+    render(
+      <Email row={mockRow} />,
+    );
+
+    // Assert that the email is rendered correctly
+    const emailElement = screen.getByText('[email protected]');
+    expect(emailElement).toBeInTheDocument();
+
+    // Assert that data-hj-suppress is present to suppress highlighting
+    expect(emailElement).toHaveAttribute('data-hj-suppress');
+  });
+
+  it('should throw a prop-type warning if the email is not provided', () => {
+    const consoleSpy = jest.spyOn(console, 'error').mockImplementation(() => {});
+
+    const invalidRow = {
+      original: {},
+    };
+
+    // Render with invalid props
+    render(<Email row={invalidRow} />);
+
+    // Assert that a prop-types error has been logged
+    expect(consoleSpy).toHaveBeenCalled();
+
+    // Clean up the spy
+    consoleSpy.mockRestore();
+  });
+});