mirror of
https://github.com/strapi/strapi.git
synced 2025-08-02 22:07:11 +00:00
Revert "admin lastname defaults to undefined"
This commit is contained in:
parent
f36e9256a2
commit
dbcf405f67
@ -98,10 +98,6 @@ const Register = ({ authType, fieldsToDisable, noSignin, onSubmit, schema }) =>
|
|||||||
|
|
||||||
if (!['password', 'confirmPassword'].includes(key) && typeof value === 'string') {
|
if (!['password', 'confirmPassword'].includes(key) && typeof value === 'string') {
|
||||||
normalizedvalue = normalizedvalue.trim();
|
normalizedvalue = normalizedvalue.trim();
|
||||||
|
|
||||||
if (key === 'lastname') {
|
|
||||||
normalizedvalue = normalizedvalue || null;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
acc[key] = normalizedvalue;
|
acc[key] = normalizedvalue;
|
||||||
|
@ -136,61 +136,6 @@ describe('ADMIN | PAGES | AUTH | Register', () => {
|
|||||||
);
|
);
|
||||||
});
|
});
|
||||||
|
|
||||||
it('Validates optional Lastname value to be null', async () => {
|
|
||||||
const spy = jest.fn();
|
|
||||||
const { getByRole, getByLabelText, user } = setup({ onSubmit: spy });
|
|
||||||
|
|
||||||
await user.type(getByLabelText(/Firstname/i), 'First name');
|
|
||||||
await user.type(getByLabelText(/Email/i), 'test@strapi.io');
|
|
||||||
await user.type(getByLabelText(/^Password/i), 'secret');
|
|
||||||
await user.type(getByLabelText(/Confirm Password/i), 'secret');
|
|
||||||
|
|
||||||
fireEvent.click(getByRole('button', { name: /let's start/i }));
|
|
||||||
|
|
||||||
await waitFor(() =>
|
|
||||||
expect(spy).toHaveBeenCalledWith(
|
|
||||||
{
|
|
||||||
firstname: 'First name',
|
|
||||||
lastname: null,
|
|
||||||
email: 'test@strapi.io',
|
|
||||||
news: false,
|
|
||||||
registrationToken: undefined,
|
|
||||||
confirmPassword: 'secret',
|
|
||||||
password: 'secret',
|
|
||||||
},
|
|
||||||
expect.any(Object)
|
|
||||||
)
|
|
||||||
);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('Validates optional Lastname value to be empty space', async () => {
|
|
||||||
const spy = jest.fn();
|
|
||||||
const { getByRole, getByLabelText, user } = setup({ onSubmit: spy });
|
|
||||||
|
|
||||||
await user.type(getByLabelText(/Firstname/i), 'First name');
|
|
||||||
await user.type(getByLabelText(/Lastname/i), ' ');
|
|
||||||
await user.type(getByLabelText(/Email/i), 'test@strapi.io');
|
|
||||||
await user.type(getByLabelText(/^Password/i), 'secret');
|
|
||||||
await user.type(getByLabelText(/Confirm Password/i), 'secret');
|
|
||||||
|
|
||||||
fireEvent.click(getByRole('button', { name: /let's start/i }));
|
|
||||||
|
|
||||||
await waitFor(() =>
|
|
||||||
expect(spy).toHaveBeenCalledWith(
|
|
||||||
{
|
|
||||||
firstname: 'First name',
|
|
||||||
lastname: null,
|
|
||||||
email: 'test@strapi.io',
|
|
||||||
news: false,
|
|
||||||
registrationToken: undefined,
|
|
||||||
confirmPassword: 'secret',
|
|
||||||
password: 'secret',
|
|
||||||
},
|
|
||||||
expect.any(Object)
|
|
||||||
)
|
|
||||||
);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('Disable fields', () => {
|
it('Disable fields', () => {
|
||||||
const { getByLabelText } = setup({
|
const { getByLabelText } = setup({
|
||||||
fieldsToDisable: ['email', 'firstname'],
|
fieldsToDisable: ['email', 'firstname'],
|
||||||
|
Loading…
x
Reference in New Issue
Block a user