add unit tests and take into account root path

DataDog · daniel-mohedano · May 9, 2025 · May 8, 2025 · May 8, 2025 · May 8, 2025
commit de21bd10c227224eb540f02d63e6191ab3ae5a38
@@ -170,7 +170,7 @@ public CIInfo(
     this.ciPipelineNumber = ciPipelineNumber;
     this.ciPipelineUrl = ciPipelineUrl;
     this.ciJobUrl = ciJobUrl;
-    this.ciWorkspace = ciWorkspace;
+    this.ciWorkspace = sanitizeWorkspace(ciWorkspace);
     this.ciNodeName = ciNodeName;
     this.ciNodeLabels = ciNodeLabels;
     this.ciEnvVars = ciEnvVars;
@@ -209,14 +209,20 @@ public String getCiJobUrl() {
     return ciJobUrl;
   }
 
-  /** @return Workspace path without the trailing separator */
-  public String getCiWorkspace() {
-    String realCiWorkspace = FileUtils.toRealPath(ciWorkspace);
-    return (realCiWorkspace == null || !realCiWorkspace.endsWith(File.separator))
+  private String sanitizeWorkspace(String workspace) {
+    String realCiWorkspace = FileUtils.toRealPath(workspace);
+    return (realCiWorkspace == null
+            || !realCiWorkspace.endsWith(File.separator)
+            || realCiWorkspace.length() == 1) // root path "/"
         ? realCiWorkspace
         : (realCiWorkspace.substring(0, realCiWorkspace.length() - 1));
   }
 
+  /** @return Workspace path without the trailing separator */
+  public String getCiWorkspace() {
+    return ciWorkspace;
+  }
+
   public String getCiNodeName() {
     return ciNodeName;
   }

@@ -0,0 +1,22 @@
+package datadog.trace.civisibility.ci
+
+
+import spock.lang.Specification
+
+class CIInfoTest extends Specification {
+
+  def "test ci workspace is correctly sanitized #iterationIndex"() {
+    def builder = CIInfo.builder(null)
+    builder.ciWorkspace(workspacePath)
+    def info = builder.build()
+
+    info.ciWorkspace == sanitizedPath
+
+    where:
+    workspacePath | sanitizedPath
+    null          | null
+    "/"           | "/"
+    "/repo/path"  | "/repo/path"
+    "/repo/path/" | "/repo/path"
+  }
+}
@@ -56,7 +56,6 @@ public GitInfo getGitInfo(@Nullable String repositoryPath) {
       repositoryPath = NULL_PATH_STRING;
     }
 
-    // normalize path to avoid creating two entries in the cache
     return gitInfoCache.computeIfAbsent(repositoryPath, this::buildGitInfo);
   }
-Original file line number
+Diff line change
@@ Expand Up / @@ -56,7 +56,6 @@ public GitInfo getGitInfo(@Nullable String repositoryPath) { @@
           repositoryPath = NULL_PATH_STRING;
         }
-        // normalize path to avoid creating two entries in the cache
         return gitInfoCache.computeIfAbsent(repositoryPath, this::buildGitInfo);
       }
@@ Expand Down @@