Modified email validation directive to have a more restrictive regex

src/modules/email-validation/email-validation.component.spec.ts

@@ -76,7 +76,18 @@ describe('Email validation', () => {
         expect(ngModel.control.pristine).toBe(false);
         expect(ngModel.control.touched).toBe(false);
     }));
-
+    it('should validate incorrect input — multiple @ symbols in email', fakeAsync(() => {
+      fixture.detectChanges();
+      tick();
+      setInput(nativeElement, 'joe@abc.com@abc.com', fixture);
+      fixture.detectChanges();
+
+      expect(nativeElement.querySelector('input').value).toBe('joe@abc.com@abc.com');
+
+      expect(ngModel.control.valid).toBe(false);
+      expect(ngModel.control.pristine).toBe(false);
+      expect(ngModel.control.touched).toBe(false);
+  }));
     it('should handle invalid and then valid input', fakeAsync(() => {
         fixture.detectChanges();
         tick();

src/modules/email-validation/email-validation.directive.ts

@@ -34,7 +34,9 @@ export class SkyEmailValidationDirective implements Validator {
   }
 
   public emailIsValid(email: string): boolean {
-    let regex = /[\w\-]+@([\w\-]+\.)+[\w\-]+/;
+    // The regex was obtained from http://emailregex.com/ — which claims to correctly handle ~99% of all email addresses.
+    // tslint:disable-next-line:max-line-length
+    let regex = /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;
     return regex.test(email);
   }
 }