| // Copyright 2016 The Bazel Authors. All rights reserved. |
| // |
| // Licensed under the Apache License, Version 2.0 (the "License"); |
| // you may not use this file except in compliance with the License. |
| // You may obtain a copy of the License at |
| // |
| // http://www.apache.org/licenses/LICENSE-2.0 |
| // |
| // Unless required by applicable law or agreed to in writing, software |
| // distributed under the License is distributed on an "AS IS" BASIS, |
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| // See the License for the specific language governing permissions and |
| // limitations under the License. |
| package com.google.devtools.build.lib.rules; |
| |
| import static com.google.common.truth.Truth.assertThat; |
| |
| import com.google.common.collect.ImmutableList; |
| import com.google.common.collect.Iterables; |
| import com.google.devtools.build.lib.actions.Artifact; |
| import com.google.devtools.build.lib.actions.util.ActionsTestUtil; |
| import com.google.devtools.build.lib.analysis.AnalysisResult; |
| import com.google.devtools.build.lib.analysis.ConfiguredAspect; |
| import com.google.devtools.build.lib.analysis.ConfiguredTarget; |
| import com.google.devtools.build.lib.analysis.FileProvider; |
| import com.google.devtools.build.lib.analysis.LicensesProvider; |
| import com.google.devtools.build.lib.analysis.LicensesProvider.TargetLicense; |
| import com.google.devtools.build.lib.analysis.RunfilesProvider; |
| import com.google.devtools.build.lib.analysis.config.BuildConfigurationValue; |
| import com.google.devtools.build.lib.analysis.util.BuildViewTestCase; |
| import com.google.devtools.build.lib.cmdline.Label; |
| import com.google.devtools.build.lib.cmdline.RepositoryName; |
| import com.google.devtools.build.lib.collect.nestedset.NestedSet; |
| import com.google.devtools.build.lib.packages.License.LicenseType; |
| import com.google.devtools.build.lib.packages.Provider; |
| import com.google.devtools.build.lib.packages.StarlarkProvider; |
| import com.google.devtools.build.lib.packages.StructImpl; |
| import com.google.devtools.build.lib.rules.cpp.CcInfo; |
| import com.google.devtools.build.lib.skyframe.AspectKeyCreator.AspectKey; |
| import java.util.Set; |
| import org.junit.Test; |
| import org.junit.runner.RunWith; |
| import org.junit.runners.JUnit4; |
| |
| /** Unit tests for the <code>alias</code> rule. */ |
| @RunWith(JUnit4.class) |
| public class AliasTest extends BuildViewTestCase { |
| @Test |
| public void smoke() throws Exception { |
| scratch.file("a/BUILD", |
| "cc_library(name='a', srcs=['a.cc'])", |
| "alias(name='b', actual='a')"); |
| |
| ConfiguredTarget b = getConfiguredTarget("//a:b"); |
| assertThat(b.get(CcInfo.PROVIDER).getCcCompilationContext()).isNotNull(); |
| } |
| |
| @Test |
| public void aliasToInputFile() throws Exception { |
| scratch.file("a/BUILD", |
| "exports_files(['a'])", |
| "alias(name='b', actual='a')"); |
| |
| ConfiguredTarget b = getConfiguredTarget("//a:b"); |
| assertThat(ActionsTestUtil.baseArtifactNames(getFilesToBuild(b))).containsExactly("a"); |
| } |
| |
| @Test |
| public void visibilityIsOverriddenAndIsOkay() throws Exception { |
| scratch.file("a/BUILD", |
| "filegroup(name='a', visibility=['//b:__pkg__'])"); |
| scratch.file("b/BUILD", |
| "alias(name='b', actual='//a:a', visibility=['//visibility:public'])"); |
| scratch.file("c/BUILD", |
| "filegroup(name='c', srcs=['//b:b'])"); |
| |
| getConfiguredTarget("//c:c"); |
| } |
| |
| @Test |
| public void visibilityIsOverriddenAndIsError() throws Exception { |
| scratch.file("a/BUILD", |
| "filegroup(name='a', visibility=['//visibility:public'])"); |
| scratch.file("b/BUILD", |
| "alias(name='b', actual='//a:a', visibility=['//visibility:private'])"); |
| scratch.file("c/BUILD", |
| "filegroup(name='c', srcs=['//b:b'])"); |
| |
| reporter.removeHandler(failFastHandler); |
| getConfiguredTarget("//c:c"); |
| assertContainsEvent( |
| "alias '//b:b' referring to target '//a:a' is not visible from target '//c:c'"); |
| } |
| |
| @Test |
| public void visibilityIsOverriddenAndIsErrorAfterMultipleAliases() throws Exception { |
| scratch.file("a/BUILD", |
| "filegroup(name='a', visibility=['//visibility:public'])"); |
| scratch.file("b/BUILD", |
| "alias(name='b', actual='//a:a', visibility=['//visibility:public'])"); |
| scratch.file("c/BUILD", |
| "alias(name='c', actual='//b:b', visibility=['//visibility:private'])"); |
| scratch.file("d/BUILD", |
| "filegroup(name='d', srcs=['//c:c'])"); |
| |
| reporter.removeHandler(failFastHandler); |
| getConfiguredTarget("//d:d"); |
| assertContainsEvent("alias '//c:c' referring to target '//a:a' through '//b:b' " |
| + "is not visible from target '//d:d'"); |
| } |
| |
| @Test |
| public void testAliasWithPrivateVisibilityAccessibleFromSamePackage() throws Exception { |
| scratch.file("a/BUILD", "exports_files(['af'])"); |
| scratch.file("b/BUILD", |
| "package(default_visibility=['//visibility:private'])", |
| "alias(name='al', actual='//a:af')", |
| "filegroup(name='ta', srcs=[':al'])"); |
| |
| getConfiguredTarget("//b:ta"); |
| } |
| |
| @Test |
| public void testAliasCycle() throws Exception { |
| scratch.file("a/BUILD", |
| "alias(name='a', actual=':b')", |
| "alias(name='b', actual=':c')", |
| "alias(name='c', actual=':a')", |
| "filegroup(name='d', srcs=[':c'])"); |
| |
| reporter.removeHandler(failFastHandler); |
| getConfiguredTarget("//a:d"); |
| assertContainsEvent("cycle in dependency graph"); |
| } |
| |
| @Test |
| public void testAliasedInvalidDependency() throws Exception { |
| scratch.file("a/BUILD", |
| "cc_library(name='a', deps=[':b'])", |
| "alias(name='b', actual=':c')", |
| "filegroup(name='c')"); |
| |
| reporter.removeHandler(failFastHandler); |
| getConfiguredTarget("//a:a"); |
| assertContainsEvent("alias '//a:b' referring to filegroup rule '//a:c' is misplaced here"); |
| } |
| |
| @Test |
| public void testAspectPropagation() throws Exception { |
| writeConfigTransitionTestFiles(); |
| scratch.file( |
| "test/aspect.bzl", |
| "load('//myinfo:myinfo.bzl', 'MyInfo')", |
| "def _impl(target, ctx):", |
| " if not target[MyInfo]:", |
| " fail('missing MyInfo')", |
| " if target[MyInfo].config != ctx.configuration:", |
| " fail('mismatched configs')", |
| " return MyInfo(", |
| " origin = 'aspect',", |
| " config = target[MyInfo].config)", |
| "MyAspect = aspect(implementation=_impl)"); |
| scratch.file( |
| "test/BUILD", |
| "alias(name = 'simple_alias', actual = '//test/starlark:test')", |
| "alias(name = 'selecting_alias',", |
| " actual = select({':arm': ':simple_alias'}))", |
| "config_setting(name = 'arm', values = {'cpu': 'armeabi-v7a'})"); |
| |
| // Set --cpu so we can test alias :selecting_alias that selects on this flag |
| useConfiguration("--cpu=armeabi-v7a"); |
| |
| // 1. Query "actual" target to establish reference values to compare to below. Make some basic |
| // assertions that tie aspect's config to underlying target. |
| AnalysisResult analysisResult = |
| update( |
| ImmutableList.of("//test/starlark:test"), |
| ImmutableList.of("//test:aspect.bzl%MyAspect"), |
| true, |
| 1, |
| true, |
| eventBus); |
| assertThat(analysisResult.getTargetsToBuild()).hasSize(1); |
| assertThat(analysisResult.getAspectsMap()).hasSize(1); |
| |
| ConfiguredTarget actualTarget = Iterables.getOnlyElement(analysisResult.getTargetsToBuild()); |
| ConfiguredAspect aspect = Iterables.getOnlyElement(analysisResult.getAspectsMap().values()); |
| AspectKey actualKey = Iterables.getOnlyElement(analysisResult.getAspectsMap().keySet()); |
| assertThat(actualKey.getBaseConfiguredTargetKey().getConfigurationKey()) |
| .isEqualTo(actualTarget.getConfigurationKey()); |
| assertThat(getMyInfoFromTarget(aspect).getValue("origin")).isEqualTo("aspect"); |
| BuildConfigurationValue actualConfig = |
| (BuildConfigurationValue) getMyInfoFromTarget(aspect).getValue("config"); |
| assertThat(actualKey.getBaseConfiguredTargetKey().getConfigurationKey().getOptions().checksum()) |
| .isEqualTo(actualConfig.checksum()); |
| |
| // 2. Query :simple_alias and assert that its aspect value is the same as above. |
| analysisResult = |
| update( |
| ImmutableList.of("//test:simple_alias"), |
| ImmutableList.of("//test:aspect.bzl%MyAspect"), |
| true, |
| 1, |
| true, |
| eventBus); |
| assertThat(analysisResult.getTargetsToBuild()).hasSize(1); |
| assertThat(analysisResult.getAspectsMap()).hasSize(1); |
| |
| ConfiguredTarget alias = Iterables.getOnlyElement(analysisResult.getTargetsToBuild()); |
| assertThat(alias.getActual()).isEqualTo(actualTarget); |
| // Alias and actual must have different configs for this test to be meaningful |
| assertThat(alias.getConfigurationKey()).isNotEqualTo(alias.getActual().getConfigurationKey()); |
| AspectKey aspectKey = Iterables.getOnlyElement(analysisResult.getAspectsMap().keySet()); |
| assertThat(aspectKey.getBaseConfiguredTargetKey().getConfigurationKey()) |
| .isEqualTo(alias.getConfigurationKey()); |
| |
| aspect = Iterables.getOnlyElement(analysisResult.getAspectsMap().values()); |
| assertThat(getMyInfoFromTarget(aspect).getValue("origin")).isEqualTo("aspect"); |
| // We should be seeing actual's config here |
| assertThat(getMyInfoFromTarget(aspect).getValue("config")).isEqualTo(actualConfig); |
| |
| // 3. Do the same with :selecting_alias, which is an indirect alias through :simple_alias. |
| // This alias also uses a (non-trivial) select to resolve its actual. |
| analysisResult = |
| update( |
| ImmutableList.of("//test:selecting_alias"), |
| ImmutableList.of("//test:aspect.bzl%MyAspect"), |
| true, |
| 1, |
| true, |
| eventBus); |
| assertThat(analysisResult.getTargetsToBuild()).hasSize(1); |
| assertThat(analysisResult.getAspectsMap()).hasSize(1); |
| |
| ConfiguredTarget indirectAlias = Iterables.getOnlyElement(analysisResult.getTargetsToBuild()); |
| assertThat(indirectAlias.getActual()).isEqualTo(actualTarget); |
| assertThat(indirectAlias.getConfigurationKey()).isEqualTo(alias.getConfigurationKey()); |
| |
| aspect = Iterables.getOnlyElement(analysisResult.getAspectsMap().values()); |
| assertThat(getMyInfoFromTarget(aspect).getValue("origin")).isEqualTo("aspect"); |
| assertThat(getMyInfoFromTarget(aspect).getValue("config")).isEqualTo(actualConfig); |
| } |
| |
| private void writeAllowlistFile() throws Exception { |
| scratch.overwriteFile( |
| "tools/allowlists/function_transition_allowlist/BUILD", |
| "package_group(", |
| " name = 'function_transition_allowlist',", |
| " packages = [", |
| " '//test/...',", |
| " ],", |
| ")"); |
| } |
| |
| private static StructImpl getMyInfoFromTarget(ConfiguredAspect configuredAspect) |
| throws Exception { |
| Provider.Key key = |
| new StarlarkProvider.Key(Label.parseCanonical("//myinfo:myinfo.bzl"), "MyInfo"); |
| return (StructImpl) configuredAspect.get(key); |
| } |
| |
| public void setupMyInfo() throws Exception { |
| scratch.file("myinfo/myinfo.bzl", "MyInfo = provider()"); |
| scratch.file("myinfo/BUILD"); |
| } |
| |
| private void writeConfigTransitionTestFiles() throws Exception { |
| writeAllowlistFile(); |
| setupMyInfo(); |
| getAnalysisMock().ccSupport().setupCcToolchainConfigForCpu(mockToolsConfig, "armeabi-v7a"); |
| scratch.file( |
| "test/starlark/my_rule.bzl", |
| "load('//myinfo:myinfo.bzl', 'MyInfo')", |
| "def transition_func(settings, attr):", |
| " return [", |
| " {'//command_line_option:cpu': 'k8'},", |
| " {'//command_line_option:cpu': 'armeabi-v7a'}", |
| " ]", |
| "my_transition = transition(implementation = transition_func, inputs = [],", |
| " outputs = ['//command_line_option:cpu'])", |
| "def impl(ctx): ", |
| " print(ctx.label, ctx.configuration)", |
| " return MyInfo(", |
| " config = ctx.configuration,", |
| " attr_deps = ctx.split_attr.deps,", |
| " attr_dep = ctx.split_attr.dep)", |
| "my_rule = rule(", |
| " implementation = impl,", |
| " attrs = {", |
| " 'deps': attr.label_list(cfg = my_transition),", |
| " 'dep': attr.label(cfg = my_transition),", |
| " '_allowlist_function_transition': attr.label(", |
| " default = '//tools/allowlists/function_transition_allowlist',", |
| " ),", |
| " })"); |
| |
| scratch.file( |
| "test/starlark/BUILD", |
| "load('//test/starlark:my_rule.bzl', 'my_rule')", |
| "my_rule(name = 'test', deps = [':main1', ':main2'], dep = ':main1')", |
| "cc_binary(name = 'main1', srcs = ['main1.c'])", |
| "cc_binary(name = 'main2', srcs = ['main2.c'])"); |
| } |
| |
| @Test |
| public void licensesAreCollected() throws Exception { |
| scratch.file("a/BUILD", |
| "filegroup(name='a', licenses=['restricted'], output_licenses=['unencumbered'])", |
| "alias(name='b', actual=':a')", |
| "filegroup(name='c', srcs=[':b'])", |
| "genrule(name='d', outs=['do'], tools=[':b'], cmd='cmd')", |
| "genrule(name='e', outs=['eo'], srcs=[':b'], cmd='cmd')"); |
| useConfiguration("--check_licenses"); |
| assertThat(getLicenses("//a:d", "//a:a")).containsExactly(LicenseType.UNENCUMBERED); |
| assertThat(getLicenses("//a:e", "//a:a")).containsExactly(LicenseType.RESTRICTED); |
| assertThat(getLicenses("//a:b", "//a:a")).containsExactly(LicenseType.RESTRICTED); |
| assertThat( |
| getConfiguredTarget("//a:b") |
| .getProvider(LicensesProvider.class) |
| .getTransitiveLicenses() |
| .toList()) |
| .hasSize(1); |
| } |
| |
| @Test |
| public void assertNoLicensesAttribute() throws Exception { |
| scratch.file("a/BUILD", |
| "filegroup(name='a')", |
| "alias(name='b', actual=':a', licenses=['unencumbered'])"); |
| |
| reporter.removeHandler(failFastHandler); |
| getConfiguredTarget("//a:b"); |
| assertContainsEvent("no such attribute 'licenses' in 'alias' rule"); |
| } |
| |
| private Set<LicenseType> getLicenses(String topLevelTarget, String licenseTarget) |
| throws Exception { |
| LicensesProvider licenses = |
| getConfiguredTarget(topLevelTarget).getProvider(LicensesProvider.class); |
| for (TargetLicense license : licenses.getTransitiveLicenses().toList()) { |
| if (license.getLabel().toString().equals(licenseTarget)) { |
| return license.getLicense().getLicenseTypes(); |
| } |
| } |
| |
| throw new IllegalStateException("License for '" + licenseTarget |
| + "' not found in the transitive closure of '" + topLevelTarget + "'"); |
| } |
| |
| @Test |
| public void passesTargetTypeCheck() throws Exception { |
| scratch.file("a/BUILD", |
| "cc_library(name='a', srcs=['a.cc'], deps=[':b'])", |
| "alias(name='b', actual=':c')", |
| "cc_library(name='c', srcs=['c.cc'])"); |
| |
| getConfiguredTarget("//a:a"); |
| } |
| |
| @Test |
| public void packageGroupInAlias() throws Exception { |
| scratch.file("a/BUILD", |
| "package_group(name='a', packages=['//a'])", |
| "alias(name='b', actual=':a')", |
| "filegroup(name='c', srcs=[':b'])"); |
| |
| reporter.removeHandler(failFastHandler); |
| getConfiguredTarget("//a:c"); |
| assertContainsEvent( |
| "in actual attribute of alias rule //a:b: package group '//a:a' is misplaced here"); |
| } |
| |
| @Test |
| public void aliasedFile() throws Exception { |
| scratch.file("a/BUILD", |
| "exports_files(['a'])", |
| "alias(name='b', actual='a')", |
| "filegroup(name='c', srcs=[':b'])"); |
| |
| ConfiguredTarget c = getConfiguredTarget("//a:c"); |
| assertThat(ActionsTestUtil.baseArtifactNames( |
| c.getProvider(FileProvider.class).getFilesToBuild())) |
| .containsExactly("a"); |
| } |
| |
| @Test |
| public void aliasedConfigSetting() throws Exception { |
| scratch.file("a/BUILD", |
| "filegroup(name='a', srcs=select({':b': ['f1'], '//conditions:default': ['f2']}))", |
| "alias(name='b', actual=':c')", |
| "config_setting(name='c', values={'define': 'foo=bar'})"); |
| |
| useConfiguration("--define=foo=bar"); |
| getConfiguredTarget("//a"); |
| } |
| |
| @Test |
| public void aliasedTestSuiteDep() throws Exception { |
| scratch.file("a/BUILD", |
| "sh_test(name='a', srcs=['a.sh'])"); |
| scratch.file("b/BUILD", |
| "alias(name='b', actual='//a:a', testonly=1)", |
| "test_suite(name='c', tests=[':b'])"); |
| |
| ConfiguredTarget c = getConfiguredTarget("//b:c"); |
| NestedSet<Artifact> runfiles = |
| c.getProvider(RunfilesProvider.class).getDataRunfiles().getAllArtifacts(); |
| assertThat(ActionsTestUtil.baseArtifactNames(runfiles)).contains("a.sh"); |
| } |
| |
| @Test |
| public void testRedirectChasing() throws Exception { |
| RepositoryName toolsRepository = ruleClassProvider.getToolsRepository(); |
| scratch.file("a/BUILD", |
| "alias(name='cc', actual='" + toolsRepository + "//tools/cpp:toolchain')", |
| "cc_library(name='a', srcs=['a.cc'])"); |
| |
| useConfiguration("--crosstool_top=//a:cc"); |
| getConfiguredTarget("//a:a"); |
| } |
| |
| @Test |
| public void testNoActual() throws Exception { |
| checkError("a", "a", "missing value for mandatory attribute 'actual'", "alias(name='a')"); |
| } |
| } |