Add BOM-related methods to coursierapi.Fetch

Author: alexarchambault

interface/src/main/java/coursierapi/Fetch.java

@@ -9,6 +9,7 @@
 public final class Fetch {
 
     private final List<Dependency> dependencies;
+    private final List<Dependency> bomDependencies;
     private final List<Repository> repositories;
     private Cache cache;
     private Boolean mainArtifacts;
@@ -19,6 +20,7 @@ public final class Fetch {
 
     private Fetch() {
         dependencies = new ArrayList<>();
+        bomDependencies = new ArrayList<>();
         repositories = new ArrayList<>(Arrays.asList(ApiHelper.defaultRepositories()));
         cache = Cache.create();
         mainArtifacts = null;
@@ -34,6 +36,7 @@ public boolean equals(Object o) {
         if (!(o instanceof Fetch)) return false;
         Fetch fetch = (Fetch) o;
         return Objects.equals(dependencies, fetch.dependencies) &&
+                Objects.equals(bomDependencies, fetch.bomDependencies) &&
                 Objects.equals(repositories, fetch.repositories) &&
                 Objects.equals(cache, fetch.cache) &&
                 Objects.equals(mainArtifacts, fetch.mainArtifacts) &&
@@ -47,6 +50,7 @@ public boolean equals(Object o) {
     public int hashCode() {
         return Objects.hash(
                 dependencies,
+                bomDependencies,
                 repositories,
                 cache,
                 mainArtifacts,
@@ -60,6 +64,7 @@ public int hashCode() {
     public String toString() {
         return "Fetch{" +
                 "dependencies=" + dependencies +
+                ", bomDependencies=" + bomDependencies +
                 ", repositories=" + repositories +
                 ", cache=" + cache +
                 ", mainArtifacts=" + mainArtifacts +
@@ -86,6 +91,17 @@ public Fetch withDependencies(Dependency... dependencies) {
         return this;
     }
 
+    public Fetch addBomDependencies(Dependency... bomDependencies) {
+        this.bomDependencies.addAll(Arrays.asList(bomDependencies));
+        return this;
+    }
+
+    public Fetch withBomDependencies(Dependency... bomDependencies) {
+        this.bomDependencies.clear();
+        this.bomDependencies.addAll(Arrays.asList(bomDependencies));
+        return this;
+    }
+
     public Fetch addRepositories(Repository... repositories) {
         this.repositories.addAll(Arrays.asList(repositories));
         return this;
@@ -170,6 +186,10 @@ public List<Dependency> getDependencies() {
         return Collections.unmodifiableList(dependencies);
     }
 
+    public List<Dependency> getBomDependencies() {
+        return Collections.unmodifiableList(bomDependencies);
+    }
+
     public List<Repository> getRepositories() {
         return Collections.unmodifiableList(repositories);
     }

interface/src/main/scala/coursier/internal/api/ApiHelper.scala

@@ -294,6 +294,12 @@ object ApiHelper {
       .map(dependency)
       .toVector
 
+    val bomDependencies = fetch
+      .getBomDependencies
+      .asScala
+      .map(dependency)
+      .toVector
+
     val repositories = fetch
       .getRepositories
       .asScala
@@ -311,17 +317,22 @@ object ApiHelper {
 
     val params = resolutionParams(fetch.getResolutionParams)
 
-    var f = Fetch()
+    // temporarily creating a Resolve and and Artifacts manually,
+    // to work around missing BOM-related methods on Fetch
+    val resolve = Resolve()
       .withDependencies(dependencies)
+      .withBomDependencies(bomDependencies)
       .withRepositories(repositories)
       .withCache(cache0)
+      .withResolutionParams(params)
+    var artifacts = Artifacts()
       .withMainArtifacts(fetch.getMainArtifacts)
       .withClassifiers(classifiers)
-      .withFetchCache(Option(fetch.getFetchCacheIKnowWhatImDoing))
-      .withResolutionParams(params)
     if (fetch.getArtifactTypes != null)
-      f = f.withArtifactTypes(fetch.getArtifactTypes.asScala.toSet[String].map(Type(_)))
-    f
+      artifacts = artifacts.withArtifactTypes(fetch.getArtifactTypes.asScala.toSet[String].map(Type(_)))
+
+    Fetch(resolve, artifacts, None)
+      .withFetchCache(Option(fetch.getFetchCacheIKnowWhatImDoing))
   }
 
   def fetch(fetch: Fetch[Task]): coursierapi.Fetch = {