Skip to content

Commit d842ed6

Browse files
Linda PengLinda Peng
Linda Peng
authored and
Linda Peng
committed
first attempt to fix signup test warning
1 parent 5292775 commit d842ed6

File tree

1 file changed

+38
-36
lines changed

1 file changed

+38
-36
lines changed

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

+38-36
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import React from 'react';
2-
import { fireEvent, render } from '@testing-library/react';
2+
import { fireEvent, render, act } from '@testing-library/react';
33
import '@testing-library/jest-dom/extend-expect';
44
import { BrowserRouter } from 'react-router-dom';
55
import AuthForm from './AuthForm';
@@ -40,46 +40,48 @@ describe('AuthForm', () => {
4040
});
4141

4242
describe('Signup', () => {
43-
it('Register a new user on the signup form', async () => {
44-
const { getByText, getByLabelText } = render(
45-
<BrowserRouter>
46-
<SignUpForm />
47-
</BrowserRouter>
48-
);
43+
fit('Register a new user on the signup form', async () => {
44+
await act(async () => {
45+
const { getByText, getByLabelText } = render(
46+
<BrowserRouter>
47+
<SignUpForm />
48+
</BrowserRouter>
49+
);
4950

50-
const mockRegisterResponse = jest.fn().mockResolvedValue({
51-
data: {
52-
username: 'Carolyne.Carter',
53-
token:
54-
'eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1c2VybmFtZSI6IkNhcm9seW5lLkNhcnRlciIsImlhdCI6MTU4NDMzODQ4NiwiZXhwIjoxNTg0MzQyMDg2LCJ1c2VyX2lkIjo4MCwib3JpZ19pYXQiOjE1ODQzMzg0ODZ9.saO6OCOKV1uwHjTbM-iDGmhbkMNCnzrGFj4TBYnTv2E',
55-
first_name: 'Carolyne',
56-
last_name: 'Carter',
57-
58-
},
59-
});
51+
const mockRegisterResponse = jest.fn().mockResolvedValue({
52+
data: {
53+
username: 'Carolyne.Carter',
54+
token:
55+
'eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1c2VybmFtZSI6IkNhcm9seW5lLkNhcnRlciIsImlhdCI6MTU4NDMzODQ4NiwiZXhwIjoxNTg0MzQyMDg2LCJ1c2VyX2lkIjo4MCwib3JpZ19pYXQiOjE1ODQzMzg0ODZ9.saO6OCOKV1uwHjTbM-iDGmhbkMNCnzrGFj4TBYnTv2E',
56+
first_name: 'Carolyne',
57+
last_name: 'Carter',
58+
59+
},
60+
});
6061

61-
fireEvent.change(getByLabelText(/username/i), {
62-
target: { value: 'Carolyne.Carter' },
63-
});
62+
fireEvent.change(getByLabelText(/username/i), {
63+
target: { value: 'Carolyne.Carter' },
64+
});
6465

65-
fireEvent.change(getByLabelText(/password/i), {
66-
target: { value: 'password' },
67-
});
68-
fireEvent.change(getByLabelText(/email/i), {
69-
target: { value: '[email protected]' },
70-
});
71-
fireEvent.change(getByLabelText(/first name/i), {
72-
target: { value: 'Carolyne' },
73-
});
74-
fireEvent.change(getByLabelText(/last name/i), {
75-
target: { value: 'Carter' },
76-
});
77-
const submit = getByText('Sign Up');
78-
fireEvent.click(submit);
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+
});
78+
const submit = getByText('Sign Up');
79+
fireEvent.click(submit);
7980

80-
await mockRegisterResponse();
81+
await mockRegisterResponse();
8182

82-
expect(mockRegisterResponse).toHaveBeenCalledTimes(1);
83+
expect(mockRegisterResponse).toHaveBeenCalledTimes(1);
84+
});
8385
});
8486
});
8587

0 commit comments

Comments
 (0)