diff --git a/src/app/access-control/epeople-registry/eperson-form/eperson-form.component.spec.ts b/src/app/access-control/epeople-registry/eperson-form/eperson-form.component.spec.ts index 6969b44f832..50edb7fcd25 100644 --- a/src/app/access-control/epeople-registry/eperson-form/eperson-form.component.spec.ts +++ b/src/app/access-control/epeople-registry/eperson-form/eperson-form.component.spec.ts @@ -316,15 +316,28 @@ describe('EPersonFormComponent', () => { }); }); + describe('with uppercased email specified', () => { + beforeEach(() => { + component.formGroup.controls.firstName.setValue('test'); + component.formGroup.controls.lastName.setValue('test'); + component.formGroup.controls.email.setValue('TEST@test.com'); + fixture.detectChanges(); + }); + + it('passes validation check', () => { + expect(component.formGroup.controls.email.valid).toBeTrue(); + expect(component.formGroup.controls.email.errors).toBeNull(); + }); + }); describe('after inserting email wrong should show pattern validation error', () => { beforeEach(() => { - component.formGroup.controls.email.setValue('test@test'); + component.formGroup.controls.email.setValue('test'); fixture.detectChanges(); }); it('email should not be valid because the email pattern', () => { expect(component.formGroup.controls.email.valid).toBeFalse(); - expect(component.formGroup.controls.email.errors.pattern).toBeTruthy(); + expect(component.formGroup.controls.email.errors.email).toBeTruthy(); }); }); diff --git a/src/app/access-control/epeople-registry/eperson-form/eperson-form.component.ts b/src/app/access-control/epeople-registry/eperson-form/eperson-form.component.ts index 14241938fca..ad3c389aa43 100644 --- a/src/app/access-control/epeople-registry/eperson-form/eperson-form.component.ts +++ b/src/app/access-control/epeople-registry/eperson-form/eperson-form.component.ts @@ -300,12 +300,12 @@ export class EPersonFormComponent implements OnInit, OnDestroy { name: 'email', validators: { required: null, - pattern: '^[a-z0-9._%+-]+@[a-z0-9.-]+\\.[a-z]{2,4}$', + email: null, }, required: true, errorMessages: { emailTaken: 'error.validation.emailTaken', - pattern: 'error.validation.NotValidEmail', + email: 'error.validation.NotValidEmail', }, hint: this.translateService.instant(`${this.messagePrefix}.emailHint`), });