blob: ad24db49be630d206ae6d1ded81f2b52ff6a8380 [file]
// Copyright 2026 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.skyframe;
import static com.google.common.truth.Truth.assertThat;
import com.google.common.collect.ImmutableSet;
import com.google.devtools.build.lib.actions.ActionExecutionContext;
import com.google.devtools.build.lib.actions.ActionExecutionException;
import com.google.devtools.build.lib.actions.ActionInputMap;
import com.google.devtools.build.lib.actions.ActionResult;
import com.google.devtools.build.lib.actions.Artifact;
import com.google.devtools.build.lib.actions.InputMetadataProvider;
import com.google.devtools.build.lib.actions.util.TestAction;
import com.google.devtools.build.lib.collect.nestedset.NestedSet;
import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder;
import com.google.devtools.build.lib.collect.nestedset.Order;
import com.google.devtools.build.lib.vfs.FileSystemUtils;
import java.util.concurrent.atomic.AtomicReference;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.junit.runners.JUnit4;
/** Timestamp builder tests for inputs that are only discovered after an action was executed. */
@RunWith(JUnit4.class)
public class LateDiscoveredInputsTest extends TimestampBuilderTestCase {
/**
* The action file system reads the {@link ActionInputMap} that backs the {@link
* InputMetadataProvider} handed to the action, and it outlives the action: build events
* referencing it are uploaded asynchronously. Since the map is only thread-compatible, inputs
* discovered after execution must not be added to it. See
* https://github.com/bazelbuild/bazel/issues/30683.
*/
@Test
public void inputsDiscoveredAfterExecution_notAddedToInputMetadataProviderOfAction()
throws Exception {
Artifact hello = createSourceArtifact("hello");
hello.getPath().getParentDirectory().createDirectoryAndParents();
FileSystemUtils.writeContentAsLatin1(hello.getPath(), "content1");
Artifact late = createSourceArtifact("late");
FileSystemUtils.writeContentAsLatin1(late.getPath(), "late1");
Artifact goodbye = createDerivedArtifact("goodbye");
Button button = new Button();
AtomicReference<InputMetadataProvider> inputMetadataProvider = new AtomicReference<>();
registerAction(
new LateInputDiscoveringAction(
button,
NestedSetBuilder.create(Order.STABLE_ORDER, hello),
ImmutableSet.of(goodbye),
late,
inputMetadataProvider));
button.pressed = false;
buildArtifacts(cachingBuilder(), goodbye);
assertThat(button.pressed).isTrue(); // built
assertThat(inputMetadataProvider.get().getInput(late.getExecPath())).isNull();
// The late-discovered input is still tracked by the action cache.
button.pressed = false;
buildArtifacts(cachingBuilder(), goodbye);
assertThat(button.pressed).isFalse(); // not rebuilt
FileSystemUtils.writeContentAsLatin1(late.getPath(), "late2");
button.pressed = false;
buildArtifacts(cachingBuilder(), goodbye);
assertThat(button.pressed).isTrue(); // rebuilt
}
/** A {@link TestAction} that only learns about {@code late} once it has been executed. */
private static final class LateInputDiscoveringAction extends TestAction {
private final Artifact late;
private final AtomicReference<InputMetadataProvider> inputMetadataProvider;
LateInputDiscoveringAction(
Runnable effect,
NestedSet<Artifact> inputs,
ImmutableSet<Artifact> outputs,
Artifact late,
AtomicReference<InputMetadataProvider> inputMetadataProvider) {
super(effect, inputs, outputs);
this.late = late;
this.inputMetadataProvider = inputMetadataProvider;
}
@Override
public boolean discoversInputs() {
return true;
}
@Override
public NestedSet<Artifact> discoverInputs(ActionExecutionContext actionExecutionContext) {
return NestedSetBuilder.emptySet(Order.STABLE_ORDER);
}
@Override
public ActionResult execute(ActionExecutionContext actionExecutionContext)
throws ActionExecutionException, InterruptedException {
inputMetadataProvider.set(actionExecutionContext.getInputMetadataProvider());
ActionResult result = super.execute(actionExecutionContext);
updateInputs(
NestedSetBuilder.<Artifact>stableOrder()
.addTransitive(getMandatoryInputs())
.add(late)
.build());
return result;
}
}
}