update usage to show isLastPage now works correctly

algolia · Jun 3, 2020 · 19873e4 · 19873e4
1 parent 68b1dfe
commit 19873e4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/src/components/Pagination/Pagination.js b/src/components/Pagination/Pagination.js
@@ -48,7 +48,7 @@ class Pagination extends Component {
     return this.pageLink({
       ariaLabel: 'Previous',
       additionalClassName: this.props.cssClasses.previousPageItem,
-      isDisabled: this.props.nbHits === 0 || isFirstPage,
+      isDisabled: isFirstPage,
       label: this.props.templates.previous,
       pageNumber: currentPage - 1,
       createURL,
@@ -59,7 +59,7 @@ class Pagination extends Component {
     return this.pageLink({
       ariaLabel: 'Next',
       additionalClassName: this.props.cssClasses.nextPageItem,
-      isDisabled: this.props.nbHits === 0 || isLastPage,
+      isDisabled: isLastPage,
       label: this.props.templates.next,
       pageNumber: currentPage + 1,
       createURL,
@@ -70,7 +70,7 @@ class Pagination extends Component {
     return this.pageLink({
       ariaLabel: 'First',
       additionalClassName: this.props.cssClasses.firstPageItem,
-      isDisabled: this.props.nbHits === 0 || isFirstPage,
+      isDisabled: isFirstPage,
       label: this.props.templates.first,
       pageNumber: 0,
       createURL,
@@ -81,7 +81,7 @@ class Pagination extends Component {
     return this.pageLink({
       ariaLabel: 'Last',
       additionalClassName: this.props.cssClasses.lastPageItem,
-      isDisabled: this.props.nbHits === 0 || isLastPage,
+      isDisabled: isLastPage,
       label: this.props.templates.last,
       pageNumber: nbPages - 1,
       createURL,

diff --git a/src/components/Pagination/__tests__/Pagination-test.js b/src/components/Pagination/__tests__/Pagination-test.js
@@ -99,6 +99,11 @@ describe('Pagination', () => {
   });
 
   it('should have all buttons disabled if there are no results', () => {
+    const localPager = new Paginator({
+      currentPage: 0,
+      total: 0,
+      padding: 3,
+    });
     const wrapper = mount(
       <Pagination
         {...defaultProps}
@@ -109,7 +114,9 @@ describe('Pagination', () => {
         currentPage={0}
         nbHits={0}
         nbPages={0}
-        pages={[0]}
+        pages={localPager.pages()}
+        isFirstPage={localPager.isFirstPage()}
+        isLastPage={localPager.isLastPage()}
       />
     );