minor: Specified violation messages in SeparatorWrapCheck

Vyom-Yadav · romani · commit e12eee4f01b3 · 2022-01-29T15:10:06.000-08:00
diff --git a/src/test/java/com/puppycrawl/tools/checkstyle/bdd/InlineConfigParser.java b/src/test/java/com/puppycrawl/tools/checkstyle/bdd/InlineConfigParser.java
@@ -114,7 +114,6 @@ public final class InlineConfigParser {
             "com.puppycrawl.tools.checkstyle.checks.naming.AbstractClassNameCheck",
             "com.puppycrawl.tools.checkstyle.checks.sizes.MethodCountCheck",
             "com.puppycrawl.tools.checkstyle.checks.coding.PackageDeclarationCheck",
-            "com.puppycrawl.tools.checkstyle.checks.whitespace.SeparatorWrapCheck",
             "com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocTypeCheck",
             "com.puppycrawl.tools.checkstyle.checks.blocks.LeftCurlyCheck",
             "com.puppycrawl.tools.checkstyle.checks.regexp.RegexpCheck",
diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/separatorwrap/InputSeparatorWrapForArrayDeclarator.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/separatorwrap/InputSeparatorWrapForArrayDeclarator.java
@@ -14,7 +14,6 @@ class InputSeparatorWrapForArrayDeclarator {
             ] {1, 2};
 
     protected int[] arrayDeclarationWithBadWrapping = new int
-            [] {1, 2}; // violation
-
+            [] {1, 2}; // violation ''\[' should be on the previous line'
 }
 
diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/separatorwrap/InputSeparatorWrapForEllipsis.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/separatorwrap/InputSeparatorWrapForEllipsis.java
@@ -16,7 +16,7 @@ public void testMethodWithGoodWrapping(String...
     }
 
     public void testMethodWithBadWrapping(String
-            ...parameters) { // violation
+            ...parameters) { // violation ''...' should be on the previous line'
 
     }
 
diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/separatorwrap/InputSeparatorWrapForTestDot.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/separatorwrap/InputSeparatorWrapForTestDot.java
@@ -35,7 +35,7 @@ class badCaseForTestDot<T extends FooForTestDot &  BarForTestDot> {
 
     public void goodCaseForTestDot(int... aFoo)throws FooExceptionForTestDot, BarExceptionForTestDot
     {
-        String s = "ffffooooString"; // violation below
+        String s = "ffffooooString"; // violation below ''.' should be on a new line'
         s.
                 isEmpty(); //bad wrapping
         try {
diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/separatorwrap/InputSeparatorWrapForTestMethodRef.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/separatorwrap/InputSeparatorWrapForTestMethodRef.java
@@ -21,7 +21,7 @@ void goodCase() {
 
     void badCase() {
         String[] stringArray = { "Barbara", "James", "Mary", "John",
-            "Patricia", "Robert", "Michael", "Linda" }; // violation below
+            "Patricia", "Linda" }; // violation below ''::' should be on a new line'
         /* violation */ Arrays.sort(stringArray, String::
                 compareToIgnoreCase);
     }

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@ class InputSeparatorWrapForArrayDeclarator {`
`14`	`14`	`] {1, 2};`
`15`	`15`
`16`	`16`	`protected int[] arrayDeclarationWithBadWrapping = new int`
`17`		`- [] {1, 2}; // violation`
`18`		`-`
	`17`	`+ [] {1, 2}; // violation ''\[' should be on the previous line'`
`19`	`18`	`}`
`20`	`19`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ public void testMethodWithGoodWrapping(String...`
`16`	`16`	`}`
`17`	`17`
`18`	`18`	`public void testMethodWithBadWrapping(String`
`19`		`- ...parameters) { // violation`
	`19`	`+ ...parameters) { // violation ''...' should be on the previous line'`
`20`	`20`
`21`	`21`	`}`
`22`	`22`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ class badCaseForTestDot<T extends FooForTestDot & BarForTestDot> {`
`35`	`35`
`36`	`36`	`public void goodCaseForTestDot(int... aFoo)throws FooExceptionForTestDot, BarExceptionForTestDot`
`37`	`37`	`{`
`38`		`- String s = "ffffooooString"; // violation below`
	`38`	`+ String s = "ffffooooString"; // violation below ''.' should be on a new line'`
`39`	`39`	`s.`
`40`	`40`	`isEmpty(); //bad wrapping`
`41`	`41`	`try {`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ void goodCase() {`
`21`	`21`
`22`	`22`	`void badCase() {`
`23`	`23`	`String[] stringArray = { "Barbara", "James", "Mary", "John",`
`24`		`- "Patricia", "Robert", "Michael", "Linda" }; // violation below`
	`24`	`+ "Patricia", "Linda" }; // violation below ''::' should be on a new line'`
`25`	`25`	`/* violation */ Arrays.sort(stringArray, String::`
`26`	`26`	`compareToIgnoreCase);`
`27`	`27`	`}`