Enforce new naming convention "InputXpath{Checkname}Xxxx.java" in IT area

[MANISH-K-07]

Follow-up of #6981

Until now, XpathRegressionTest validated the pattern SuppressionXpathRegressionXxxx.java for input filenames
All our existing files are named that way, so we bypass them until this issue is fixed.

The pattern has been updated to InputXpath{Checkname}Xxxx.java as part of PR #14713
All existing modules have been suppressed from validation.

As part of this issue, one module needs to be picked and all input files need to be renamed. Note that the xpath queries need to be verified in respective XpathRegression{Checkname}Test after filename change. Also, the set RENAME_INPUT_XPATH needs to be updated by removing the updated module to re-enable pattern validation.

The list of suppressions can be found at :

checkstyle/src/test/java/com/puppycrawl/tools/checkstyle/internal/XpathRegressionTest.java

Lines 121 to 261 in cd1afd9

	private static final Set<String> RENAME_INPUT_XPATH = Set.of(
	"AbbreviationAsWordInName",
	"AbstractClassName",
	"AnnotationLocation",
	"AnnotationOnSameLine",
	"AnnotationUseStyle",
	"AnonInnerLength",
	"ArrayTrailingComma",
	"ArrayTypeStyle",
	"AvoidDoubleBraceInitialization",
	"AvoidEscapedUnicodeCharacters",
	"AvoidInlineConditionals",
	"AvoidNestedBlocks",
	"AvoidNoArgumentSuperConstructorCall",
	"AvoidStarImport",
	"AvoidStaticImport",
	"BooleanExpressionComplexity",
	"CatchParameterName",
	"ClassMemberImpliedModifier",
	"CommentsIndentation",
	"ConstantName",
	"CovariantEquals",
	"CustomImportOrder",
	"CyclomaticComplexity",
	"DeclarationOrder",
	"DefaultComesLast",
	"EmptyBlock",
	"EmptyCatchBlock",
	"EmptyForInitializerPad",
	"EmptyForIteratorPad",
	"EmptyLineSeparator",
	"EmptyStatement",
	"EqualsAvoidNull",
	"EqualsHashCode",
	"ExplicitInitialization",
	"FallThrough",
	"FinalClass",
	"FinalLocalVariable",
	"FinalParameters",
	"GenericWhitespace",
	"HiddenField",
	"IllegalCatch",
	"IllegalImport",
	"IllegalThrows",
	"IllegalToken",
	"IllegalTokenText",
	"IllegalType",
	"ImportControl",
	"ImportOrder",
	"Indentation",
	"InnerAssignment",
	"InnerTypeLast",
	"InterfaceIsType",
	"InterfaceMemberImpliedModifier",
	"InvalidJavadocPosition",
	"JavadocContentLocation",
	"JavadocMethod",
	"JavadocType",
	"JavadocVariable",
	"JavaNCSS",
	"LambdaBodyLength",
	"LambdaParameterName",
	"LeftCurly",
	"LocalFinalVariableName",
	"MagicNumber",
	"MatchXpath",
	"MemberName",
	"MethodCount",
	"MethodLength",
	"MethodName",
	"MethodParamPad",
	"MethodTypeParameterName",
	"MissingCtor",
	"MissingJavadocMethod",
	"MissingJavadocPackage",
	"MissingJavadocType",
	"MissingOverride",
	"MissingSwitchDefault",
	"ModifierOrder",
	"MultipleStringLiterals",
	"MultipleVariableDeclarations",
	"NeedBraces",
	"NestedForDepth",
	"NestedIfDepth",
	"NestedTryDepth",
	"NoArrayTrailingComma",
	"NoClone",
	"NoEnumTrailingComma",
	"NoFinalizer",
	"NoLineWrap",
	"NoWhitespaceAfter",
	"NoWhitespaceBefore",
	"NoWhitespaceBeforeCaseDefaultColon",
	"NPathComplexity",
	"OneStatementPerLine",
	"OneTopLevelClass",
	"OperatorWrap",
	"OuterTypeFilename",
	"OuterTypeNumber",
	"OverloadMethodsDeclarationOrder",
	"PackageName",
	"ParameterAssignment",
	"ParameterName",
	"ParameterNumber",
	"ParenPad",
	"RedundantImport",
	"RequireThis",
	"ReturnCount",
	"RightCurly",
	"SimplifyBooleanExpression",
	"SimplifyBooleanReturn",
	"SingleSpaceSeparator",
	"StaticVariableName",
	"StringLiteralEquality",
	"SuperClone",
	"ThrowsCount",
	"TodoComment",
	"TrailingComment",
	"TypecastParenPad",
	"TypeName",
	"UncommentedMain",
	"UnnecessaryParentheses",
	"UnnecessarySemicolonAfterOuterTypeDeclaration",
	"UnnecessarySemicolonAfterTypeMemberDeclaration",
	"UnnecessarySemicolonInEnumeration",
	"UnnecessarySemicolonInTryWithResources",
	"UnusedImports",
	"UnusedLocalVariable",
	"UpperEll",
	"VariableDeclarationUsageDistance",
	"VisibilityModifier",
	"WhitespaceAfter",
	"WhitespaceAround",
	"IllegalIdentifierName",
	"IllegalInstantiation",
	"PackageAnnotation",
	"PackageDeclaration",
	"PatternVariableName",
	"RecordComponentName",
	"RecordComponentNumber",
	"RecordTypeParameterName");

Example of update : e70b8b5

[Kexin2000]

Because #14719 fixes the failing build in master, list in the first post does not have a MethodTypeParameterName. I am on that :)

[MANISH-K-07]

I am on VisibilityModifier

[romani]

@MANISH-K-07, I highly recommend to do fixes by packages or by packs of 20 files. to fnish this update quicker and not making a an epic from it.

[romani]

@MANISH-K-07 , please update issue description to reference list of suppression and do udpates based on it.
checkboxes will be a problem to maintain, as fix is easy

[Lmh-java]

I am on RecordTypeParameterName

[romani]

@Lmh-java, we appreciate your help with easy issues, you are always welcome, but I recommend you next month to focus on more complicated issues to increase your chances to be selected to gsoc.

Ohm there are more participants, same suggestion for others.

[MANISH-K-07]

@romani , like you said, let's not make an epic of this issue. I'll close this with multiple PRs at a time in 2 batches like @rnveach suggested at #14725 (comment)

[romani]

ok, lets let only @MANISH-K-07 to work on this issue.