Skip to content

Commit

Permalink
Fix 5 warnings in SelectAsyncControl test
Browse files Browse the repository at this point in the history
  • Loading branch information
lyndsiWilliams committed Feb 2, 2023
1 parent 56d4bd0 commit 745dc59
Showing 1 changed file with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -65,17 +65,17 @@ beforeEach(() => {
jest.resetAllMocks();
});

test('Should render', () => {
test('Should render', async () => {
const props = createProps();
render(<SelectAsyncControl {...props} />, { useRedux: true });
expect(screen.getByTestId('select-test')).toBeInTheDocument();
expect(await screen.findByTestId('select-test')).toBeInTheDocument();
});

test('Should send correct props to Select component - value props', () => {
test('Should send correct props to Select component - value props', async () => {
const props = createProps();
render(<SelectAsyncControl {...props} />, { useRedux: true });

expect(screen.getByTestId('select-test')).toHaveAttribute(
expect(await screen.findByTestId('select-test')).toHaveAttribute(
'data-value',
JSON.stringify(props.value),
);
Expand All @@ -89,20 +89,20 @@ test('Should send correct props to Select component - value props', () => {
);
});

test('Should send correct props to Select component - function onChange multi:true', () => {
test('Should send correct props to Select component - function onChange multi:true', async () => {
const props = createProps();
render(<SelectAsyncControl {...props} />, { useRedux: true });
expect(props.onChange).toBeCalledTimes(0);
userEvent.click(screen.getByText('onChange'));
userEvent.click(await screen.findByText('onChange'));
expect(props.onChange).toBeCalledTimes(1);
});

test('Should send correct props to Select component - function onChange multi:false', () => {
test('Should send correct props to Select component - function onChange multi:false', async () => {
const props = createProps();
render(<SelectAsyncControl {...{ ...props, multi: false }} />, {
useRedux: true,
});
expect(props.onChange).toBeCalledTimes(0);
userEvent.click(screen.getByText('onChange'));
userEvent.click(await screen.findByText('onChange'));
expect(props.onChange).toBeCalledTimes(1);
});

0 comments on commit 745dc59

Please sign in to comment.