refactor(component): enum Colors contains string and improved special chars check

Author: AnthonyNahas

src/module/component/password-strength/password-strength.component.ts

@@ -1,9 +1,9 @@
 import {Component, Input, OnChanges, OnInit, SimpleChanges} from '@angular/core';
 
 export enum Colors {
-  primary,
-  accent,
-  warn
+  primary = 'primary',
+  accent = 'accent',
+  warn = 'warn'
 }
 
 @Component({
@@ -38,31 +38,30 @@ export class PasswordStrengthComponent implements OnInit, OnChanges {
   ngOnChanges(changes: SimpleChanges): void {
     console.log('on change: ', changes);
     if (changes.externalError && changes.externalError.firstChange) {
-      this._color = Colors[Colors.primary];
+      this._color = Colors.primary;
       return;
     }
     if (changes.externalError && changes.externalError.currentValue) {
-      this._color = Colors[Colors.warn];
+      this._color = Colors.warn;
       return;
     }
     this.password && this.password.length > 0 ?
-      this._calculatePasswordStrength() : this._strength = 0;
+      this.calculatePasswordStrength() : this._strength = 0;
   }
 
 
   get strength(): number {
-    // console.log('strength = ', this._strength);
-    return this._strength;
+    return this._strength ? this._strength : 0;
   }
 
   get color(): string {
 
     if (this._strength <= 20) {
-      return Colors[Colors.warn];
+      return Colors.warn;
     } else if (this._strength <= 80) {
-      return Colors[Colors.accent];
+      return Colors.accent;
     } else {
-      return Colors[Colors.primary];
+      return Colors.primary;
     }
   }
 
@@ -83,11 +82,10 @@ export class PasswordStrengthComponent implements OnInit, OnChanges {
   }
 
   private _containAtLeastOneSpecialChar(): boolean {
-    return RegExp(/^(?=.*?[#?!@$%^&*-])/).test(this.password);
+    return RegExp(/^(?=.*?[" !"#$%&'()*+,-./:;<=>?@[\]^_`{|}~"])/).test(this.password);
   }
 
-  private _calculatePasswordStrength() {
-    console.log('on _calculatePasswordStrength()');
+  calculatePasswordStrength() {
     const requirements: Array<boolean> = [];
     const unit = 100 / 5;