diff options
author | 2017-02-24 16:30:15 +0000 | |
---|---|---|
committer | 2017-02-27 15:05:00 +0000 | |
commit | 777b30d06700f76ba580715429f3663de3fa0529 (patch) | |
tree | d3abdc6f42d04256c486270add24e5b0c5d3abbd /src/test/java/com/google/devtools/build/lib/exec | |
parent | 25aa033ad5657a5cfa16e8307464648b9374be2d (diff) |
Provide more reporting options to SkyFunctions
With more specific information to be reported by Skyfunctions, e.g.,
to inform the build-event protocol on missing files, the EventHandler
interface is no longer enough. Therefore, provide an enriched context
for reporting events.
--
Change-Id: I2d06166fe4d5b9054e24ad8c752fafc039e3f9f8
Reviewed-on: https://cr.bazel.build/8794
PiperOrigin-RevId: 148463437
MOS_MIGRATED_REVID=148463437
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/exec')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/exec/BlazeExecutorTest.java | 3 | ||||
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/exec/util/TestExecutorBuilder.java | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/exec/BlazeExecutorTest.java b/src/test/java/com/google/devtools/build/lib/exec/BlazeExecutorTest.java index 105829a8c2..05c65a7e1d 100644 --- a/src/test/java/com/google/devtools/build/lib/exec/BlazeExecutorTest.java +++ b/src/test/java/com/google/devtools/build/lib/exec/BlazeExecutorTest.java @@ -17,6 +17,7 @@ import static com.google.common.truth.Truth.assertThat; import com.google.common.base.Predicate; import com.google.common.collect.Iterables; +import com.google.common.eventbus.EventBus; import com.google.devtools.build.lib.actions.SpawnActionContext; import com.google.devtools.build.lib.analysis.BlazeDirectories; import com.google.devtools.build.lib.analysis.config.BinTools; @@ -60,7 +61,7 @@ public class BlazeExecutorTest { OptionsParser parser = OptionsParser.newOptionsParser(TestExecutorBuilder.DEFAULT_OPTIONS); parser.parse("--debug_print_action_contexts"); - Reporter reporter = new Reporter(); + Reporter reporter = new Reporter(new EventBus()); StoredEventHandler storedEventHandler = new StoredEventHandler(); reporter.addHandler(storedEventHandler); diff --git a/src/test/java/com/google/devtools/build/lib/exec/util/TestExecutorBuilder.java b/src/test/java/com/google/devtools/build/lib/exec/util/TestExecutorBuilder.java index 972fd9eb21..d18e004570 100644 --- a/src/test/java/com/google/devtools/build/lib/exec/util/TestExecutorBuilder.java +++ b/src/test/java/com/google/devtools/build/lib/exec/util/TestExecutorBuilder.java @@ -45,8 +45,8 @@ public class TestExecutorBuilder { public static final List<Class<? extends OptionsBase>> DEFAULT_OPTIONS = ImmutableList.of( ExecutionOptions.class, CommonCommandOptions.class); private final BlazeDirectories directories; - private Reporter reporter = new Reporter(); private EventBus bus = new EventBus(); + private Reporter reporter = new Reporter(bus); private OptionsParser optionsParser = OptionsParser.newOptionsParser(DEFAULT_OPTIONS); private List<ActionContext> strategies = new ArrayList<>(); private Map<String, SpawnActionContext> spawnStrategyMap = |