TouchInstinct
/
checkstyle
306
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Issue #3803: added new case for if and or operator for IndentationCheck

This commit is contained in:
rnveach 2017-02-13 10:15:58 -05:00 committed by Roman Ivanov
parent 2e29226c84
commit dcd5615e9e
2 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/test/java/com/puppycrawl/tools/checkstyle/checks/indentation/IndentationCheckTest.java
View File

@ -1061,6 +1061,10 @@ public class IndentationCheckTest extends BaseCheckTestSupport {
"245: " + getCheckMessage(MSG_ERROR, "if rparen", 6, 8),
"251: " + getCheckMessage(MSG_ERROR, "if lparen", 6, 8),
"253: " + getCheckMessage(MSG_ERROR, "if rparen", 6, 8),
"256: " + getCheckMessage(MSG_ERROR, "if", 0, 8),
"257: " + getCheckMessage(MSG_CHILD_ERROR, "if", 0, 12),
"258: " + getCheckMessage(MSG_CHILD_ERROR, "if", 0, 12),
"259: " + getCheckMessage(MSG_ERROR, "if rcurly", 0, 8),
};
verifyWarns(checkConfig, fileName, expected);
}

4
src/test/resources/com/puppycrawl/tools/checkstyle/checks/indentation/InputInvalidIfIndent.java
View File

@ -253,6 +253,10 @@ System.getProperty("blah"); //indent:0 exp:12 warn
) { //indent:6 exp:8 warn
System.getProperty("blah"); //indent:12 exp:12
} //indent:8 exp:8
if (test //indent:0 exp:8 warn
|| test) { //indent:0 exp:12 warn
System.getProperty("blah"); //indent:0 exp:12 warn
} //indent:0 exp:8 warn
} //indent:4 exp:4