Przeglądaj źródła

added test for issue #979

Sebastian Stenzel 5 lat temu
rodzic
commit
e2f400340b

+ 1 - 2
main/ui/src/main/java/org/cryptomator/ui/common/PasswordStrengthUtil.java

@@ -31,8 +31,7 @@ public class PasswordStrengthUtil {
 	public PasswordStrengthUtil(ResourceBundle resourceBundle) {
 		this.resourceBundle = resourceBundle;
 		this.zxcvbn = new Zxcvbn();
-		this.sanitizedInputs = new ArrayList<>();
-		this.sanitizedInputs.add("cryptomator");
+		this.sanitizedInputs = List.of("cryptomator");
 	}
 
 	public int computeRate(String password) {

+ 11 - 0
main/ui/src/test/java/org/cryptomator/ui/common/PasswordStrengthUtilTest.java

@@ -2,6 +2,7 @@ package org.cryptomator.ui.common;
 
 import com.google.common.base.Strings;
 import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.mockito.Mockito;
 
@@ -18,5 +19,15 @@ public class PasswordStrengthUtilTest {
 			util.computeRate(longPw);
 		});
 	}
+	
+	@Test
+	@Disabled("waiting on upstream fix")
+	public void testIssue979() {
+		PasswordStrengthUtil util = new PasswordStrengthUtil(Mockito.mock(ResourceBundle.class));
+		int result1 = util.computeRate("backed derrick buckling mountains glove client procedures desire destination sword hidden ram");
+		int result2 = util.computeRate("backed derrick buckling mountains glove client procedures desire destination sword hidden ram escalation");
+		Assertions.assertEquals(4, result1);
+		Assertions.assertEquals(4, result2);
+	}
 
 }