Refactor password validation logic and enhance test coverage

Author: Payman IB

Project-CLI-Treasure-Hunt

@@ -0,0 +1 @@
+Subproject commit 6668c42dc5412e54026ca0fa8cb5691017ef6350

Sprint-3/3-stretch/password-validator.js

@@ -1,6 +1,41 @@
+// function isLongEnough(password) {
+//     return password.length >= 5; 
+// }
+// function isUpperCase(password) {
+//     return /[A-Z]/.test(password);
+// }
+
+// function isLowerCase(password) {
+//     return /[a-z]/.test(password);
+// }
+
+// function hasNumber(password) {
+//     return /[0-9]/.test(password);
+// }
+
+// function hasSpecialChar(password) {
+//     return /[!#$%.*&]/.test(password);
+// } 
+// function passwordValidator(password) {
+    
+//     if (isLongEnough(password) && isUpperCase(password) && isLowerCase(password) && hasNumber(password) && hasSpecialChar(password)) 
+//         return true 
+//     else 
+//         return false; 
+// }
+
+/////////////////////////////////////////////////////////////////////
+
 function passwordValidator(password) {
-    return password.length < 5 ? false : true
+  const isLongEnough = password.length >= 5;
+  const hasUpper = /[A-Z]/.test(password);
+  const hasLower = /[a-z]/.test(password);
+  const hasNumber = /[0-9]/.test(password);
+  const hasSpecial = /[!#$%.*&]/.test(password);
+
+  return isLongEnough && hasUpper && hasLower && hasNumber && hasSpecial;
 }
 
+console.log(passwordValidator("Password123!")); 
 
 module.exports = passwordValidator;

Sprint-3/3-stretch/password-validator.test.js

@@ -16,11 +16,32 @@ You must breakdown this problem in order to solve it. Find one test case first a
 */
 const isValidPassword = require("./password-validator");
 test("password has at least 5 characters", () => {
-    // Arrange
-    const password = "12345";
-    // Act
+    const password = "12345Aa!";
     const result = isValidPassword(password);
-    // Assert
     expect(result).toEqual(true);
 }
-);
+);
+test("password missing uppercase letter", () => {
+    const password = "12345aa!";
+    const result = isValidPassword(password);
+    expect(result).toEqual(false);
+}
+);
+test("password missing lowercase letter", () => {
+    const password = "12345AA!";
+    const result = isValidPassword(password);
+    expect(result).toEqual(false);
+}
+);
+test("password missing number", () => {
+    const password = "Abcde!";
+    const result = isValidPassword(password);
+    expect(result).toEqual(false);
+}
+);
+test("password missing special character", () => {
+    const password = "12345Aa";
+    const result = isValidPassword(password);
+    expect(result).toEqual(false);
+}
+);