Merge pull request #1325 from vl4ds4m/bugfix-1321

Fix a bug with missed check a closing bracket in nested expressions

Author: yegor256

qulice-checkstyle/src/main/java/com/qulice/checkstyle/BracketsStructureCheck.java

@@ -102,7 +102,10 @@ private void checkLines(final DetailAST node, final int start,
      */
     private void checkExpressionList(final DetailAST elist, final int end) {
         if (elist.getChildCount() > 0) {
-            final DetailAST last = elist.getLastChild();
+            DetailAST last = elist.getLastChild();
+            while (last.getChildCount() > 0) {
+                last = last.getLastChild();
+            }
             final int lline = last.getLineNo();
             if (lline == end) {
                 this.log(lline, "Closing bracket should be on a new line");

qulice-checkstyle/src/test/resources/com/qulice/checkstyle/ChecksTest/BracketsStructureCheck/Invalid.java

@@ -41,4 +41,31 @@ public void main() {
       Invalid d1 = new Invalid(
         x, y);
     }
+    // Check nested method call
+    public String nest(String... lines) {
+        nest(
+          nest(
+            "Good"
+          )
+        );
+        nest(
+          nest(
+            "Bad"
+          ));
+        nest(
+          nest(
+            "Good"
+          ),
+          nest(
+             "Bad"
+          ));
+        nest(
+          nest(
+            "Good"
+          ),
+          nest(
+            "Good"
+          )
+        );
+    }
 }

qulice-checkstyle/src/test/resources/com/qulice/checkstyle/ChecksTest/BracketsStructureCheck/violations.txt

@@ -14,3 +14,5 @@
 37:Closing bracket should be on a new line
 40:Closing bracket should be on a new line
 42:Closing bracket should be on a new line
+54:Closing bracket should be on a new line
+61:Closing bracket should be on a new line