GEODE-8819: StressNewTestHelper should handle non org.apache.geode ch…

…anged files (apache#5885)
ringles · Jan 8, 2021 · b7774f4 · b7774f4
1 parent 5f2a1fa
commit b7774f4
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/geode-junit/src/main/java/org/apache/geode/test/util/StressNewTestHelper.java b/geode-junit/src/main/java/org/apache/geode/test/util/StressNewTestHelper.java
@@ -109,7 +109,13 @@ public void add(String javaFile) {
     TestClassInfo testClassInfo = createTestClassInfo(javaFile);
     List<TestClassInfo> extenders = whatExtends(testClassInfo);
 
-    if (!scanResult.getClassInfo(testClassInfo.className).isAbstract()) {
+    ClassInfo classInfo = scanResult.getClassInfo(testClassInfo.className);
+    // This is a possibility for non org.apache.geode files
+    if (classInfo == null) {
+      return;
+    }
+
+    if (!classInfo.isAbstract()) {
       extenders.add(testClassInfo);
     }
 

diff --git a/geode-junit/src/test/java/org/apache/geode/test/util/WhatExtendsJUnitTest.java b/geode-junit/src/test/java/org/apache/geode/test/util/WhatExtendsJUnitTest.java
@@ -84,6 +84,12 @@ public void ignoreAcceptanceTestSourcesForNow() {
     assertThat(scanner.buildGradleCommand()).isEqualTo("-PtestCount=0");
   }
 
+  @Test
+  public void ignoreNonGeodeClasses() {
+    scanner.add("foo/src/test/java/org/example/Foo.java");
+    assertThat(scanner.buildGradleCommand()).isEqualTo("-PtestCount=0");
+  }
+
   private String getClassLocation(Class<?> clazz) {
     String codeSource = clazz.getProtectionDomain().getCodeSource().getLocation().getFile();
     String classFile = clazz.getName().replace(".", "/");