diff options
Diffstat (limited to 'src/test/java/com/puppycrawl/tools/checkstyle/internal/AllChecksTest.java')
-rw-r--r-- | src/test/java/com/puppycrawl/tools/checkstyle/internal/AllChecksTest.java | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/test/java/com/puppycrawl/tools/checkstyle/internal/AllChecksTest.java b/src/test/java/com/puppycrawl/tools/checkstyle/internal/AllChecksTest.java index 38c24c739..f82155485 100644 --- a/src/test/java/com/puppycrawl/tools/checkstyle/internal/AllChecksTest.java +++ b/src/test/java/com/puppycrawl/tools/checkstyle/internal/AllChecksTest.java @@ -330,8 +330,6 @@ public class AllChecksTest extends AbstractModuleTestSupport { public void testAllModulesAreReferencedInConfigFile() throws Exception { final Set<String> modulesReferencedInConfig = CheckUtil.getConfigCheckStyleModules(); final Set<String> moduleNames = CheckUtil.getSimpleNames(CheckUtil.getCheckstyleModules()); - //Issue: https://github.com/checkstyle/checkstyle/issues/4421 - moduleNames.remove("SuppressionXpathFilter"); moduleNames.stream().filter(check -> !modulesReferencedInConfig.contains(check)) .forEach(check -> { @@ -428,8 +426,6 @@ public class AllChecksTest extends AbstractModuleTestSupport { // these are documented on non-'config_' pages checkstyleModulesNames.remove("TreeWalker"); checkstyleModulesNames.remove("Checker"); - //Issue: https://github.com/checkstyle/checkstyle/issues/4421 - checkstyleModulesNames.remove("SuppressionXpathFilter"); checkstyleModulesNames.stream() .filter(moduleName -> !modulesNamesWhichHaveXdocs.contains(moduleName)) @@ -445,8 +441,6 @@ public class AllChecksTest extends AbstractModuleTestSupport { public void testAllCheckstyleModulesInCheckstyleConfig() throws Exception { final Set<String> configChecks = CheckUtil.getConfigCheckStyleModules(); final Set<String> moduleNames = CheckUtil.getSimpleNames(CheckUtil.getCheckstyleModules()); - //Issue: https://github.com/checkstyle/checkstyle/issues/4421 - moduleNames.remove("SuppressionXpathFilter"); for (String moduleName : moduleNames) { Assert.assertTrue("checkstyle_checks.xml is missing module: " + moduleName, @@ -548,6 +542,7 @@ public class AllChecksTest extends AbstractModuleTestSupport { * Checks that an array is a subset of other array. * @param array to check whether it is a subset. * @param arrayToCheckIn array to check in. + * @return {@code true} if all elements in {@code array} are in {@code arrayToCheckIn}. */ private static boolean isSubset(int[] array, int... arrayToCheckIn) { Arrays.sort(arrayToCheckIn); |