From fc92c555ad4c4cde7653f6fd0f42b6edca59b1cd Mon Sep 17 00:00:00 2001 From: binsky Date: Fri, 18 Feb 2022 10:37:25 +0100 Subject: [PATCH 1/2] fix vault test encoding to run maven test on linux console --- src/main/java/Vault.java | 6 +++--- src/test/java/VaultTest.java | 7 ++++--- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/src/main/java/Vault.java b/src/main/java/Vault.java index 1fe8849..d2f9e40 100644 --- a/src/main/java/Vault.java +++ b/src/main/java/Vault.java @@ -116,7 +116,7 @@ public class Vault implements VaultInterface { println("Your PWs contain now capitals."); }else if(input.equals("no")){ haveCapitals = false; - println("Your PWs don´t have capitals anymore."); + println("Your PWs don't have capitals anymore."); } } @@ -128,7 +128,7 @@ public class Vault implements VaultInterface { println("Your PWs contain now special characters."); }else if(input.equals("no")){ hasSpecialChars = false; - println("Your PWs don´t have special characters anymore."); + println("Your PWs don't have special characters anymore."); } } @@ -139,7 +139,7 @@ public class Vault implements VaultInterface { println("Your PWs contains now numbers."); }else if(input.equals("no")){ hasNumbers = false; - println("Your PWs don´t have numbers anymore."); + println("Your PWs don't have numbers anymore."); } } diff --git a/src/test/java/VaultTest.java b/src/test/java/VaultTest.java index a4dac9d..1e2661f 100644 --- a/src/test/java/VaultTest.java +++ b/src/test/java/VaultTest.java @@ -180,7 +180,8 @@ public class VaultTest { vlt.setCapital("no"); assertFalse(vlt.haveCapitals = false); - assertTrue(outputStream.toString().contains("don´t")); + assertTrue(outputStream.toString().contains("don't")); + System.out.println(outputStream.toString()); } /*@Test @@ -204,7 +205,7 @@ public class VaultTest { vlt.setSpecialChar("no"); assertFalse(vlt.hasSpecialChars = false); - assertTrue(outputStream.toString().contains("don´t")); + assertTrue(outputStream.toString().contains("don't")); } /*@Test @@ -229,7 +230,7 @@ public class VaultTest { vlt.setNumbers("no"); assertFalse(vlt.hasNumbers = false); - assertTrue(outputStream.toString().contains("don´t")); + assertTrue(outputStream.toString().contains("don't")); } private ByteArrayInputStream getEmptyStringInputStream() { From e3a9523d2b558ff27cc604cd8d951aff47d0761d Mon Sep 17 00:00:00 2001 From: binsky Date: Fri, 18 Feb 2022 11:06:40 +0100 Subject: [PATCH 2/2] add test and package hints to readme --- README.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/README.md b/README.md index 471c5eb..881fec4 100644 --- a/README.md +++ b/README.md @@ -22,3 +22,9 @@ - binsky - Timo Triebensky - fdai6352 - Pascal Schubert - fdai5728 - Claudia Metzler + +## Beim Testen beachten! +- zwischen den Commits `d1652f044327f4845e11db0cce6ef6c4fb392113` und `a81194074a49eabe723e7ad599411d45545a5b1d` werden die Maven Tests auf Grund einer fehlerhaften Änderung in der pom.xml nicht ausgeführt. Die JUnit Tests sollten aber dennoch funktionieren. Maven sagt lediglich, dass alles in Ordnung ist, auch wenn keine Tests ausgeführt werden. Daher ist das Problem erst etwas später aufgefallen. +- ab dem Commit `a81194074a49eabe723e7ad599411d45545a5b1d` kann auf die oben beschriebene Weise ein Artefakt erfolgreich gebaut und ausgeführt werden. Bauen hat auch vorher schon funktioniert, aber die jar Datei konnte nicht ausgeführt werden. +- für die Tests ist eine Netzwerkverbindung, sowie funktionierendes DNS nötig, da eine Verbindung zu api.pwnedpasswords.com aufgebaut wird. +- da für ein erstes Release noch nicht genug Funktionalität implementiert ist, gab es bisher noch keinen Merge/Pull Request auf den main Branch.