commit 720194d2e3d382b481a296dc418519132c7fe3de
author pcloudy <pcloudy@google.com> Fri Jun 25 07:39:12 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Jun 25 07:40:36 2021 -0700
tree 12d632032b9ad7b424b6ac5e27d528ba6c4df1da
parent 1a89ce1757e75f8ba9bda76d7373a7e8527bcfc5

bzlmod: Improve setBzlFile function in RepoSpec.java

RELNOTES: None
PiperOrigin-RevId: 381468614

src/main/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpec.java

diff --git a/src/main/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpec.java b/src/main/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpec.java
index 6bde0a7..844168e 100644
--- a/src/main/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpec.java
+++ b/src/main/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpec.java
@@ -41,7 +41,7 @@
   /** The builder for {@link RepoSpec} */
   @AutoValue.Builder
   public abstract static class Builder {
-    public abstract Builder setBzlFile(Optional<String> bzlFile);
+    public abstract Builder setBzlFile(String bzlFile);
 
     public abstract Builder setRuleClassName(String name);
 

src/test/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpecTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpecTest.java b/src/test/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpecTest.java
index 99c06d6..754f65d 100644
--- a/src/test/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpecTest.java
+++ b/src/test/java/com/google/devtools/build/lib/bazel/bzlmod/RepoSpecTest.java
@@ -18,7 +18,6 @@
 import static com.google.common.truth.Truth8.assertThat;
 
 import com.google.common.collect.ImmutableMap;
-import java.util.Optional;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.junit.runners.JUnit4;
@@ -44,7 +43,7 @@
   public void starlarkRepoSpecTest() {
     RepoSpec repoSpec =
         RepoSpec.builder()
-            .setBzlFile(Optional.of("//pkg:repo.bzl"))
+            .setBzlFile("//pkg:repo.bzl")
             .setRuleClassName("my_repo")
             .setAttributes(ImmutableMap.of("attr1", "foo", "attr2", "bar"))
             .build();