we have rule to have 3 tests in XpathRegressionXxxxxxXxxxxxxTest
But as it is not automated we always loose time on this.
examples of missed:
#18657 (comment)
#18312 (comment)
Find a way to make junit test or Checkstyle Xpath test to make sure that all
under https://github.com/checkstyle/checkstyle/tree/master/src/it/java/org/checkstyle/suppressionxpathfilter
should 3 methods with @Test and in error message we should mention that we need 3 test that generates xpath different by nodes (structure).
Demand for 3 tests were added at #19074
Goal of this issue is to remove suppressions at
|
<!-- until https://github.com/checkstyle/checkstyle/issues/19064 --> |
Each file should be removed in separate PR
ATTENTION: all test methods should generate Xpath with different structure, so make a diversity by placing violation in inner classes, more nested code blocks, in anonymous classes, in .....
we have rule to have 3 tests in XpathRegressionXxxxxxXxxxxxxTest
But as it is not automated we always loose time on this.
examples of missed:
#18657 (comment)
#18312 (comment)
Find a way to make junit test or Checkstyle Xpath test to make sure that all
under https://github.com/checkstyle/checkstyle/tree/master/src/it/java/org/checkstyle/suppressionxpathfilter
should 3 methods with
@Testand in error message we should mention that we need 3 test that generates xpath different by nodes (structure).Demand for 3 tests were added at #19074
Goal of this issue is to remove suppressions at
checkstyle/config/checkstyle-non-main-files-suppressions.xml
Line 413 in 4fdd0fc
Each file should be removed in separate PR
ATTENTION: all test methods should generate Xpath with different structure, so make a diversity by placing violation in inner classes, more nested code blocks, in anonymous classes, in .....