The MessageFormat class must not be used to format check message when there isn't any parameter. Indeed, otherwise the message can't contain some patterns like "if(){".

1 files changed, 8 insertions, 2 deletions

diff --git a/sonar-squid/src/test/java/org/sonar/squid/api/CheckMessageTest.java b/sonar-squid/src/test/java/org/sonar/squid/api/CheckMessageTest.java
index d2f59c2ef79..1f7e5734918 100644
--- a/sonar-squid/src/test/java/org/sonar/squid/api/CheckMessageTest.java
+++ b/sonar-squid/src/test/java/org/sonar/squid/api/CheckMessageTest.java
@@ -19,11 +19,11 @@
  */
 package org.sonar.squid.api;
 
-import org.junit.Test;
-
 import static org.hamcrest.Matchers.is;
 import static org.junit.Assert.assertThat;
 
+import org.junit.Test;
+
 public class CheckMessageTest {
 
   @Test
@@ -32,4 +32,10 @@ public class CheckMessageTest {
     assertThat(message.formatDefaultMessage(), is("Value is 3, expected value is 7."));
   }
 
+  @Test
+  public void testNotFormatMessageWithoutParameters() {
+    CheckMessage message = new CheckMessage(null, "public void main(){."); // This message can't be used as a pattern by the MessageFormat
+                                                                           // class
+    assertThat(message.formatDefaultMessage(), is("public void main(){."));
+  }
 }