Use test classes and resources dirs for artifacts with tests classifier

aloubyansky · Oct 13, 2021 · 4f60d2c · 4f60d2c
1 parent d320c33
commit 4f60d2c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 16 deletions.
diff --git a/core/deployment/src/main/java/io/quarkus/deployment/index/ApplicationArchiveBuildStep.java b/core/deployment/src/main/java/io/quarkus/deployment/index/ApplicationArchiveBuildStep.java
@@ -167,7 +167,6 @@ public void addIndexDependencyPaths(List<IndexDependencyBuildItem> indexDependen
                         GACTV.TYPE_JAR);
                 final ResolvedDependency artifact = userMap.get(key);
                 if (artifact == null) {
-                    userMap.keySet().forEach(k -> System.out.println(" - " + k.getClass().getSimpleName() + " " + k));
                     throw new RuntimeException(
                             "Could not resolve artifact " + key + " among the runtime dependencies of the application");
                 }

diff --git a/...c/main/java/io/quarkus/bootstrap/resolver/maven/DeploymentInjectingDependencyVisitor.java b/...c/main/java/io/quarkus/bootstrap/resolver/maven/DeploymentInjectingDependencyVisitor.java
@@ -640,21 +640,12 @@ public static ResolvedDependencyBuilder toAppArtifact(Artifact artifact, Workspa
     public static PathCollection getResolvedPaths(Artifact artifact, WorkspaceModule module, boolean preferWorkspacePaths) {
         if (preferWorkspacePaths && module != null) {
             final PathList.Builder pathBuilder = PathList.builder();
-            for (ProcessedSources src : module.getMainSources()) {
-                if (src.getDestinationDir().exists()) {
-                    final Path p = src.getDestinationDir().toPath();
-                    if (!pathBuilder.contains(p)) {
-                        pathBuilder.add(p);
-                    }
-                }
-            }
-            for (ProcessedSources src : module.getMainResources()) {
-                if (src.getDestinationDir().exists()) {
-                    final Path p = src.getDestinationDir().toPath();
-                    if (!pathBuilder.contains(p)) {
-                        pathBuilder.add(p);
-                    }
-                }
+            if ("tests".equals(artifact.getClassifier())) {
+                collectResolvedPaths(pathBuilder, module.getTestSources());
+                collectResolvedPaths(pathBuilder, module.getTestResources());
+            } else {
+                collectResolvedPaths(pathBuilder, module.getMainSources());
+                collectResolvedPaths(pathBuilder, module.getMainResources());
             }
             if (!pathBuilder.isEmpty()) {
                 return pathBuilder.build();
@@ -663,6 +654,17 @@ public static PathCollection getResolvedPaths(Artifact artifact, WorkspaceModule
         return artifact.getFile() == null ? PathList.empty() : PathList.of(artifact.getFile().toPath());
     }
 
+    private static void collectResolvedPaths(final PathList.Builder pathBuilder, Collection<ProcessedSources> srcs) {
+        for (ProcessedSources src : srcs) {
+            if (src.getDestinationDir().exists()) {
+                final Path p = src.getDestinationDir().toPath();
+                if (!pathBuilder.contains(p)) {
+                    pathBuilder.add(p);
+                }
+            }
+        }
+    }
+
     private static String toGactv(Artifact a) {
         return a.getGroupId() + ":" + a.getArtifactId() + ":" + a.getClassifier() + ":" + a.getExtension() + ":"
                 + a.getVersion();