From 522e149b6774c2d4c6c9b5229fca2b5143609ce9 Mon Sep 17 00:00:00 2001
From: Mihai Toader <mtoader@gmail.com>
Date: Wed, 4 Sep 2024 00:47:46 -0700
Subject: [PATCH] [#6721] Don't set `ExternalWorkspace.repoName()` to null when
 unchanged

---
 .../base/model/ExternalWorkspaceData.java     |  6 ++---
 .../model/primitives/ExternalWorkspace.java   | 25 +++++++------------
 ...ExternalWorkspaceReferenceBzlModeTest.java |  3 +--
 .../blaze/base/ExternalWorkspaceFixture.java  | 12 ++++-----
 4 files changed, 19 insertions(+), 27 deletions(-)

diff --git a/base/src/com/google/idea/blaze/base/model/ExternalWorkspaceData.java b/base/src/com/google/idea/blaze/base/model/ExternalWorkspaceData.java
index 9d5424a259b..5834842ea8a 100644
--- a/base/src/com/google/idea/blaze/base/model/ExternalWorkspaceData.java
+++ b/base/src/com/google/idea/blaze/base/model/ExternalWorkspaceData.java
@@ -9,7 +9,6 @@
 
 import javax.annotation.Nullable;
 
-
 public final class ExternalWorkspaceData implements ProtoWrapper<ProjectData.ExternalWorkspaceData> {
   public ImmutableMap<String, ExternalWorkspace> workspaces;
 
@@ -25,14 +24,15 @@ public static ExternalWorkspaceData create(ImmutableList<ExternalWorkspace> work
             .stream()
             .collect(
                 ImmutableMap.toImmutableMap(
-                    ExternalWorkspace::repositoryName,
+                    ExternalWorkspace::repoName,
                     Functions.identity()))
     );
   }
 
   @Override
   public ProjectData.ExternalWorkspaceData toProto() {
-    ProjectData.ExternalWorkspaceData.Builder builder = ProjectData.ExternalWorkspaceData.newBuilder();
+    ProjectData.ExternalWorkspaceData.Builder builder =
+        ProjectData.ExternalWorkspaceData.newBuilder();
 
     for (ExternalWorkspace externalWorkspace : workspaces.values()) {
       builder = builder.addWorkspaces(externalWorkspace.toProto());
diff --git a/base/src/com/google/idea/blaze/base/model/primitives/ExternalWorkspace.java b/base/src/com/google/idea/blaze/base/model/primitives/ExternalWorkspace.java
index ce4b1fe9308..8f1ed868b89 100644
--- a/base/src/com/google/idea/blaze/base/model/primitives/ExternalWorkspace.java
+++ b/base/src/com/google/idea/blaze/base/model/primitives/ExternalWorkspace.java
@@ -12,12 +12,7 @@ public abstract class ExternalWorkspace implements ProtoWrapper<ProjectData.Exte
 
   public abstract String name();
 
-  @Nullable
-  protected abstract String repoName();
-
-  public String repositoryName() {
-    return repoName() != null ? repoName() : name();
-  }
+  public abstract String repoName();
 
   public static ExternalWorkspace fromProto(ProjectData.ExternalWorkspace proto) {
     return create(proto.getName(), proto.getRepoName());
@@ -25,19 +20,17 @@ public static ExternalWorkspace fromProto(ProjectData.ExternalWorkspace proto) {
 
   @Override
   public ProjectData.ExternalWorkspace toProto() {
-    return
-        ProjectData.ExternalWorkspace.newBuilder()
-            .setName(name())
-            .setRepoName(repoName())
-            .build();
+    return ProjectData.ExternalWorkspace.newBuilder()
+               .setName(name())
+               .setRepoName(repoName())
+               .build();
   }
 
   public static ExternalWorkspace create(String name, String repoName) {
-    ExternalWorkspace.Builder builder = ExternalWorkspace.builder().setName(name);
-    if (repoName != null && !repoName.isEmpty() && repoName.compareTo(name) != 0) {
-      builder = builder.setRepoName(repoName);
-    }
-    return builder.build();
+    return ExternalWorkspace.builder()
+               .setName(name)
+               .setRepoName(repoName)
+               .build();
   }
 
   public static ExternalWorkspace.Builder builder() {
diff --git a/base/tests/integrationtests/com/google/idea/blaze/base/lang/buildfile/references/ExternalWorkspaceReferenceBzlModeTest.java b/base/tests/integrationtests/com/google/idea/blaze/base/lang/buildfile/references/ExternalWorkspaceReferenceBzlModeTest.java
index 8c870e7f54c..e35a8ef8ba5 100644
--- a/base/tests/integrationtests/com/google/idea/blaze/base/lang/buildfile/references/ExternalWorkspaceReferenceBzlModeTest.java
+++ b/base/tests/integrationtests/com/google/idea/blaze/base/lang/buildfile/references/ExternalWorkspaceReferenceBzlModeTest.java
@@ -3,7 +3,6 @@
 import com.google.common.collect.ImmutableList;
 import com.google.idea.blaze.base.ExternalWorkspaceFixture;
 import com.google.idea.blaze.base.lang.buildfile.BuildFileIntegrationTestCase;
-import com.google.idea.blaze.base.lang.buildfile.psi.BuildFile;
 import com.google.idea.blaze.base.model.ExternalWorkspaceData;
 import com.google.idea.blaze.base.model.primitives.ExternalWorkspace;
 import com.google.idea.blaze.base.model.primitives.Label;
@@ -36,7 +35,7 @@ protected ExternalWorkspaceData mockExternalWorkspaceData() {
         ExternalWorkspace.create("workspace_two", "com_workspace_two"), fileSystem);
 
     return ExternalWorkspaceData.create(
-        ImmutableList.of(workspaceOne.w, workspaceTwoMapped.w));
+        ImmutableList.of(workspaceOne.workspace, workspaceTwoMapped.workspace));
   }
 
   @Before
diff --git a/base/tests/utils/integration/com/google/idea/blaze/base/ExternalWorkspaceFixture.java b/base/tests/utils/integration/com/google/idea/blaze/base/ExternalWorkspaceFixture.java
index b95abc23e33..99191f8a9ab 100644
--- a/base/tests/utils/integration/com/google/idea/blaze/base/ExternalWorkspaceFixture.java
+++ b/base/tests/utils/integration/com/google/idea/blaze/base/ExternalWorkspaceFixture.java
@@ -18,13 +18,13 @@
 import static org.junit.Assert.assertNotNull;
 
 public class ExternalWorkspaceFixture {
-  public final ExternalWorkspace w;
+  public final ExternalWorkspace workspace;
 
   final TestFileSystem fileSystem;
   WorkspaceFileSystem workspaceFileSystem;
 
-  public ExternalWorkspaceFixture(ExternalWorkspace w, TestFileSystem fileSystem) {
-    this.w = w;
+  public ExternalWorkspaceFixture(ExternalWorkspace workspace, TestFileSystem fileSystem) {
+    this.workspace = workspace;
     this.fileSystem = fileSystem;
   }
 
@@ -44,7 +44,7 @@ WorkspaceFileSystem getWorkspaceFileSystem() {
 
       WorkspaceRoot workspaceRoot = new WorkspaceRoot(Paths.get(
           blazeProjectData.getBlazeInfo().getOutputBase().getAbsolutePath(),
-          "external", w.name()).normalize().toFile());
+          "external", workspace.name()).normalize().toFile());
 
       File workspaceRootFile = workspaceRoot.directory();
       assertThat(workspaceRootFile).isNotNull();
@@ -55,6 +55,6 @@ WorkspaceFileSystem getWorkspaceFileSystem() {
   }
 
   public Label createLabel(WorkspacePath packagePath, TargetName targetName) {
-    return Label.create(w.repositoryName(), packagePath, targetName);
+    return Label.create(workspace.repoName(), packagePath, targetName);
   }
-}
\ No newline at end of file
+}