Fix Paginate so it does not require currentPage (#4509)

Author: bobsilverberg

src/core/components/Paginate/index.js

@@ -20,7 +20,7 @@ export class PaginateBase extends React.Component {
   static propTypes = {
     LinkComponent: PropTypes.func,
     count: PropTypes.number.isRequired,
-    currentPage: PropTypes.number.isRequired,
+    currentPage: PropTypes.number,
     i18n: PropTypes.object.isRequired,
     pathname: PropTypes.string.isRequired,
     perPage: PropTypes.number,
@@ -31,6 +31,7 @@ export class PaginateBase extends React.Component {
   static defaultProps = {
     perPage: 25, // The default number of results per page returned by the API.
     showPages: 0,
+    currentPage: 1,
   }
 
   pageCount() {
@@ -74,9 +75,6 @@ export class PaginateBase extends React.Component {
     if (count === undefined) {
       throw new Error('The count property cannot be undefined');
     }
-    if (currentPage === undefined) {
-      throw new Error('The currentPage property cannot be undefined');
-    }
     if (pathname === undefined) {
       throw new Error('The pathname property cannot be undefined');
     }

tests/unit/core/components/TestPaginate.js

@@ -30,11 +30,6 @@ describe('<Paginate />', () => {
           .toThrowError(/count property cannot be undefined/);
       });
 
-      it('does not allow an undefined currentPage', () => {
-        expect(() => renderPaginate({ currentPage: undefined }))
-          .toThrowError(/currentPage property cannot be undefined/);
-      });
-
       it('does not allow an undefined pathname', () => {
         expect(() => renderPaginate({ pathname: undefined }))
           .toThrowError(/pathname property cannot be undefined/);
@@ -183,4 +178,11 @@ describe('<Paginate />', () => {
     expect(firstLink).toHaveProp('pathname', pathname);
     expect(firstLink).toHaveProp('pageCount', pageCount);
   });
+
+  it('defaults currentPage to 1', () => {
+    const root = renderPaginate({ currentPage: undefined, count: 30 });
+
+    const firstLink = root.find(PaginatorLink).first();
+    expect(firstLink).toHaveProp('currentPage', 1);
+  });
 });