fixes for pagination logic

Author: kantord

src/components/__tests__/Dashboard.test.tsx

@@ -324,7 +324,7 @@ describe("Dashboard", () => {
     await waitFor(() => {
       expect(
         within(screen.getByTestId("alerts-table")).getAllByRole("row"),
-      ).toHaveLength(15);
+      ).toHaveLength(16);
     });
   });
 
@@ -338,7 +338,7 @@ describe("Dashboard", () => {
 
       expect(
         within(screen.getByTestId("alerts-table")).getAllByRole("row").length,
-      ).toBeLessThan(15);
+      ).toBeLessThan(16);
     });
 
     // on the last page, we cannot go further
@@ -357,7 +357,7 @@ describe("Dashboard", () => {
 
       expect(
         within(screen.getByTestId("alerts-table")).getAllByRole("row").length,
-      ).toEqual(15);
+      ).toEqual(16);
     });
   });
 });

src/hooks/useClientSidePagination.ts

@@ -4,12 +4,12 @@ export function useClientSidePagination<T>(
   pageSize: number,
 ) {
   const pageStart = page * pageSize;
-  const pageEnd = page * pageSize + pageSize - 1;
+  const pageEnd = page * pageSize + pageSize;
 
   const dataView = data.slice(pageStart, pageEnd);
 
   const hasPreviousPage = page > 0;
-  const hasNextPage = pageEnd + 1 < data.length;
+  const hasNextPage = pageEnd < data.length;
 
   return { pageStart, pageEnd, dataView, hasPreviousPage, hasNextPage };
 }