From 66df06dac14230931fb793d4b2e49f8fc948c3d6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Vincent=20Membr=C3=A9?= <vincent.membre@normation.com>
Date: Fri, 6 Dec 2019 10:05:20 +0100
Subject: [PATCH] Fixes #16357: Broken test on whitespace matching and regex is
 invalid for multiline

---
 .../src/main/scala/com/normation/rudder/ncf/Technique.scala | 5 +++--
 .../com/normation/rudder/ncf/TestTechniqueWriter.scala      | 6 +++---
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/webapp/sources/rudder/rudder-core/src/main/scala/com/normation/rudder/ncf/Technique.scala b/webapp/sources/rudder/rudder-core/src/main/scala/com/normation/rudder/ncf/Technique.scala
index 464df879118..a6e6c640f39 100644
--- a/webapp/sources/rudder/rudder-core/src/main/scala/com/normation/rudder/ncf/Technique.scala
+++ b/webapp/sources/rudder/rudder-core/src/main/scala/com/normation/rudder/ncf/Technique.scala
@@ -37,9 +37,10 @@
 
 package com.normation.rudder.ncf
 
+import java.util.regex.Pattern
+
 import com.normation.errors.PureResult
 import com.normation.errors.Unexpected
-
 import cats.data.NonEmptyList
 import com.normation.inventory.domain.Version
 import com.normation.inventory.domain.AgentType
@@ -202,7 +203,7 @@ object Constraint {
 
   case object NoWhiteSpace extends  Constraint {
     def check(value: String): CheckResult = {
-      if (value.matches("""^(?!\s).*(?<!\s)$""")) {
+      if (Pattern.compile("""^(?!\s).*(?<!\s)$""", Pattern.DOTALL).asPredicate().test(value)) {
         OK
       } else {
         NOK(one("Must not have leading or trailing whitespaces"))
diff --git a/webapp/sources/rudder/rudder-core/src/test/scala/com/normation/rudder/ncf/TestTechniqueWriter.scala b/webapp/sources/rudder/rudder-core/src/test/scala/com/normation/rudder/ncf/TestTechniqueWriter.scala
index 2eeba964299..55d41a30f38 100644
--- a/webapp/sources/rudder/rudder-core/src/test/scala/com/normation/rudder/ncf/TestTechniqueWriter.scala
+++ b/webapp/sources/rudder/rudder-core/src/test/scala/com/normation/rudder/ncf/TestTechniqueWriter.scala
@@ -408,12 +408,14 @@ class TestTechniqueWriter extends Specification with ContentMatchers with Loggab
       val value3 = "S"
       val value4 = "ééé ```"
       val value5 = "sdfsqdfsqfsdf sfhdskjhdfs jkhsdkfjhksqdhf"
+      val value6 = ""
 
       Constraint.NoWhiteSpace.check(value1) must equalTo(Constraint.OK)
       Constraint.NoWhiteSpace.check(value2) must equalTo(Constraint.OK)
       Constraint.NoWhiteSpace.check(value3) must equalTo(Constraint.OK)
       Constraint.NoWhiteSpace.check(value4) must equalTo(Constraint.OK)
       Constraint.NoWhiteSpace.check(value5) must equalTo(Constraint.OK)
+      Constraint.NoWhiteSpace.check(value6) must equalTo(Constraint.OK)
     }
 
     "Correctly refuse text starting or ending with withspace" in {
@@ -422,14 +424,12 @@ class TestTechniqueWriter extends Specification with ContentMatchers with Loggab
         """ Some
           |text""".stripMargin
       val value3 = " "
-      val value4 = ""
-      val value5 = "sdfsqdfsqfsdf sfhdskjhdfs jkhsdkfjhksqdhf "
+      val value4 = "sdfsqdfsqfsdf sfhdskjhdfs jkhsdkfjhksqdhf "
 
       Constraint.NoWhiteSpace.check(value1) must haveClass[Constraint.NOK]
       Constraint.NoWhiteSpace.check(value2) must haveClass[Constraint.NOK]
       Constraint.NoWhiteSpace.check(value3) must haveClass[Constraint.NOK]
       Constraint.NoWhiteSpace.check(value4) must haveClass[Constraint.NOK]
-      Constraint.NoWhiteSpace.check(value5) must haveClass[Constraint.NOK]
     }
   }