Skip to content

Commit bf9c00b

Browse files
Linda PengLinda Peng
Linda Peng
authored and
Linda Peng
committed
Attempt to fix console warnings re: react testing library using await act(async () =>
1 parent d842ed6 commit bf9c00b

File tree

1 file changed

+44
-26
lines changed

1 file changed

+44
-26
lines changed

Diff for: src/components/Auth/AuthForm.spec.js

+44-26
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ describe('AuthForm', () => {
4040
});
4141

4242
describe('Signup', () => {
43-
fit('Register a new user on the signup form', async () => {
43+
it('Register a new user on the signup form', async () => {
4444
await act(async () => {
4545
const { getByText, getByLabelText } = render(
4646
<BrowserRouter>
@@ -58,27 +58,40 @@ describe('Signup', () => {
5858
5959
},
6060
});
61+
await act(async () =>
62+
fireEvent.change(getByLabelText(/username/i), {
63+
target: { value: 'Carolyne.Carter' },
64+
})
65+
);
6166

62-
fireEvent.change(getByLabelText(/username/i), {
63-
target: { value: 'Carolyne.Carter' },
64-
});
67+
await act(async () =>
68+
fireEvent.change(getByLabelText(/password/i), {
69+
target: { value: 'password' },
70+
})
71+
);
72+
73+
await act(async () =>
74+
fireEvent.change(getByLabelText(/email/i), {
75+
target: { value: '[email protected]' },
76+
})
77+
);
78+
79+
await act(async () =>
80+
fireEvent.change(getByLabelText(/first name/i), {
81+
target: { value: 'Carolyne' },
82+
})
83+
);
84+
85+
await act(async () =>
86+
fireEvent.change(getByLabelText(/last name/i), {
87+
target: { value: 'Carter' },
88+
})
89+
);
6590

66-
fireEvent.change(getByLabelText(/password/i), {
67-
target: { value: 'password' },
68-
});
69-
fireEvent.change(getByLabelText(/email/i), {
70-
target: { value: '[email protected]' },
71-
});
72-
fireEvent.change(getByLabelText(/first name/i), {
73-
target: { value: 'Carolyne' },
74-
});
75-
fireEvent.change(getByLabelText(/last name/i), {
76-
target: { value: 'Carter' },
77-
});
7891
const submit = getByText('Sign Up');
79-
fireEvent.click(submit);
92+
await act(async () => fireEvent.click(submit));
8093

81-
await mockRegisterResponse();
94+
await act(async () => mockRegisterResponse());
8295

8396
expect(mockRegisterResponse).toHaveBeenCalledTimes(1);
8497
});
@@ -100,16 +113,21 @@ describe('Login', () => {
100113
username: 'Carolyne.Carter',
101114
},
102115
});
103-
fireEvent.change(getByLabelText(/username/i), {
104-
target: { value: 'Carolyne.Carter' },
105-
});
116+
await act(async () =>
117+
fireEvent.change(getByLabelText(/username/i), {
118+
target: { value: 'Carolyne.Carter' },
119+
})
120+
);
121+
122+
await act(async () =>
123+
fireEvent.change(getByLabelText(/password/i), {
124+
target: { value: 'password' },
125+
})
126+
);
106127

107-
fireEvent.change(getByLabelText(/password/i), {
108-
target: { value: 'password' },
109-
});
110128
const submit = getByRole('button');
111-
fireEvent.click(submit);
112-
await mockLoginResponse();
129+
await act(async () => fireEvent.click(submit));
130+
await act(async () => mockLoginResponse());
113131
expect(mockLoginResponse).toHaveBeenCalledTimes(1);
114132
});
115133
});

0 commit comments

Comments
 (0)