[ https://issues.apache.org/jira/browse/SUREFIRE-2179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17737678#comment-17737678 ]
ASF GitHub Bot commented on SUREFIRE-2179: ------------------------------------------ kwin commented on code in PR #667: URL: https://github.com/apache/maven-surefire/pull/667#discussion_r1243772435 ########## maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java: ########## @@ -281,6 +285,21 @@ public abstract class AbstractSurefireMojo extends AbstractMojo implements Suref @Parameter(property = "maven.test.additionalClasspath") private String[] additionalClasspathElements; + /** + * Additional Maven dependencies to be used in the test execution classpath. + * Each element supports the parametrization like documented in <a href="https://maven.apache.org/pom.html#dependencies">POM Reference: Dependencies</a>. + * <p> + * Those dependencies are automatically collected (i.e. have their full dependency tree calculated) and then all underlying artifacts are resolved from the repository (including their transitive dependencies). + * Afterwards the resolved artifacts are filtered to only contain {@code compile} and {@code runtime} scoped ones and appended to the test execution classpath + * (after the ones from {@link #additionalClasspathElements}). + * <p> + * The dependency management from the project is not taken into account. + * + * @since 3.2 + */ + @Parameter(property = "maven.test.additionalClasspathDependencies") + private Dependency[] additionalClasspathDependencies; Review Comment: This is just to be more in line with existing parameters. But I can switch to `List`. The `null` check is not necessary for either array nor Collection, as plexus.inject will inject an empty array. > additionalClasspathElements should support Maven coordinates > ------------------------------------------------------------ > > Key: SUREFIRE-2179 > URL: https://issues.apache.org/jira/browse/SUREFIRE-2179 > Project: Maven Surefire > Issue Type: Improvement > Components: classloading > Affects Versions: 3.1.2 > Reporter: Konrad Windszus > Assignee: Konrad Windszus > Priority: Major > > Currently the parameter {{additionalClasspathElements}} > (https://maven.apache.org/surefire/maven-surefire-plugin/test-mojo.html#additionalclasspathelements) > only supports file paths. That usually requires to add an additional step to > first download the necessary artifact to a temporary folder with another > Mojo. In addition {{additionalClasspathElements}} only support full paths to > JARs but no wildcards which makes configuration very verbose. > For these reasons there should be an additional parameter supporting Maven > coordinates which are then resolved automatically (even transitively) and > added to the test execution classpath. -- This message was sent by Atlassian Jira (v8.20.10#820010)