fix onChange error when total is string (#357)

* fix: isInteger

* new  test case

Co-authored-by: machixian <[email protected]>

Author: cwjTerrace

src/Pagination.jsx

@@ -8,10 +8,14 @@ import LOCALE from './locale/zh_CN';
 
 function noop() {}
 
-function isInteger(value) {
+function isInteger(v) {
+  const value = Number(v);
   return (
     // eslint-disable-next-line no-restricted-globals
-    typeof value === 'number' && isFinite(value) && Math.floor(value) === value
+    typeof value === 'number' &&
+    !isNaN(value) &&
+    isFinite(value) &&
+    Math.floor(value) === value
   );
 }
 

tests/index.test.js

@@ -416,3 +416,26 @@ describe('current value on onShowSizeChange when total is 0', () => {
     expect(wrapper4.exists('.rc-pagination-options-size-changer')).toBe(true);
   });
 });
+
+describe('should emit onChange when total is string', () => {
+  let wrapper;
+  const onChange = jest.fn();
+
+  beforeEach(() => {
+    wrapper = mount(
+      <Pagination total="100" pageSize={10} onChange={onChange} />,
+    );
+  });
+
+  afterEach(() => {
+    wrapper.unmount();
+    onChange.mockReset();
+  });
+
+  it('onChange should be called when click page', () => {
+    const pagers = wrapper.find('.rc-pagination-item-3');
+    const page1 = pagers.at(0);
+    page1.simulate('click');
+    expect(onChange).toBeCalledWith(3, 10);
+  });
+});