Skip to content

Commit

Permalink
Issue checkstyle#6207: Add xpath regression test for AvoidInlineCondi…
Browse files Browse the repository at this point in the history
…tionals
  • Loading branch information
mincong-h committed Aug 16, 2019
1 parent bfd13dc commit a24e112
Show file tree
Hide file tree
Showing 2 changed files with 72 additions and 0 deletions.
@@ -0,0 +1,64 @@
////////////////////////////////////////////////////////////////////////////////
// checkstyle: Checks Java source code for adherence to a set of rules.
// Copyright (C) 2001-2019 the original author or authors.
//
// This library is free software; you can redistribute it and/or
// modify it under the terms of the GNU Lesser General Public
// License as published by the Free Software Foundation; either
// version 2.1 of the License, or (at your option) any later version.
//
// This library is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
// Lesser General Public License for more details.
//
// You should have received a copy of the GNU Lesser General Public
// License along with this library; if not, write to the Free Software
// Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
////////////////////////////////////////////////////////////////////////////////

package org.checkstyle.suppressionxpathfilter;

import java.io.File;
import java.util.Arrays;
import java.util.List;

import org.junit.Test;

import com.puppycrawl.tools.checkstyle.DefaultConfiguration;
import com.puppycrawl.tools.checkstyle.checks.coding.AvoidInlineConditionalsCheck;

public class XpathRegressionAvoidInlineConditionalsTest extends AbstractXpathTestSupport {

@Override
protected String getCheckName() {
return AvoidInlineConditionalsCheck.class.getSimpleName();
}

@Test
public void testInlineConditionals() throws Exception {
final File fileToProcess =
new File(getPath("SuppressionXpathRegressionAvoidInlineConditionals.java"));

final DefaultConfiguration moduleConfig =
createModuleConfig(AvoidInlineConditionalsCheck.class);

final String[] expectedViolation = {
"5:50: " + getCheckMessage(AvoidInlineConditionalsCheck.class,
AvoidInlineConditionalsCheck.MSG_KEY),
};

final List<String> expectedXpathQueries = Arrays.asList(
"/CLASS_DEF[./IDENT[@text='SuppressionXpathRegressionAvoidInlineConditionals']]"
+ "/OBJBLOCK/METHOD_DEF[./IDENT[@text='substring']]/SLIST"
+ "/VARIABLE_DEF[./IDENT[@text='b']]/ASSIGN/EXPR",
"/CLASS_DEF[./IDENT[@text='SuppressionXpathRegressionAvoidInlineConditionals']]"
+ "/OBJBLOCK/METHOD_DEF[./IDENT[@text='substring']]/SLIST"
+ "/VARIABLE_DEF[./IDENT[@text='b']]/ASSIGN/EXPR/QUESTION"
);

runVerifications(moduleConfig, fileToProcess, expectedViolation,
expectedXpathQueries);
}

}
@@ -0,0 +1,8 @@
package org.checkstyle.suppressionxpathfilter.avoidinlineconditionals;

public class SuppressionXpathRegressionAvoidInlineConditionals {
String substring(String a) {
String b = (a == null || a.length() < 1) ? null : a.substring(1); // warn
return b;
}
}

0 comments on commit a24e112

Please sign in to comment.