improve test class detection in nullability rules

Author: wakingrufus

archrules-nullability/src/archRules/java/com/netflix/nebula/archrules/nullability/HaveNoTests.java

@@ -11,6 +11,10 @@ public HaveNoTests() {
     @Override
     public boolean test(JavaClass javaClass) {
         return javaClass.getMembers().stream()
-                .noneMatch(it -> it.isAnnotatedWith("org.junit.jupiter.api.Test"));
+                .noneMatch(it -> it.isMetaAnnotatedWith("org.junit.platform.commons.annotation.Testable"));
+    }
+
+    public static HaveNoTests haveNoTests() {
+        return new HaveNoTests();
     }
 }

archrules-nullability/src/archRules/java/com/netflix/nebula/archrules/nullability/NebulaNullabilityArchRules.java

@@ -10,6 +10,7 @@
 import java.util.HashMap;
 import java.util.Map;
 
+import static com.netflix.nebula.archrules.nullability.HaveNoTests.haveNoTests;
 import static com.tngtech.archunit.lang.conditions.ArchConditions.fullyQualifiedName;
 
 public class NebulaNullabilityArchRules implements ArchRulesService {
@@ -18,7 +19,7 @@ public class NebulaNullabilityArchRules implements ArchRulesService {
             .areTopLevelClasses()
             .and().arePublic()
             .and().containAnyMembersThat(HasModifiers.Predicates.modifier(JavaModifier.PUBLIC))
-            .and(new HaveNoTests())
+            .and(haveNoTests())
             .and().areNotAnnotatedWith("kotlin.Metadata")
             .should().beAnnotatedWith("org.jspecify.annotations.NullMarked")
             .allowEmptyShould(true)

archrules-nullability/src/archRulesTest/java/com/netflix/nebula/archrules/nullability/HaveNoTestsTest.java

@@ -0,0 +1,54 @@
+package com.netflix.nebula.archrules.nullability;
+
+import com.tngtech.archunit.core.domain.JavaClass;
+import com.tngtech.archunit.core.importer.ClassFileImporter;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.ValueSource;
+
+import static com.netflix.nebula.archrules.nullability.HaveNoTests.haveNoTests;
+import static org.assertj.core.api.Assertions.assertThat;
+
+class HaveNoTestsTest {
+    @Test
+    public void test_jupiter_test() {
+        JavaClass codeToTest = new ClassFileImporter().importClass(UsesJupiterTest.class);
+        HaveNoTests instance = haveNoTests();
+        assertThat(instance.test(codeToTest)).isFalse();
+    }
+
+    @Test
+    public void test_jupiter_parameterized_test() {
+        JavaClass codeToTest = new ClassFileImporter().importClass(UsesJupiterParameterizedTest.class);
+        HaveNoTests instance = haveNoTests();
+        assertThat(instance.test(codeToTest)).isFalse();
+    }
+
+    @Test
+    public void test_nothing() {
+        JavaClass codeToTest = new ClassFileImporter().importClass(UsesNothing.class);
+        HaveNoTests instance = haveNoTests();
+        assertThat(instance.test(codeToTest)).isTrue();
+    }
+
+    static class UsesJupiterTest {
+        @Test
+        void test() {
+
+        }
+    }
+
+    static class UsesJupiterParameterizedTest {
+        @ParameterizedTest
+        @ValueSource(strings = {"a", "b"})
+        void test(String param) {
+
+        }
+    }
+
+    static class UsesNothing {
+        void test() {
+
+        }
+    }
+}