diff options
| author |  ruperts <ruperts@google.com> | 2017-10-07 00:46:20 +0200 |
|---|---|---|
| committer |  Klaus Aehlig <aehlig@google.com> | 2017-10-07 11:07:30 +0200 |
| commit | 4050a89a1e3fb6a9a9c6bd080cf4e3081a9f2012 (patch) | |
| tree | 0df531d72015ad33f3a6c8ff670846808714e8bd /src/main/java/com/google/devtools/build/lib/analysis/test | |
| parent | 5f39475bb51f48182180e34e4310edb650bea462 (diff) | |
More SpawnResult-related plumbing changes to Actions, Strategies, ActionContexts, etc., so that SpawnResult metadata is returned upwards.
Note that the TODOs mostly refer to changes that will appear in a subsequent CL (a CL to return SpawnResults, contained in ActionResults, from Actions/AbstractActions). I split off the remaining SpawnResult-related changes into this CL and kept the ActionResult-related changes separate.
RELNOTES: None.
PiperOrigin-RevId: 171355611
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/analysis/test')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/test/TestActionContext.java | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/test/TestActionContext.java b/src/main/java/com/google/devtools/build/lib/analysis/test/TestActionContext.java index cc483f6ab6..44ae134d9d 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/test/TestActionContext.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/test/TestActionContext.java @@ -16,9 +16,11 @@ package com.google.devtools.build.lib.analysis.test; import com.google.devtools.build.lib.actions.ActionContext; import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ExecException; +import com.google.devtools.build.lib.actions.SpawnResult; import com.google.devtools.build.lib.vfs.Path; import com.google.devtools.build.lib.view.test.TestStatus.TestResultData; import java.io.IOException; +import java.util.Set; /** * A context for the execution of test actions ({@link TestRunnerAction}). @@ -26,11 +28,13 @@ import java.io.IOException; public interface TestActionContext extends ActionContext { /** - * Executes the test command, directing standard out / err to {@code outErr}. The status of - * the test should be communicated by posting a {@link TestResult} object to the eventbus. + * Executes the test command, directing standard out / err to {@code outErr}. The status of the + * test should be communicated by posting a {@link TestResult} object to the eventbus. + * + * @return a set of SpawnResults created during execution of the test command, if any */ - void exec(TestRunnerAction action, - ActionExecutionContext actionExecutionContext) throws ExecException, InterruptedException; + Set<SpawnResult> exec(TestRunnerAction action, ActionExecutionContext actionExecutionContext) + throws ExecException, InterruptedException; /** * Creates a cached test result. |