fix: check the property targerModult not null (#435)

STAMP-project · May 16, 2018 · 9022da6 · 9022da6
1 parent 7dbcf35
commit 9022da6
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/dspot/src/main/java/fr/inria/stamp/diff/SelectorOnDiff.java b/dspot/src/main/java/fr/inria/stamp/diff/SelectorOnDiff.java
@@ -177,13 +177,15 @@ private Set<CtMethod> getTestMethodsOfModifiedTestClasses(Set<String> modifiedJa
         return modifiedJavaFiles.stream()
                 .filter(presentInBothVersion)
                 .map(modifiedJavaFile -> {
-                    final String directoryPath = (this.configuration.getProperty("targetModule") +
-                            (this.configuration.getProperty("targetModule").isEmpty() ? "" : "/") +
-                            this.configuration.getRelativeSourceCodeDir());
+                    final String directoryPath =
+                            (this.configuration.getProperty("targetModule") != null ?
+                                    this.configuration.getProperty("targetModule") + "/" : ""
+                            ) + this.configuration.getRelativeSourceCodeDir();
                     final String qualifiedNameWithExtension = modifiedJavaFile.substring(directoryPath.length() + 2).replaceAll("/", ".");
                     return qualifiedNameWithExtension.substring(0, qualifiedNameWithExtension.length() - ".java".length());
                 })
                 .map(fullQualifiedName -> this.factory.Class().get(fullQualifiedName))
+                .filter(Objects::nonNull)
                 .filter(potentialTestClass -> potentialTestClass.getMethods().stream().anyMatch(AmplificationChecker::isTest))
                 .flatMap(testClass -> testClass.getMethods().stream().filter(AmplificationChecker::isTest))
                 .collect(Collectors.toSet());