pagination test fixup

This commit is contained in:
Jake McDermott 2019-01-07 07:05:08 -05:00
parent 6a7ba87a02
commit 5bff942110
No known key found for this signature in database
GPG Key ID: 9A6F084352C3A0B7
2 changed files with 17 additions and 16 deletions

View File

@ -122,7 +122,6 @@ describe('<Pagination />', () => {
test('submit a new page by typing in input works', () => {
const textInputSelector = '.pf-l-split__item.pf-m-main .pf-c-form-control';
const submitFormSelector = '.pf-l-split__item.pf-m-main form';
const onSetPage = jest.fn();
pagination = mount(
@ -137,6 +136,7 @@ describe('<Pagination />', () => {
/>
</I18nProvider>
);
const textInput = pagination.find(textInputSelector);
expect(textInput.length).toBe(1);
textInput.simulate('change');
@ -145,7 +145,7 @@ describe('<Pagination />', () => {
const submitForm = pagination.find(submitFormSelector);
expect(submitForm.length).toBe(1);
submitForm.simulate('submit');
pagination.setState({ value: 'invalid' });
pagination.find('Pagination').instance().setState({ value: 'invalid' });
submitForm.simulate('submit');
});

View File

@ -18,8 +18,7 @@ class Pagination extends Component {
constructor (props) {
super(props);
const { page } = this.props;
const { page } = props;
this.state = { value: page, isOpen: false };
this.onPageChange = this.onPageChange.bind(this);
@ -70,18 +69,14 @@ class Pagination extends Component {
const { onSetPage, page, page_size } = this.props;
const previousPage = page - 1;
if (previousPage >= 1) {
onSetPage(previousPage, page_size);
}
onSetPage(previousPage, page_size);
}
onNext () {
const { onSetPage, page, pageCount, page_size } = this.props;
const nextPage = page + 1;
if (nextPage <= pageCount) {
onSetPage(nextPage, page_size);
}
onSetPage(nextPage, page_size);
}
onLast () {
@ -143,14 +138,20 @@ class Pagination extends Component {
direction={up}
isOpen={isOpen}
toggle={(
<DropdownToggle className="togglePageSize" onToggle={this.onTogglePageSize}>
{ page_size }
<DropdownToggle
className="togglePageSize"
onToggle={this.onTogglePageSize}
>
{page_size}
</DropdownToggle>
)}
>
{opts.map(option => (
<DropdownItem key={option} component="button">
{ option }
<DropdownItem
key={option}
component="button"
>
{option}
</DropdownItem>
))}
</Dropdown>
@ -159,7 +160,7 @@ class Pagination extends Component {
<LevelItem>
<Split gutter="md" className="pf-u-display-flex pf-u-align-items-center">
<SplitItem>
<Trans>{ itemMin } - { itemMax } of { count }</Trans>
<Trans>{itemMin} - {itemMax} of {count}</Trans>
</SplitItem>
<SplitItem>
<div className="pf-c-input-group">
@ -200,7 +201,7 @@ class Pagination extends Component {
value={value}
type="text"
onChange={this.onPageChange}
/> of { pageCount }
/> of {pageCount}
</Trans>
</form>
</SplitItem>