(applications) Checkstyle rule to prevent use of junit4

jhipster · Oct 27, 2023 · 175939c · 175939c
1 parent 164b260
commit 175939c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/main/resources/generator/server/javatool/checkstyle/main/checkstyle.xml.mustache b/src/main/resources/generator/server/javatool/checkstyle/main/checkstyle.xml.mustache
@@ -4,5 +4,10 @@
   <module name="TreeWalker">
     <module name="RedundantImport" />
     <module name="UnusedImports" />
+    <module name="IllegalImport">
+      <property name="regexp" value="true"/>
+      <!-- Reject any org.junit import that's not also org.junit.jupiter: -->
+      <property name="illegalClasses" value="^org\.junit\.(?!jupiter\.)(?!platform\.).+"/>
+    </module>
   </module>
 </module>
diff --git a/...hipster/lite/generator/server/javatool/checkstyle/domain/CheckstyleModuleFactoryTest.java b/...hipster/lite/generator/server/javatool/checkstyle/domain/CheckstyleModuleFactoryTest.java
@@ -1,6 +1,7 @@
 package tech.jhipster.lite.generator.server.javatool.checkstyle.domain;
 
-import static tech.jhipster.lite.module.infrastructure.secondary.JHipsterModulesAssertions.*;
+import static tech.jhipster.lite.module.infrastructure.secondary.JHipsterModulesAssertions.assertThatModuleWithFiles;
+import static tech.jhipster.lite.module.infrastructure.secondary.JHipsterModulesAssertions.pomFile;
 
 import org.junit.jupiter.api.Test;
 import tech.jhipster.lite.TestFileUtils;
@@ -49,6 +50,7 @@ void shouldBuildModule() {
       )
       .and()
       .hasFile("checkstyle.xml")
+      .containing("<module name=\"IllegalImport\">")
       .containing("<module name=\"Checker\">")
       .containing("<module name=\"TreeWalker\">")
       .containing("<module name=\"UnusedImports\" />");