add tests for prinfo merging

Author: daniel-mohedano

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/GitDataUploaderImpl.java

@@ -112,8 +112,7 @@ private void uploadGitData() {
         return;
       }
 
-      if (config.isCiVisibilityGitUnshallowDefer()
-          && gitRepoUnshallow.unshallow()) {
+      if (config.isCiVisibilityGitUnshallowDefer() && gitRepoUnshallow.unshallow()) {
         latestCommits = gitClient.getLatestCommits();
         commitsToSkip = gitDataApi.searchCommits(remoteUrl, latestCommits);
       }

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/GitRepoUnshallow.java

@@ -3,8 +3,6 @@
 import datadog.trace.api.Config;
 import datadog.trace.civisibility.utils.ShellCommandExecutor;
 import java.io.IOException;
-import java.util.concurrent.CompletableFuture;
-import java.util.concurrent.Future;
 import java.util.concurrent.TimeoutException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;

dd-java-agent/agent-ci-visibility/src/test/groovy/datadog/trace/civisibility/ci/PullRequestInfoTest.groovy

@@ -0,0 +1,16 @@
+package datadog.trace.civisibility.ci
+
+import spock.lang.Specification
+
+class PullRequestInfoTest extends Specification {
+  def "test merging of informations"() {
+    PullRequestInfo.merge(infoA, infoB) == result
+
+    where:
+    infoA                                      | infoB                                    | result
+    new PullRequestInfo("branchA", "a", "a")   | new PullRequestInfo("branchB", "b", "b") | new PullRequestInfo("branchA", "a", "a")
+    new PullRequestInfo(null, null, null)      | new PullRequestInfo("branchB", "b", "b") | new PullRequestInfo("branchB", "b", "b")
+    new PullRequestInfo("branchA", null, null) | new PullRequestInfo("branchB", "b", "b") | new PullRequestInfo("branchA", "b", "b")
+    new PullRequestInfo("branchA", null, null) | new PullRequestInfo(null, null, null)    | new PullRequestInfo("branchA", null, null)
+  }
+}