From 84b7836641d5ca83f3e5d4af4b8867c4bef734e4 Mon Sep 17 00:00:00 2001
From: Nik Everett <nik9000@gmail.com>
Date: Wed, 29 Jun 2022 09:52:49 -0400
Subject: [PATCH] Checkstyle: don't require javadoc on test methods

Test methods tend to have descriptive names for the cases that they are
testing. Sometimes they require javadoc, for sure, but not often enough
to add a little "WEAK WARNING" marker on every single test method in my
IDE.
---
 .../src/main/resources/checkstyle_ide_fragment.xml              | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/build-tools-internal/src/main/resources/checkstyle_ide_fragment.xml b/build-tools-internal/src/main/resources/checkstyle_ide_fragment.xml
index db6b025dea2dc..c4870cbf6e5c1 100644
--- a/build-tools-internal/src/main/resources/checkstyle_ide_fragment.xml
+++ b/build-tools-internal/src/main/resources/checkstyle_ide_fragment.xml
@@ -16,7 +16,7 @@
         <!-- Exclude short methods from this check - we don't want to have to document getters -->
         <property name="minLineCount" value="5" />
         <property name="allowedAnnotations" value="Override,Before,BeforeClass,After,AfterClass,Inject,TaskAction" />
-        <property name="ignoreMethodNamesRegex" value="^main$"/>
+        <property name="ignoreMethodNamesRegex" value="^(main|test.*)$"/>
         <message key="javadoc.missing" value="Public methods should be documented." />
     </module>