test(package): added a unit test for the min input property of the component

Author: AnthonyNahas

src/module/component/mat-password-strength-info/mat-password-strength-info.component.html

@@ -58,7 +58,7 @@
     </div>
 
     <div class="info-row" @items *ngIf="passwordComponent.enableLengthRule">
-      <div *ngIf="passwordComponent.containAtLeastEightChars; then done else error">
+      <div *ngIf="passwordComponent.containAtLeastMinChars; then done else error">
       </div>
       <ng-template #done>
         <mat-icon @positiveState color="primary">done</mat-icon>

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

@@ -75,6 +75,29 @@ describe('PasswordStrengthComponent', () => {
     expect(calculatePasswordStrengthSpy).toHaveBeenCalled();
   });
 
+  it('should have min input', () => {
+    const calculatePasswordStrengthSpy = jest.spyOn(component, 'calculatePasswordStrength');
+    component.ngOnInit();
+    // default values
+    expect(component.min).toEqual(8);
+    expect(component.containAtLeastMinChars).toBeUndefined();
+    component.min = 5;
+    component.password = '1234';
+    component.ngOnChanges({
+      min: new SimpleChange(component.min, 5, false),
+      password: new SimpleChange(undefined, component.password, false),
+    });
+    fixture.detectChanges();
+    expect(calculatePasswordStrengthSpy).toHaveBeenCalled();
+    expect(component.containAtLeastMinChars).toBeFalsy();
+    component.password = '12345';
+    component.ngOnChanges({
+      password: new SimpleChange('1234', component.password, false),
+    });
+    fixture.detectChanges();
+    expect(component.containAtLeastMinChars).toBeTruthy();
+  });
+
   it('should strength = 20 and color = warn when the password only contain one char ', () => {
     const testChars = ['A', '1', 'a', '.'];
     testChars.forEach(char => {

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

@@ -42,7 +42,7 @@ export class MatPasswordStrengthComponent implements OnInit, OnChanges {
 
   criteriaMap = new Map<Criteria, RegExp>();
 
-  containAtLeastEightChars: boolean;
+  containAtLeastMinChars: boolean;
   containAtLeastOneLowerCaseLetter: boolean;
   containAtLeastOneUpperCaseLetter: boolean;
   containAtLeastOneDigit: boolean;
@@ -86,9 +86,9 @@ export class MatPasswordStrengthComponent implements OnInit, OnChanges {
     }
   }
 
-  private _containAtLeastEightChars(): boolean {
-    this.containAtLeastEightChars = this.password.length >= this.min;
-    return this.containAtLeastEightChars;
+  private _containAtLeastMinChars(): boolean {
+    this.containAtLeastMinChars = this.password.length >= this.min;
+    return this.containAtLeastMinChars;
   }
 
   private _containAtLeastOneLowerCaseLetter(): boolean {
@@ -148,15 +148,15 @@ export class MatPasswordStrengthComponent implements OnInit, OnChanges {
 
   }
 
-  calculatePasswordStrength() {
+  calculatePasswordStrength(): void {
     const requirements: Array<boolean> = [];
     const unit = 100 / this.criteriaMap.size;
 
     // console.log('this.criteriaMap.size = ', this.criteriaMap.size);
     // console.log('unit = ', unit);
 
     requirements.push(
-      this.enableLengthRule ? this._containAtLeastEightChars() : false,
+      this.enableLengthRule ? this._containAtLeastMinChars() : false,
       this.enableLowerCaseLetterRule ? this._containAtLeastOneLowerCaseLetter() : false,
       this.enableUpperCaseLetterRule ? this._containAtLeastOneUpperCaseLetter() : false,
       this.enableDigitRule ? this._containAtLeastOneDigit() : false,
@@ -169,7 +169,7 @@ export class MatPasswordStrengthComponent implements OnInit, OnChanges {
 
   reset() {
     this._strength = 0;
-    this.containAtLeastEightChars =
+    this.containAtLeastMinChars =
       this.containAtLeastOneLowerCaseLetter =
         this.containAtLeastOneUpperCaseLetter =
           this.containAtLeastOneDigit =