Merge pull request #17099 from stuartwdouglas/17086

Jacoco does not report on other modules
quarkusio · May 10, 2021 · 0f2659a · 0f2659a
2 parents fdfe4e6 + 9d987ab
commit 0f2659a
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/...amework/jacoco/deployment/src/main/java/io/quarkus/jacoco/deployment/JacocoProcessor.java b/...amework/jacoco/deployment/src/main/java/io/quarkus/jacoco/deployment/JacocoProcessor.java
@@ -14,7 +14,6 @@
 import org.codehaus.plexus.util.StringUtils;
 import org.jacoco.core.instr.Instrumenter;
 import org.jacoco.core.runtime.OfflineInstrumentationAccessGenerator;
-import org.jacoco.report.MultiSourceFileLocator;
 import org.jboss.jandex.ClassInfo;
 
 import io.quarkus.bootstrap.model.AppArtifactKey;
@@ -103,11 +102,10 @@ public byte[] apply(String className, byte[] bytes) {
             info.classFiles = classes;
 
             Set<String> sources = new HashSet<>();
-            MultiSourceFileLocator sourceFileLocator = new MultiSourceFileLocator(4);
             if (BuildToolHelper.isMavenProject(targetdir.toPath())) {
                 Set<AppArtifactKey> runtimeDeps = new HashSet<>();
                 for (AppDependency i : curateOutcomeBuildItem.getEffectiveModel().getUserDependencies()) {
-                    runtimeDeps.add(i.getArtifact().getKey());
+                    runtimeDeps.add(new AppArtifactKey(i.getArtifact().getGroupId(), i.getArtifact().getArtifactId()));
                 }
                 LocalProject project = LocalProject.loadWorkspace(targetdir.toPath());
                 runtimeDeps.add(project.getKey());