Add tests for Label.workspace_root Starlark API
RELNOTES: None
PiperOrigin-RevId: 294978673
diff --git a/src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java
index 8939a54..d530a26 100644
--- a/src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java
@@ -21,6 +21,7 @@
import com.google.common.base.Joiner;
import com.google.common.base.Preconditions;
+import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
import com.google.common.truth.Correspondence;
import com.google.devtools.build.lib.actions.Artifact;
@@ -114,6 +115,51 @@
}
@Test
+ public void testMainRepoLabelWorkspaceRoot() throws Exception {
+ scratch.file(
+ "test/skylark/extension.bzl",
+ "load('//myinfo:myinfo.bzl', 'MyInfo')",
+ "def _impl(ctx):",
+ " return [MyInfo(result = ctx.label.workspace_root)]",
+ "my_rule = rule(implementation = _impl, attrs = { })");
+ scratch.file(
+ "test/skylark/BUILD",
+ "load('//test/skylark:extension.bzl', 'my_rule')",
+ "my_rule(name='t')");
+
+ ConfiguredTarget myTarget = getConfiguredTarget("//test/skylark:t");
+ String result = (String) getMyInfoFromTarget(myTarget).getValue("result");
+ assertThat(result).isEmpty();
+ }
+
+ @Test
+ public void testExternalRepoLabelWorkspaceRoot() throws Exception {
+ scratch.overwriteFile(
+ "WORKSPACE",
+ new ImmutableList.Builder<String>()
+ .addAll(analysisMock.getWorkspaceContents(mockToolsConfig))
+ .add("local_repository(name='r', path='/r')")
+ .build());
+
+ scratch.file("/r/WORKSPACE");
+ scratch.file(
+ "/r/test/skylark/extension.bzl",
+ "load('@//myinfo:myinfo.bzl', 'MyInfo')",
+ "def _impl(ctx):",
+ " return [MyInfo(result = ctx.label.workspace_root)]",
+ "my_rule = rule(implementation = _impl, attrs = { })");
+ scratch.file(
+ "/r/BUILD", "load('//:test/skylark/extension.bzl', 'my_rule')", "my_rule(name='t')");
+
+ // Required since we have a new WORKSPACE file.
+ invalidatePackages(true);
+
+ ConfiguredTarget myTarget = getConfiguredTarget("@r//:t");
+ String result = (String) getMyInfoFromTarget(myTarget).getValue("result");
+ assertThat(result).isEqualTo("external/r");
+ }
+
+ @Test
public void testSameMethodNames() throws Exception {
// The alias feature of load() may hide the fact that two methods in the stack trace have the
// same name. This is perfectly legal as long as these two methods are actually distinct.
