Issue #451: add multiple MatchXpath to checkstyle config

Author: Calixte

config/checkstyle-checks.xml

@@ -366,6 +366,57 @@
             <property name="ignoreCase" value="true"/>
         </module>
         <module name="InnerAssignment"/>
+        <module name="MatchXpath">
+            <property name="query" value="//CLASS_DEF[@text!='Checker' and @text!='Main']
+                //LITERAL_CATCH//METHOD_CALL[.//IDENT[@text = 'printStackTrace']]/.."/>
+            <message key="matchxpath.match" value="Avoid using 'printStackTrace'."/>
+        </module>
+        <module name="MatchXpath">
+            <property name="query" value="//METHOD_DEF/MODIFIERS//
+                ANNOTATION[./IDENT[@text='Test']]/ANNOTATION_MEMBER_VALUE_PAIR
+                [./IDENT[@text='expected']]"/>
+            <message key="matchxpath.match" value="Avoid using 'expected' attribute in Test annotation."/>
+        </module>
+        <module name="MatchXpath">
+            <property name="query" value="//ANNOTATION[./IDENT[@text='Issue']]"/>
+            <message key="matchxpath.match" value="Avoid using @Issue annotation."/>
+        </module>
+        <module name="MatchXpath">
+            <property name="id" value="noUsageOfGetFileContentsMethod"/>
+            <property name="query" value="//CLASS_DEF[
+               ./EXTENDS_CLAUSE[./IDENT[@text='AbstractCheck'
+                                     or @text='AbstractJavadocCheck'
+                                     or @text='AbstractSuperCheck'
+                                     or @text='AbstractClassCouplingCheck'
+                                     or @text='AbstractNameCheck'
+                                     or @text='AbstractParenPadCheck']]
+               and .//METHOD_CALL[.//IDENT[@text = 'getFileContents']]]"/>
+            <message key="matchxpath.match" value="Avoid using getFileContents method in
+                                            AbstractCheck subclasses."/>
+        </module>
+        <!-- until https://github.com/checkstyle/checkstyle/issues/5234 -->
+        <module name="MatchXpath">
+            <property name="id" value="MatchXPathBranchContains"/>
+            <property name="query" value="//METHOD_CALL//DOT[.//IDENT[@text = 'branchContains']]"/>
+            <message key="matchxpath.match"
+               value="Avoid using deprecated method 'DetailAst.branchContains()'."/>
+        </module>
+        <module name="MatchXpath">
+            <property name="query" value="//COMMENT_CONTENT[contains(@text, '@noinspection')
+                      and not(contains(@text, '@noinspectionreason'))]"/>
+            <message key="matchxpath.match"
+               value="'@noinspection' Javadoc tags should be accompanied by a
+                     '@noinspectionreason' tag, explaining why we suppressed inspection."/>
+        </module>
+        <module name="MatchXpath">
+            <property name="id" value="exampleTestsExtendAbstractExamplesModuleTestSupport"/>
+            <property name="query"
+                value="//CLASS_DEF[./IDENT[ends-with(@text, 'ExamplesTest')]
+                and ./MODIFIERS/LITERAL_PUBLIC
+                and ./EXTENDS_CLAUSE/IDENT[not(@text='AbstractExamplesModuleTestSupport')]]"/>
+            <message key="matchxpath.match"
+               value="All example test classes should extend AbstractExamplesModuleTestSupport."/>
+        </module>
         <module name="MissingCtor">
             <!--
               we will not use that fanatic validation, extra code is not good