Add no-break space character handling to IsEqualCompressingWhiteSpace

ris58h · nhojpatrick · commit 636a24830300 · 2020-07-19T19:54:43.000+01:00
diff --git a/hamcrest/src/main/java/org/hamcrest/text/IsEqualCompressingWhiteSpace.java b/hamcrest/src/main/java/org/hamcrest/text/IsEqualCompressingWhiteSpace.java
@@ -39,7 +39,7 @@ public void describeTo(Description description) {
     }
 
     public String stripSpaces(String toBeStripped) {
-        return toBeStripped.replaceAll("\\s+", " ").trim();
+        return toBeStripped.replaceAll("[\\p{Z}\\p{C}]+", " ").trim();
     }
 
     /**
diff --git a/hamcrest/src/test/java/org/hamcrest/text/IsEqualCompressingWhiteSpaceTest.java b/hamcrest/src/test/java/org/hamcrest/text/IsEqualCompressingWhiteSpaceTest.java
@@ -41,4 +41,8 @@ public void testHasAReadableDescription() {
         assertDescription("a string equal to \" Hello World   how\\n are we? \" compressing white space",
                         matcher);
     }
+
+    public void testPassesIfWhitespacesContainsNoBreakSpace() {
+        assertMatches(matcher, "Hello" + ((char)160) + "World how are we?");
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public void describeTo(Description description) {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`public String stripSpaces(String toBeStripped) {`
`42`		`- return toBeStripped.replaceAll("\\s+", " ").trim();`
	`42`	`+ return toBeStripped.replaceAll("[\\p{Z}\\p{C}]+", " ").trim();`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -41,4 +41,8 @@ public void testHasAReadableDescription() {`
`41`	`41`	`assertDescription("a string equal to \" Hello World how\\n are we? \" compressing white space",`
`42`	`42`	`matcher);`
`43`	`43`	`}`
	`44`	`+`
	`45`	`+ public void testPassesIfWhitespacesContainsNoBreakSpace() {`
	`46`	`+ assertMatches(matcher, "Hello" + ((char)160) + "World how are we?");`
	`47`	`+ }`
`44`	`48`	`}`