Have RefasterTemplateCollection verify template test class names

PicnicSupermarket · Sep 18, 2022 · 32a4716 · 32a4716
1 parent b30562b
commit 32a4716
Show file tree

Hide file tree

Showing 7 changed files with 77 additions and 3 deletions.
diff --git a/...sources/tech/picnic/errorprone/refastertemplates/AssertJEnumerableTemplatesTestInput.java b/...sources/tech/picnic/errorprone/refastertemplates/AssertJEnumerableTemplatesTestInput.java
@@ -7,7 +7,7 @@
 import org.assertj.core.api.EnumerableAssert;
 import tech.picnic.errorprone.refaster.test.RefasterTemplateTestCase;
 
-final class AssertJEnumableTemplatesTest implements RefasterTemplateTestCase {
+final class AssertJEnumerableTemplatesTest implements RefasterTemplateTestCase {
   @Override
   public ImmutableSet<?> elidedTypesAndStaticImports() {
     return ImmutableSet.of(Iterables.class);

diff --git a/...ources/tech/picnic/errorprone/refastertemplates/AssertJEnumerableTemplatesTestOutput.java b/...ources/tech/picnic/errorprone/refastertemplates/AssertJEnumerableTemplatesTestOutput.java
@@ -7,7 +7,7 @@
 import org.assertj.core.api.EnumerableAssert;
 import tech.picnic.errorprone.refaster.test.RefasterTemplateTestCase;
 
-final class AssertJEnumableTemplatesTest implements RefasterTemplateTestCase {
+final class AssertJEnumerableTemplatesTest implements RefasterTemplateTestCase {
   @Override
   public ImmutableSet<?> elidedTypesAndStaticImports() {
     return ImmutableSet.of(Iterables.class);

diff --git a/...upport/src/main/java/tech/picnic/errorprone/refaster/test/RefasterTemplateCollection.java b/...upport/src/main/java/tech/picnic/errorprone/refaster/test/RefasterTemplateCollection.java
@@ -29,6 +29,7 @@
 import com.google.errorprone.fixes.SuggestedFix;
 import com.google.errorprone.matchers.Description;
 import com.google.errorprone.util.ASTHelpers;
+import com.sun.source.tree.ClassTree;
 import com.sun.source.tree.CompilationUnitTree;
 import com.sun.source.tree.LineMap;
 import com.sun.source.tree.MethodTree;
@@ -68,6 +69,7 @@ public final class RefasterTemplateCollection extends BugChecker
       "RefasterTemplateCollection:TemplateCollection";
   private static final String TEST_METHOD_NAME_PREFIX = "test";
 
+  private final String templateCollectionUnderTest;
   private final ImmutableSortedSet<String> templatesUnderTest;
   private final Refaster delegate;
 
@@ -77,7 +79,7 @@ public final class RefasterTemplateCollection extends BugChecker
    * @param flags Any provided command line flags.
    */
   public RefasterTemplateCollection(ErrorProneFlags flags) {
-    String templateCollectionUnderTest = getTemplateCollectionUnderTest(flags);
+    templateCollectionUnderTest = getTemplateCollectionUnderTest(flags);
     delegate = createRefasterChecker(templateCollectionUnderTest);
     templatesUnderTest = getTemplatesUnderTest(templateCollectionUnderTest);
   }
@@ -131,6 +133,8 @@ public static void validate(Class<?> clazz) {
 
   @Override
   public Description matchCompilationUnit(CompilationUnitTree tree, VisitorState state) {
+    reportIncorrectClassName(tree, state);
+
     List<Description> matches = new ArrayList<>();
     delegate.matchCompilationUnit(
         tree,
@@ -147,6 +151,30 @@ public Description matchCompilationUnit(CompilationUnitTree tree, VisitorState s
     return Description.NO_MATCH;
   }
 
+  private void reportIncorrectClassName(CompilationUnitTree tree, VisitorState state) {
+    String expectedClassName = templateCollectionUnderTest + "Test";
+
+    for (Tree typeDeclaration : tree.getTypeDecls()) {
+      if (typeDeclaration instanceof ClassTree) {
+        if (!((ClassTree) typeDeclaration).getSimpleName().contentEquals(expectedClassName)) {
+          state.reportMatch(
+              describeMatch(
+                  typeDeclaration,
+                  SuggestedFix.prefixWith(
+                      typeDeclaration,
+                      String.format(
+                          "/* ERROR: Class should be named `%s`. */\n", expectedClassName))));
+        }
+      } else {
+        state.reportMatch(
+            describeMatch(
+                typeDeclaration,
+                SuggestedFix.prefixWith(
+                    typeDeclaration, "/* ERROR: Unexpected declaration. */\n")));
+      }
+    }
+  }
+
   private static ImmutableRangeMap<Integer, String> indexTemplateMatches(
       List<Description> matches, EndPosTable endPositions) {
     ImmutableRangeMap.Builder<Integer, String> templateMatches = ImmutableRangeMap.builder();

diff --git a/...upport/src/test/java/tech/picnic/errorprone/refaster/test/MisnamedTestClassTemplates.java b/...upport/src/test/java/tech/picnic/errorprone/refaster/test/MisnamedTestClassTemplates.java
@@ -0,0 +1,21 @@
+package tech.picnic.errorprone.refaster.test;
+
+import com.google.errorprone.refaster.annotation.AfterTemplate;
+import com.google.errorprone.refaster.annotation.BeforeTemplate;
+
+/** Refaster template collection to validate reporting of a misnamed test class. */
+final class MisnamedTestClassTemplates {
+  private MisnamedTestClassTemplates() {}
+
+  static final class StringIsEmpty {
+    @BeforeTemplate
+    boolean before(String string) {
+      return string.equals("");
+    }
+
+    @AfterTemplate
+    boolean after(String string) {
+      return string.isEmpty();
+    }
+  }
+}
diff --git a/...rt/src/test/java/tech/picnic/errorprone/refaster/test/RefasterTemplateCollectionTest.java b/...rt/src/test/java/tech/picnic/errorprone/refaster/test/RefasterTemplateCollectionTest.java
@@ -19,6 +19,7 @@ final class RefasterTemplateCollectionTest {
       classes = {
         MatchInWrongMethodTemplates.class,
         MethodWithoutPrefixTemplates.class,
+        MisnamedTestClassTemplates.class,
         MissingTestAndWrongTestTemplates.class,
         PartialTestMatchTemplates.class,
         TemplateWithoutTestTemplates.class,

diff --git a/...t/resources/tech/picnic/errorprone/refaster/test/MisnamedTestClassTemplatesTestInput.java b/...t/resources/tech/picnic/errorprone/refaster/test/MisnamedTestClassTemplatesTestInput.java
@@ -0,0 +1,11 @@
+package tech.picnic.errorprone.refaster.test;
+
+/** Code to test the Refaster templates from {@link MisnamedTestClassTemplates}. */
+final class IncorrectNameTemplatesTest implements RefasterTemplateTestCase {
+  boolean testStringIsEmpty() {
+    return "foo".equals("");
+  }
+}
+
+// This is a comment to appease CheckStyle.
+;
diff --git a/.../resources/tech/picnic/errorprone/refaster/test/MisnamedTestClassTemplatesTestOutput.java b/.../resources/tech/picnic/errorprone/refaster/test/MisnamedTestClassTemplatesTestOutput.java
@@ -0,0 +1,13 @@
+package tech.picnic.errorprone.refaster.test;
+
+/** Code to test the Refaster templates from {@link MisnamedTestClassTemplates}. */
+/* ERROR: Class should be named `MisnamedTestClassTemplatesTest`. */
+final class IncorrectNameTemplatesTest implements RefasterTemplateTestCase {
+  boolean testStringIsEmpty() {
+    return "foo".isEmpty();
+  }
+}
+
+// This is a comment to appease CheckStyle.
+/* ERROR: Unexpected declaration. */
+;