Merge pull request #51 from Johannestegner/master

hotfix: Regex allowing | (pipe).
personnummer · Oct 28, 2021 · f2cd1aa · f2cd1aa
2 parents 8b1d80c + 26adf77
commit f2cd1aa
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/main/java/dev/personnummer/Personnummer.java b/src/main/java/dev/personnummer/Personnummer.java
@@ -15,7 +15,7 @@ public final class Personnummer implements Comparable<Personnummer> {
     private static final Pattern regexPattern;
 
     static {
-        regexPattern = Pattern.compile("^(\\d{2})?(\\d{2})(\\d{2})(\\d{2})([-|+]?)?((?!000)\\d{3})(\\d?)$");
+        regexPattern = Pattern.compile("^(\\d{2})?(\\d{2})(\\d{2})(\\d{2})([-+]?)?((?!000)\\d{3})(\\d?)$");
     }
 
     /**

diff --git a/src/test/java/PersonnummerTest.java b/src/test/java/PersonnummerTest.java
@@ -37,10 +37,10 @@ public void testConstructorCoord(PersonnummerData ssn) {
     @ParameterizedTest
     @MethodSource({"DataProvider#getInvalidPersonnummer", "DataProvider#getValidCoordinationNumbers"})
     public void testConstructorInvalid(PersonnummerData ssn) {
-        assertThrows(PersonnummerException.class, () -> new Personnummer(ssn.longFormat, new Options(false)));
-        assertThrows(PersonnummerException.class, () -> new Personnummer(ssn.shortFormat, new Options(false)));
-        assertThrows(PersonnummerException.class, () -> new Personnummer(ssn.separatedFormat, new Options(false)));
-        assertThrows(PersonnummerException.class, () -> new Personnummer(ssn.separatedFormat, new Options(false)));
+        assertThrows(PersonnummerException.class, () -> new Personnummer(ssn.longFormat, new Options(false)), ssn.longFormat);
+        assertThrows(PersonnummerException.class, () -> new Personnummer(ssn.shortFormat, new Options(false)), ssn.shortFormat);
+        assertThrows(PersonnummerException.class, () -> new Personnummer(ssn.separatedFormat, new Options(false)), ssn.separatedFormat);
+        assertThrows(PersonnummerException.class, () -> new Personnummer(ssn.separatedLong, new Options(false)), ssn.separatedLong);
     }
 
     @ParameterizedTest