| // Copyright 2021 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.bazel.bzlmod; |
| |
| import static com.google.common.truth.Truth.assertThat; |
| import static com.google.devtools.build.lib.bazel.bzlmod.BzlmodTestUtil.createModuleKey; |
| import static org.junit.Assert.assertThrows; |
| import static org.junit.Assert.fail; |
| |
| import com.google.common.base.Suppliers; |
| import com.google.common.collect.ImmutableBiMap; |
| import com.google.common.collect.ImmutableList; |
| import com.google.common.collect.ImmutableMap; |
| import com.google.common.collect.ImmutableSet; |
| import com.google.devtools.build.lib.actions.FileValue; |
| import com.google.devtools.build.lib.analysis.BlazeDirectories; |
| import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider; |
| import com.google.devtools.build.lib.analysis.ServerDirectories; |
| import com.google.devtools.build.lib.analysis.util.AnalysisMock; |
| import com.google.devtools.build.lib.bazel.bzlmod.BzlmodTestUtil.InterimModuleBuilder; |
| import com.google.devtools.build.lib.bazel.bzlmod.ModuleFileValue.RootModuleFileValue; |
| import com.google.devtools.build.lib.bazel.repository.starlark.StarlarkRepositoryModule; |
| import com.google.devtools.build.lib.clock.BlazeClock; |
| import com.google.devtools.build.lib.cmdline.RepositoryName; |
| import com.google.devtools.build.lib.packages.semantics.BuildLanguageOptions; |
| import com.google.devtools.build.lib.pkgcache.PathPackageLocator; |
| import com.google.devtools.build.lib.rules.repository.LocalRepositoryFunction; |
| import com.google.devtools.build.lib.rules.repository.LocalRepositoryRule; |
| import com.google.devtools.build.lib.rules.repository.RepositoryDelegatorFunction; |
| import com.google.devtools.build.lib.rules.repository.RepositoryFunction; |
| import com.google.devtools.build.lib.skyframe.BazelSkyframeExecutorConstants; |
| import com.google.devtools.build.lib.skyframe.BzlmodRepoRuleFunction; |
| import com.google.devtools.build.lib.skyframe.ClientEnvironmentFunction; |
| import com.google.devtools.build.lib.skyframe.ExternalFilesHelper; |
| import com.google.devtools.build.lib.skyframe.ExternalFilesHelper.ExternalFileAction; |
| import com.google.devtools.build.lib.skyframe.FileFunction; |
| import com.google.devtools.build.lib.skyframe.FileStateFunction; |
| import com.google.devtools.build.lib.skyframe.PrecomputedFunction; |
| import com.google.devtools.build.lib.skyframe.PrecomputedValue; |
| import com.google.devtools.build.lib.skyframe.SkyFunctions; |
| import com.google.devtools.build.lib.starlarkbuildapi.repository.RepositoryBootstrap; |
| import com.google.devtools.build.lib.testutil.FoundationTestCase; |
| import com.google.devtools.build.lib.testutil.TestRuleClassProvider; |
| import com.google.devtools.build.lib.util.io.TimestampGranularityMonitor; |
| import com.google.devtools.build.lib.vfs.FileStateKey; |
| import com.google.devtools.build.lib.vfs.Root; |
| import com.google.devtools.build.lib.vfs.SyscallCache; |
| import com.google.devtools.build.skyframe.EvaluationContext; |
| import com.google.devtools.build.skyframe.EvaluationResult; |
| import com.google.devtools.build.skyframe.InMemoryMemoizingEvaluator; |
| import com.google.devtools.build.skyframe.MemoizingEvaluator; |
| import com.google.devtools.build.skyframe.RecordingDifferencer; |
| import com.google.devtools.build.skyframe.SequencedRecordingDifferencer; |
| import com.google.devtools.build.skyframe.SkyFunction; |
| import com.google.devtools.build.skyframe.SkyFunctionName; |
| import com.google.devtools.build.skyframe.SkyKey; |
| import java.util.LinkedHashMap; |
| import java.util.Map; |
| import java.util.Optional; |
| import java.util.concurrent.atomic.AtomicBoolean; |
| import java.util.concurrent.atomic.AtomicReference; |
| import net.starlark.java.eval.Dict; |
| import net.starlark.java.eval.EvalException; |
| import net.starlark.java.eval.StarlarkSemantics; |
| import net.starlark.java.syntax.Location; |
| import org.junit.Before; |
| import org.junit.Test; |
| import org.junit.runner.RunWith; |
| import org.junit.runners.JUnit4; |
| |
| /** Tests for {@link ModuleFileFunction}. */ |
| @RunWith(JUnit4.class) |
| public class ModuleFileFunctionTest extends FoundationTestCase { |
| |
| private MemoizingEvaluator evaluator; |
| private RecordingDifferencer differencer; |
| private EvaluationContext evaluationContext; |
| private FakeRegistry.Factory registryFactory; |
| |
| @Before |
| public void setup() throws Exception { |
| setUpWithBuiltinModules(ImmutableMap.of()); |
| } |
| |
| private void setUpWithBuiltinModules(ImmutableMap<String, NonRegistryOverride> builtinModules) { |
| differencer = new SequencedRecordingDifferencer(); |
| evaluationContext = |
| EvaluationContext.newBuilder().setNumThreads(8).setEventHandler(reporter).build(); |
| registryFactory = new FakeRegistry.Factory(); |
| AtomicReference<PathPackageLocator> packageLocator = |
| new AtomicReference<>( |
| new PathPackageLocator( |
| outputBase, |
| ImmutableList.of(Root.fromPath(rootDirectory)), |
| BazelSkyframeExecutorConstants.BUILD_FILES_BY_PRIORITY)); |
| BlazeDirectories directories = |
| new BlazeDirectories( |
| new ServerDirectories(rootDirectory, outputBase, rootDirectory), |
| rootDirectory, |
| /* defaultSystemJavabase= */ null, |
| AnalysisMock.get().getProductName()); |
| ExternalFilesHelper externalFilesHelper = |
| ExternalFilesHelper.createForTesting( |
| packageLocator, |
| ExternalFileAction.DEPEND_ON_EXTERNAL_PKG_FOR_EXTERNAL_REPO_PATHS, |
| directories); |
| ConfiguredRuleClassProvider.Builder builder = new ConfiguredRuleClassProvider.Builder(); |
| TestRuleClassProvider.addStandardRules(builder); |
| builder |
| .clearWorkspaceFileSuffixForTesting() |
| .addStarlarkBootstrap(new RepositoryBootstrap(new StarlarkRepositoryModule())); |
| ConfiguredRuleClassProvider ruleClassProvider = builder.build(); |
| |
| ImmutableMap<String, RepositoryFunction> repositoryHandlers = |
| ImmutableMap.of(LocalRepositoryRule.NAME, new LocalRepositoryFunction()); |
| evaluator = |
| new InMemoryMemoizingEvaluator( |
| ImmutableMap.<SkyFunctionName, SkyFunction>builder() |
| .put(FileValue.FILE, new FileFunction(packageLocator, directories)) |
| .put( |
| FileStateKey.FILE_STATE, |
| new FileStateFunction( |
| Suppliers.ofInstance( |
| new TimestampGranularityMonitor(BlazeClock.instance())), |
| SyscallCache.NO_CACHE, |
| externalFilesHelper)) |
| .put( |
| SkyFunctions.MODULE_FILE, |
| new ModuleFileFunction(registryFactory, rootDirectory, builtinModules)) |
| .put(SkyFunctions.PRECOMPUTED, new PrecomputedFunction()) |
| .put( |
| SkyFunctions.REPOSITORY_DIRECTORY, |
| new RepositoryDelegatorFunction( |
| repositoryHandlers, |
| null, |
| new AtomicBoolean(true), |
| ImmutableMap::of, |
| directories, |
| BazelSkyframeExecutorConstants.EXTERNAL_PACKAGE_HELPER)) |
| .put( |
| BzlmodRepoRuleValue.BZLMOD_REPO_RULE, |
| new BzlmodRepoRuleFunction(ruleClassProvider, directories)) |
| .put( |
| SkyFunctions.CLIENT_ENVIRONMENT_VARIABLE, |
| new ClientEnvironmentFunction(new AtomicReference<>(ImmutableMap.of()))) |
| .buildOrThrow(), |
| differencer); |
| |
| PrecomputedValue.STARLARK_SEMANTICS.set( |
| differencer, |
| StarlarkSemantics.builder().setBool(BuildLanguageOptions.ENABLE_BZLMOD, true).build()); |
| RepositoryDelegatorFunction.REPOSITORY_OVERRIDES.set(differencer, ImmutableMap.of()); |
| RepositoryDelegatorFunction.DEPENDENCY_FOR_UNCONDITIONAL_FETCHING.set( |
| differencer, RepositoryDelegatorFunction.DONT_FETCH_UNCONDITIONALLY); |
| PrecomputedValue.PATH_PACKAGE_LOCATOR.set(differencer, packageLocator.get()); |
| RepositoryDelegatorFunction.RESOLVED_FILE_INSTEAD_OF_WORKSPACE.set( |
| differencer, Optional.empty()); |
| PrecomputedValue.REPO_ENV.set(differencer, ImmutableMap.of()); |
| RepositoryDelegatorFunction.OUTPUT_VERIFICATION_REPOSITORY_RULES.set( |
| differencer, ImmutableSet.of()); |
| RepositoryDelegatorFunction.RESOLVED_FILE_FOR_VERIFICATION.set(differencer, Optional.empty()); |
| ModuleFileFunction.IGNORE_DEV_DEPS.set(differencer, false); |
| ModuleFileFunction.MODULE_OVERRIDES.set(differencer, ImmutableMap.of()); |
| YankedVersionsUtil.ALLOWED_YANKED_VERSIONS.set(differencer, ImmutableList.of()); |
| } |
| |
| @Test |
| public void testRootModule() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(", |
| " name='aaa',", |
| " version='0.1',", |
| " compatibility_level=4,", |
| ")", |
| "bazel_dep(name='bbb',version='1.0')", |
| "bazel_dep(name='ccc',version='2.0',repo_name='see')", |
| "register_toolchains('//my:toolchain', '//my:toolchain2')", |
| "register_execution_platforms('//my:platform', '//my:platform2')", |
| "single_version_override(module_name='ddd',version='18')", |
| "local_path_override(module_name='eee',path='somewhere/else')", |
| "multiple_version_override(module_name='fff',versions=['1.0','2.0'])", |
| "archive_override(module_name='ggg',urls=['https://hello.com/world.zip'])"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| EvaluationResult<RootModuleFileValue> result = |
| evaluator.evaluate( |
| ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| if (result.hasError()) { |
| fail(result.getError().toString()); |
| } |
| RootModuleFileValue rootModuleFileValue = result.get(ModuleFileValue.KEY_FOR_ROOT_MODULE); |
| assertThat(rootModuleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("aaa", "0.1", 4) |
| .setKey(ModuleKey.ROOT) |
| .addExecutionPlatformsToRegister( |
| ImmutableList.of("//my:platform", "//my:platform2")) |
| .addToolchainsToRegister(ImmutableList.of("//my:toolchain", "//my:toolchain2")) |
| .addDep("bbb", createModuleKey("bbb", "1.0")) |
| .addDep("see", createModuleKey("ccc", "2.0")) |
| .build()); |
| assertThat(rootModuleFileValue.getOverrides()) |
| .containsExactly( |
| "ddd", |
| SingleVersionOverride.create( |
| Version.parse("18"), "", ImmutableList.of(), ImmutableList.of(), 0), |
| "eee", LocalPathOverride.create("somewhere/else"), |
| "fff", |
| MultipleVersionOverride.create( |
| ImmutableList.of(Version.parse("1.0"), Version.parse("2.0")), ""), |
| "ggg", |
| ArchiveOverride.create( |
| ImmutableList.of("https://hello.com/world.zip"), |
| ImmutableList.of(), |
| ImmutableList.of(), |
| "", |
| "", |
| 0)); |
| assertThat(rootModuleFileValue.getNonRegistryOverrideCanonicalRepoNameLookup()) |
| .containsExactly( |
| RepositoryName.create("eee~override"), "eee", |
| RepositoryName.create("ggg~override"), "ggg"); |
| } |
| |
| @Test |
| public void testRootModule_noModuleFunctionIsOkay() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "bazel_dep(name='bbb',version='1.0')"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| EvaluationResult<RootModuleFileValue> result = |
| evaluator.evaluate( |
| ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| if (result.hasError()) { |
| fail(result.getError().toString()); |
| } |
| RootModuleFileValue rootModuleFileValue = result.get(ModuleFileValue.KEY_FOR_ROOT_MODULE); |
| assertThat(rootModuleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("", "") |
| .setKey(ModuleKey.ROOT) |
| .addDep("bbb", createModuleKey("bbb", "1.0")) |
| .build()); |
| assertThat(rootModuleFileValue.getOverrides()).isEmpty(); |
| assertThat(rootModuleFileValue.getNonRegistryOverrideCanonicalRepoNameLookup()).isEmpty(); |
| } |
| |
| @Test |
| public void testRootModule_badSelfOverride() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='aaa')", |
| "single_version_override(module_name='aaa',version='7')"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| EvaluationResult<RootModuleFileValue> result = |
| evaluator.evaluate( |
| ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| assertThat(result.hasError()).isTrue(); |
| assertThat(result.getError().toString()).contains("invalid override for the root module"); |
| } |
| |
| @Test |
| public void testRegistriesCascade() throws Exception { |
| // Registry1 has no module B@1.0; registry2 and registry3 both have it. We should be using the |
| // B@1.0 from registry2. |
| FakeRegistry registry1 = registryFactory.newFakeRegistry("/foo"); |
| FakeRegistry registry2 = |
| registryFactory |
| .newFakeRegistry("/bar") |
| .addModule( |
| createModuleKey("bbb", "1.0"), |
| "module(name='bbb',version='1.0');bazel_dep(name='ccc',version='2.0')"); |
| FakeRegistry registry3 = |
| registryFactory |
| .newFakeRegistry("/baz") |
| .addModule( |
| createModuleKey("bbb", "1.0"), |
| "module(name='bbb',version='1.0');bazel_dep(name='ddd',version='3.0')"); |
| ModuleFileFunction.REGISTRIES.set( |
| differencer, ImmutableList.of(registry1.getUrl(), registry2.getUrl(), registry3.getUrl())); |
| |
| SkyKey skyKey = ModuleFileValue.key(createModuleKey("bbb", "1.0"), null); |
| EvaluationResult<ModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| fail(result.getError().toString()); |
| } |
| ModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("bbb", "1.0") |
| .addDep("ccc", createModuleKey("ccc", "2.0")) |
| .setRegistry(registry2) |
| .build()); |
| } |
| |
| @Test |
| public void testLocalPathOverride() throws Exception { |
| // There is an override for B to use the local path "code_for_b", so we shouldn't even be |
| // looking at the registry. |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='aaa',version='0.1')", |
| "local_path_override(module_name='bbb',path='code_for_b')"); |
| scratch.file( |
| rootDirectory.getRelative("code_for_b/MODULE.bazel").getPathString(), |
| "module(name='bbb',version='1.0')", |
| "bazel_dep(name='ccc',version='2.0')"); |
| scratch.file(rootDirectory.getRelative("code_for_b/WORKSPACE").getPathString()); |
| FakeRegistry registry = |
| registryFactory |
| .newFakeRegistry("/foo") |
| .addModule( |
| createModuleKey("bbb", "1.0"), |
| "module(name='bbb',version='1.0');bazel_dep(name='ccc',version='3.0')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| // The version is empty here due to the override. |
| SkyKey skyKey = |
| ModuleFileValue.key(createModuleKey("bbb", ""), LocalPathOverride.create("code_for_b")); |
| EvaluationResult<ModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| fail(result.getError().toString()); |
| } |
| ModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("bbb", "1.0") |
| .setKey(createModuleKey("bbb", "")) |
| .addDep("ccc", createModuleKey("ccc", "2.0")) |
| .build()); |
| } |
| |
| @Test |
| public void testCommandLineModuleOverrides() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='aaa',version='0.1')", |
| "bazel_dep(name = \"bbb\", version = \"1.0\")", |
| "local_path_override(module_name='bbb', path='ignored_override')"); |
| |
| // Command line override has the priority. Thus, "used_override" with dependency on 'ccc' |
| // should be selected. |
| scratch.file( |
| rootDirectory.getRelative("ignored_override/MODULE.bazel").getPathString(), |
| "module(name='bbb',version='1.0')"); |
| scratch.file(rootDirectory.getRelative("ignored_override/WORKSPACE").getPathString()); |
| scratch.file( |
| rootDirectory.getRelative("used_override/MODULE.bazel").getPathString(), |
| "module(name='bbb',version='1.0')", |
| "bazel_dep(name='ccc',version='2.0')"); |
| scratch.file(rootDirectory.getRelative("used_override/WORKSPACE").getPathString()); |
| |
| // ModuleFileFuncion.MODULE_OVERRIDES should be filled from command line options |
| // Inject for testing |
| Map<String, ModuleOverride> moduleOverride = |
| new LinkedHashMap<>(ImmutableMap.of("bbb", LocalPathOverride.create("used_override"))); |
| ModuleFileFunction.MODULE_OVERRIDES.set(differencer, ImmutableMap.copyOf(moduleOverride)); |
| |
| FakeRegistry registry = |
| registryFactory |
| .newFakeRegistry("/foo") |
| .addModule( |
| createModuleKey("bbb", "1.0"), |
| "module(name='bbb',version='1.0');bazel_dep(name='ccc',version='3.0')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| // The version is empty here due to the override. |
| SkyKey skyKey = |
| ModuleFileValue.key(createModuleKey("bbb", ""), LocalPathOverride.create("used_override")); |
| EvaluationResult<ModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| fail(result.getError().toString()); |
| } |
| ModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("bbb", "1.0") |
| .setKey(createModuleKey("bbb", "")) |
| .addDep("ccc", createModuleKey("ccc", "2.0")) |
| .build()); |
| } |
| |
| @Test |
| public void testRegistryOverride() throws Exception { |
| FakeRegistry registry1 = |
| registryFactory |
| .newFakeRegistry("/foo") |
| .addModule( |
| createModuleKey("bbb", "1.0"), |
| "module(name='bbb',version='1.0',compatibility_level=4)", |
| "bazel_dep(name='ccc',version='2.0')"); |
| FakeRegistry registry2 = |
| registryFactory |
| .newFakeRegistry("/foo") |
| .addModule( |
| createModuleKey("bbb", "1.0"), |
| "module(name='bbb',version='1.0',compatibility_level=6)", |
| "bazel_dep(name='ccc',version='3.0')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry1.getUrl())); |
| |
| // Override the registry for B to be registry2 (instead of the default registry1). |
| SkyKey skyKey = |
| ModuleFileValue.key( |
| createModuleKey("bbb", "1.0"), |
| SingleVersionOverride.create( |
| Version.EMPTY, registry2.getUrl(), ImmutableList.of(), ImmutableList.of(), 0)); |
| EvaluationResult<ModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| fail(result.getError().toString()); |
| } |
| ModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("bbb", "1.0", 6) |
| .addDep("ccc", createModuleKey("ccc", "3.0")) |
| .setRegistry(registry2) |
| .build()); |
| } |
| |
| @Test |
| public void testModuleExtensions_good() throws Exception { |
| FakeRegistry registry = |
| registryFactory |
| .newFakeRegistry("/foo") |
| .addModule( |
| createModuleKey("mymod", "1.0"), |
| "module(name='mymod',version='1.0')", |
| "myext1 = use_extension('//:defs.bzl','myext1')", |
| "use_repo(myext1, 'repo1')", |
| "myext1.tag(key='val')", |
| "myext2 = use_extension('//:defs.bzl','myext2')", |
| "use_repo(myext2, 'repo2', other_repo1='repo1')", |
| "myext2.tag1(key1='val1')", |
| "myext2.tag2(key2='val2')", |
| "bazel_dep(name='rules_jvm_external',version='2.0')", |
| "maven = use_extension('@rules_jvm_external//:defs.bzl','maven')", |
| "use_repo(maven, mvn='maven')", |
| "maven.dep(coord='junit')", |
| "use_repo(maven, 'junit', 'guava')", |
| "maven.dep(coord='guava')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| ModuleKey myMod = createModuleKey("mymod", "1.0"); |
| SkyKey skyKey = ModuleFileValue.key(myMod, null); |
| EvaluationResult<ModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| throw result.getError().getException(); |
| } |
| ModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("mymod", "1.0") |
| .addDep("rules_jvm_external", createModuleKey("rules_jvm_external", "2.0")) |
| .setRegistry(registry) |
| .addExtensionUsage( |
| ModuleExtensionUsage.builder() |
| .setExtensionBzlFile("@mymod//:defs.bzl") |
| .setExtensionName("myext1") |
| .setIsolationKey(Optional.empty()) |
| .setUsingModule(myMod) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 2, 23)) |
| .setImports(ImmutableBiMap.of("repo1", "repo1")) |
| .setDevImports(ImmutableSet.of()) |
| .setHasDevUseExtension(false) |
| .setHasNonDevUseExtension(true) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("key", "val") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 4, 11)) |
| .build()) |
| .build()) |
| .addExtensionUsage( |
| ModuleExtensionUsage.builder() |
| .setExtensionBzlFile("@mymod//:defs.bzl") |
| .setExtensionName("myext2") |
| .setIsolationKey(Optional.empty()) |
| .setUsingModule(myMod) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 5, 23)) |
| .setImports(ImmutableBiMap.of("other_repo1", "repo1", "repo2", "repo2")) |
| .setDevImports(ImmutableSet.of()) |
| .setHasDevUseExtension(false) |
| .setHasNonDevUseExtension(true) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag1") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("key1", "val1") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 7, 12)) |
| .build()) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag2") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("key2", "val2") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 8, 12)) |
| .build()) |
| .build()) |
| .addExtensionUsage( |
| ModuleExtensionUsage.builder() |
| .setExtensionBzlFile("@rules_jvm_external//:defs.bzl") |
| .setExtensionName("maven") |
| .setIsolationKey(Optional.empty()) |
| .setUsingModule(myMod) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 10, 22)) |
| .setImports( |
| ImmutableBiMap.of("mvn", "maven", "junit", "junit", "guava", "guava")) |
| .setDevImports(ImmutableSet.of()) |
| .setHasDevUseExtension(false) |
| .setHasNonDevUseExtension(true) |
| .addTag( |
| Tag.builder() |
| .setTagName("dep") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("coord", "junit") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 12, 10)) |
| .build()) |
| .addTag( |
| Tag.builder() |
| .setTagName("dep") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("coord", "guava") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 14, 10)) |
| .build()) |
| .build()) |
| .build()); |
| } |
| |
| @Test |
| public void testModuleExtensions_duplicateProxy_asRoot() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "myext1 = use_extension('//:defs.bzl','myext',dev_dependency=True)", |
| "myext1.tag(name = 'tag1')", |
| "use_repo(myext1, 'alpha')", |
| "myext2 = use_extension('//:defs.bzl','myext')", |
| "myext2.tag(name = 'tag2')", |
| "use_repo(myext2, 'beta')", |
| "myext3 = use_extension('//:defs.bzl','myext',dev_dependency=True)", |
| "myext3.tag(name = 'tag3')", |
| "use_repo(myext3, 'gamma')", |
| "myext4 = use_extension('//:defs.bzl','myext')", |
| "myext4.tag(name = 'tag4')", |
| "use_repo(myext4, 'delta')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of()); |
| |
| SkyKey skyKey = ModuleFileValue.KEY_FOR_ROOT_MODULE; |
| EvaluationResult<ModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| throw result.getError().getException(); |
| } |
| ModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("", "") |
| .setKey(ModuleKey.ROOT) |
| .addExtensionUsage( |
| ModuleExtensionUsage.builder() |
| .setExtensionBzlFile("@//:defs.bzl") |
| .setExtensionName("myext") |
| .setIsolationKey(Optional.empty()) |
| .setUsingModule(ModuleKey.ROOT) |
| .setLocation(Location.fromFileLineColumn("/workspace/MODULE.bazel", 1, 23)) |
| .setImports( |
| ImmutableBiMap.of( |
| "alpha", "alpha", "beta", "beta", "gamma", "gamma", "delta", |
| "delta")) |
| .setDevImports(ImmutableSet.of("alpha", "gamma")) |
| .setHasDevUseExtension(true) |
| .setHasNonDevUseExtension(true) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("name", "tag1") |
| .buildImmutable())) |
| .setDevDependency(true) |
| .setLocation( |
| Location.fromFileLineColumn("/workspace/MODULE.bazel", 2, 11)) |
| .build()) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("name", "tag2") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn("/workspace/MODULE.bazel", 5, 11)) |
| .build()) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("name", "tag3") |
| .buildImmutable())) |
| .setDevDependency(true) |
| .setLocation( |
| Location.fromFileLineColumn("/workspace/MODULE.bazel", 8, 11)) |
| .build()) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("name", "tag4") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn("/workspace/MODULE.bazel", 11, 11)) |
| .build()) |
| .build()) |
| .build()); |
| } |
| |
| @Test |
| public void testModuleExtensions_duplicateProxy_asDep() throws Exception { |
| FakeRegistry registry = |
| registryFactory |
| .newFakeRegistry("/foo") |
| .addModule( |
| createModuleKey("mymod", "1.0"), |
| "module(name='mymod',version='1.0')", |
| "myext1 = use_extension('//:defs.bzl','myext',dev_dependency=True)", |
| "myext1.tag(name = 'tag1')", |
| "use_repo(myext1, 'alpha')", |
| "myext2 = use_extension('//:defs.bzl','myext')", |
| "myext2.tag(name = 'tag2')", |
| "use_repo(myext2, 'beta')", |
| "myext3 = use_extension('//:defs.bzl','myext',dev_dependency=True)", |
| "myext3.tag(name = 'tag3')", |
| "use_repo(myext3, 'gamma')", |
| "myext4 = use_extension('//:defs.bzl','myext')", |
| "myext4.tag(name = 'tag4')", |
| "use_repo(myext4, 'delta')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| ModuleKey myMod = createModuleKey("mymod", "1.0"); |
| SkyKey skyKey = ModuleFileValue.key(myMod, null); |
| EvaluationResult<ModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| throw result.getError().getException(); |
| } |
| ModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("mymod", "1.0") |
| .setRegistry(registry) |
| .addExtensionUsage( |
| ModuleExtensionUsage.builder() |
| .setExtensionBzlFile("@mymod//:defs.bzl") |
| .setExtensionName("myext") |
| .setIsolationKey(Optional.empty()) |
| .setUsingModule(myMod) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 5, 23)) |
| .setImports(ImmutableBiMap.of("beta", "beta", "delta", "delta")) |
| .setDevImports(ImmutableSet.of()) |
| .setHasDevUseExtension(false) |
| .setHasNonDevUseExtension(true) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("name", "tag2") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 6, 11)) |
| .build()) |
| .addTag( |
| Tag.builder() |
| .setTagName("tag") |
| .setAttributeValues( |
| AttributeValues.create( |
| Dict.<String, Object>builder() |
| .put("name", "tag4") |
| .buildImmutable())) |
| .setDevDependency(false) |
| .setLocation( |
| Location.fromFileLineColumn( |
| "fake:0/modules/mymod/1.0/MODULE.bazel", 12, 11)) |
| .build()) |
| .build()) |
| .build()); |
| } |
| |
| @Test |
| public void testModuleExtensions_repoNameCollision_localRepoName() throws Exception { |
| FakeRegistry registry = |
| registryFactory |
| .newFakeRegistry("/foo") |
| .addModule( |
| createModuleKey("mymod", "1.0"), |
| "module(name='mymod',version='1.0')", |
| "myext = use_extension('//:defs.bzl','myext')", |
| "use_repo(myext, mymod='some_repo')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| SkyKey skyKey = ModuleFileValue.key(createModuleKey("mymod", "1.0"), null); |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| |
| assertContainsEvent( |
| "The repo name 'mymod' is already being used as the current module name at"); |
| } |
| |
| @Test |
| public void testModuleExtensions_repoNameCollision_exportedRepoName() throws Exception { |
| FakeRegistry registry = |
| registryFactory |
| .newFakeRegistry("/foo") |
| .addModule( |
| createModuleKey("mymod", "1.0"), |
| "module(name='mymod',version='1.0')", |
| "myext = use_extension('//:defs.bzl','myext')", |
| "use_repo(myext, 'some_repo', again='some_repo')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| SkyKey skyKey = ModuleFileValue.key(createModuleKey("mymod", "1.0"), null); |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| |
| assertContainsEvent( |
| "The repo exported as 'some_repo' by module extension 'myext' is already imported at"); |
| } |
| |
| @Test |
| public void testModuleFileExecute_syntaxError() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='aaa',version='0.1',compatibility_level=4)", |
| "foo()"); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| assertContainsEvent("name 'foo' is not defined"); |
| } |
| |
| @Test |
| public void testModuleFileExecute_evalError() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='aaa',version='0.1',compatibility_level=\"4\")"); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("parameter 'compatibility_level' got value of type 'string', want 'int'"); |
| } |
| |
| @Test |
| public void validateModuleName() throws Exception { |
| ModuleFileGlobals.validateModuleName("abc"); |
| ModuleFileGlobals.validateModuleName("a3"); |
| ModuleFileGlobals.validateModuleName("a.e"); |
| ModuleFileGlobals.validateModuleName("a.-_e"); |
| ModuleFileGlobals.validateModuleName("a"); |
| |
| assertThrows(EvalException.class, () -> ModuleFileGlobals.validateModuleName("")); |
| assertThrows(EvalException.class, () -> ModuleFileGlobals.validateModuleName("fooBar")); |
| assertThrows(EvalException.class, () -> ModuleFileGlobals.validateModuleName("_foo")); |
| assertThrows(EvalException.class, () -> ModuleFileGlobals.validateModuleName("foo#bar")); |
| assertThrows(EvalException.class, () -> ModuleFileGlobals.validateModuleName("foo~bar")); |
| } |
| |
| @Test |
| public void badModuleName_module() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='f.',version='0.1')"); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("invalid module name 'f.'"); |
| } |
| |
| @Test |
| public void badModuleName_bazelDep() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "bazel_dep(name='f.',version='0.1')"); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("invalid module name 'f.'"); |
| } |
| |
| @Test |
| public void badRepoName_module() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='foo',version='0.1',repo_name='_foo')"); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("invalid user-provided repo name '_foo'"); |
| } |
| |
| @Test |
| public void badRepoName_bazelDep() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "bazel_dep(name='foo',version='0.1',repo_name='_foo')"); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("invalid user-provided repo name '_foo'"); |
| } |
| |
| @Test |
| public void badRepoName_useRepo() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "ext=use_extension('//:hello.bzl', 'ext')", |
| "use_repo(ext, foo='_foo')"); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("invalid user-provided repo name '_foo'"); |
| } |
| |
| @Test |
| public void badRepoName_useRepo_assignedName() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "ext=use_extension('//:hello.bzl', 'ext')", |
| "use_repo(ext, _foo='foo')"); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("invalid user-provided repo name '_foo'"); |
| } |
| |
| @Test |
| public void testBuiltinModules_forRoot() throws Exception { |
| ImmutableMap<String, NonRegistryOverride> builtinModules = |
| ImmutableMap.of( |
| "bazel_tools", |
| LocalPathOverride.create("/tools"), |
| "local_config_platform", |
| LocalPathOverride.create("/local_config_platform")); |
| setUpWithBuiltinModules(builtinModules); |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "bazel_dep(name='foo',version='1.0')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of()); |
| |
| SkyKey skyKey = ModuleFileValue.KEY_FOR_ROOT_MODULE; |
| EvaluationResult<RootModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| throw result.getError().getException(); |
| } |
| RootModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("", "") |
| .addDep("bazel_tools", createModuleKey("bazel_tools", "")) |
| .addDep("local_config_platform", createModuleKey("local_config_platform", "")) |
| .addDep("foo", createModuleKey("foo", "1.0")) |
| .build()); |
| assertThat(moduleFileValue.getOverrides()).containsExactlyEntriesIn(builtinModules); |
| } |
| |
| @Test |
| public void testBuiltinModules_forBuiltinModules() throws Exception { |
| ImmutableMap<String, NonRegistryOverride> builtinModules = |
| ImmutableMap.of( |
| "bazel_tools", |
| LocalPathOverride.create(rootDirectory.getRelative("tools").getPathString()), |
| "local_config_platform", |
| LocalPathOverride.create("/local_config_platform")); |
| setUpWithBuiltinModules(builtinModules); |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "bazel_dep(name='foo',version='1.0')"); |
| scratch.file(rootDirectory.getRelative("tools/WORKSPACE").getPathString()); |
| scratch.file( |
| rootDirectory.getRelative("tools/MODULE.bazel").getPathString(), |
| "module(name='bazel_tools',version='1.0')", |
| "bazel_dep(name='foo',version='2.0')"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of()); |
| |
| SkyKey skyKey = |
| ModuleFileValue.key(createModuleKey("bazel_tools", ""), builtinModules.get("bazel_tools")); |
| EvaluationResult<ModuleFileValue> result = |
| evaluator.evaluate(ImmutableList.of(skyKey), evaluationContext); |
| if (result.hasError()) { |
| throw result.getError().getException(); |
| } |
| ModuleFileValue moduleFileValue = result.get(skyKey); |
| assertThat(moduleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("bazel_tools", "1.0") |
| .setKey(createModuleKey("bazel_tools", "")) |
| .addDep("local_config_platform", createModuleKey("local_config_platform", "")) |
| .addDep("foo", createModuleKey("foo", "2.0")) |
| .build()); |
| } |
| |
| @Test |
| public void moduleRepoName() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='aaa',version='0.1',repo_name='bbb')"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| EvaluationResult<RootModuleFileValue> result = |
| evaluator.evaluate( |
| ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| if (result.hasError()) { |
| fail(result.getError().toString()); |
| } |
| RootModuleFileValue rootModuleFileValue = result.get(ModuleFileValue.KEY_FOR_ROOT_MODULE); |
| assertThat(rootModuleFileValue.getModule()) |
| .isEqualTo( |
| InterimModuleBuilder.create("aaa", "0.1") |
| .setKey(ModuleKey.ROOT) |
| .setRepoName("bbb") |
| .build()); |
| } |
| |
| @Test |
| public void moduleRepoName_conflict() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='aaa',version='0.1',repo_name='bbb')", |
| "bazel_dep(name='bbb',version='1.0')"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("The repo name 'bbb' is already being used as the module's own repo name"); |
| } |
| |
| @Test |
| public void module_calledTwice() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "module(name='aaa',version='0.1',repo_name='bbb')", |
| "module(name='aaa',version='0.1',repo_name='bbb')"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("the module() directive can only be called once"); |
| } |
| |
| @Test |
| public void module_calledLate() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "use_extension('//:extensions.bzl', 'my_ext')", |
| "module(name='aaa',version='0.1',repo_name='bbb')"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| |
| assertContainsEvent("if module() is called, it must be called before any other functions"); |
| } |
| |
| @Test |
| public void isolatedUsageWithoutImports() throws Exception { |
| PrecomputedValue.STARLARK_SEMANTICS.set( |
| differencer, |
| StarlarkSemantics.builder() |
| .setBool(BuildLanguageOptions.ENABLE_BZLMOD, true) |
| .setBool(BuildLanguageOptions.EXPERIMENTAL_ISOLATED_EXTENSION_USAGES, true) |
| .build()); |
| |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "isolated_ext = use_extension('//:extensions.bzl', 'my_ext', isolate = True)"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| EvaluationResult<RootModuleFileValue> result = |
| evaluator.evaluate( |
| ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| assertThat(result.hasError()).isTrue(); |
| assertThat(result.getError().toString()) |
| .contains( |
| "the isolated usage at /workspace/MODULE.bazel:1:29 of extension my_ext defined in " |
| + "@//:extensions.bzl has no effect as no repositories are imported from it. " |
| + "Either import one or more repositories generated by the extension with " |
| + "use_repo or remove the usage."); |
| } |
| |
| @Test |
| public void isolatedUsageNotExported() throws Exception { |
| PrecomputedValue.STARLARK_SEMANTICS.set( |
| differencer, |
| StarlarkSemantics.builder() |
| .setBool(BuildLanguageOptions.ENABLE_BZLMOD, true) |
| .setBool(BuildLanguageOptions.EXPERIMENTAL_ISOLATED_EXTENSION_USAGES, true) |
| .build()); |
| |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "use_extension('//:extensions.bzl', 'my_ext', isolate = True)"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| assertContainsEvent( |
| "Isolated extension usage at /workspace/MODULE.bazel:1:14 must be assigned to a " |
| + "top-level variable"); |
| } |
| |
| @Test |
| public void isolatedUsage_notEnabled() throws Exception { |
| scratch.file( |
| rootDirectory.getRelative("MODULE.bazel").getPathString(), |
| "use_extension('//:extensions.bzl', 'my_ext', isolate = True)"); |
| FakeRegistry registry = registryFactory.newFakeRegistry("/foo"); |
| ModuleFileFunction.REGISTRIES.set(differencer, ImmutableList.of(registry.getUrl())); |
| |
| reporter.removeHandler(failFastHandler); // expect failures |
| evaluator.evaluate(ImmutableList.of(ModuleFileValue.KEY_FOR_ROOT_MODULE), evaluationContext); |
| assertContainsEvent( |
| "Error in use_extension: in call to use_extension(), parameter 'isolate' is experimental " |
| + "and thus unavailable with the current flags. It may be enabled by setting " |
| + "--experimental_isolated_extension_usages"); |
| } |
| } |