fix: uncomment pagination as API is ready

Greenstand · Aug 15, 2023 · 27db765 · 27db765
1 parent 281178f
commit 27db765
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 26 deletions.
diff --git a/src/pages/MyTransfers/MyTransfers.test.js b/src/pages/MyTransfers/MyTransfers.test.js
@@ -106,8 +106,7 @@ describe('My Transfers page', function () {
  </TestWrapper>
  );
 
- // TODO: uncomment when the API is ready: github.com/Greenstand/treetracker-wallet-api/issues/385
- // expect(await screen.findByTestId('table-pagination')).toBeInTheDocument();
+ expect(await screen.findByTestId('table-pagination')).toBeInTheDocument();
  expect(await screen.findByTestId('transfers-table')).toBeInTheDocument();
  expect(await screen.findByTestId('date-range-filter')).toBeInTheDocument();
  expect(

diff --git a/src/pages/MyTransfers/TransfersTable.js b/src/pages/MyTransfers/TransfersTable.js
@@ -1,8 +1,3 @@
-{
- /* TODO: Uncomment this to enable pagination when the API is ready:
- https://github.com/Greenstand/treetracker-wallet-api/issues/385 */
-}
-/* eslint-disable no-unused-vars */
 import {
  Grid,
  Paper,
@@ -11,7 +6,7 @@ import {
  TableCell,
  TableContainer,
  TableHead,
- // TablePagination,
+ TablePagination,
  TableRow,
  Typography,
 } from '@mui/material';
@@ -229,18 +224,16 @@ const TransfersTable = ({ tableTitle, tableRows, totalRowCount }) => {
  </Table>
  </TableContainer>
 
- {/* TODO: Uncomment this to enable pagination when the API is ready:
- https://github.com/Greenstand/treetracker-wallet-api/issues/385 */}
- {/* <TablePagination
+ <TablePagination
  rowsPerPageOptions={[10, 50]}
  component={'div'}
  count={totalRowCount}
  rowsPerPage={rowsPerPage}
  onRowsPerPageChange={handleRowsPerPageChange}
  page={page}
  onPageChange={(e, newPage) => handlePageChange(e, newPage)}
- data-testid='table-pagination'
- /> */}
+ data-testid="table-pagination"
+ />
  </Grid>
  );
 };

diff --git a/src/pages/MyTransfers/TransfersTable.test.js b/src/pages/MyTransfers/TransfersTable.test.js
@@ -62,8 +62,7 @@ describe('Transfers Table', () => {
  </TestWrapper>
  );
 
- // TODO: uncomment when the API is ready: github.com/Greenstand/treetracker-wallet-api/issues/385
- // expect(await screen.findByTestId('table-pagination')).toBeInTheDocument();
+ expect(await screen.findByTestId('table-pagination')).toBeInTheDocument();
  https: expect(
  await screen.findByTestId('transfers-table')
  ).toBeInTheDocument();
@@ -78,16 +77,15 @@ describe('Transfers Table', () => {
  expect(screen.getAllByRole('cell')).toHaveLength(8 * 3);
  });
 
- // TODO: uncomment when the API is ready: github.com/Greenstand/treetracker-wallet-api/issues/385
- // expect(await screen.findByTestId('table-pagination')).toBeInTheDocument();
- // expect(
- // screen.getByRole('button', { name: 'Rows per page: 10' })
- // ).toBeInTheDocument();
- // expect(
- // screen.getByRole('button', { name: 'Go to previous page' })
- // ).toBeInTheDocument();
- // expect(
- // screen.getByRole('button', { name: 'Go to next page' })
- // ).toBeInTheDocument();
+ expect(await screen.findByTestId('table-pagination')).toBeInTheDocument();
+ expect(
+ screen.getByRole('button', { name: 'Rows per page: 10' })
+ ).toBeInTheDocument();
+ expect(
+ screen.getByRole('button', { name: 'Go to previous page' })
+ ).toBeInTheDocument();
+ expect(
+ screen.getByRole('button', { name: 'Go to next page' })
+ ).toBeInTheDocument();
  });
 });