commit 5f312dd1678878fb7563eae0cd184f2270346352
author ulfjack <ulfjack@google.com> Tue Oct 02 04:58:53 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Tue Oct 02 05:00:10 2018 -0700
tree 4d4f75cd70ba6b7bbc23999325244a645b48c0df
parent f59fad7c97ed9b4d5ae577e6ef50cefd3927e4bd

Fix event id for action_completed BEP events

The id should be the exec path not the absolute path. Otherwise the
stdout and stderr don't line up correctly.

PiperOrigin-RevId: 215369211

src/test/java/com/google/devtools/build/lib/runtime/BuildEventStreamerTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/runtime/BuildEventStreamerTest.java b/src/test/java/com/google/devtools/build/lib/runtime/BuildEventStreamerTest.java
index 53d4a74..02b130d 100644
--- a/src/test/java/com/google/devtools/build/lib/runtime/BuildEventStreamerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/runtime/BuildEventStreamerTest.java
@@ -86,6 +86,7 @@
 
   private static final ActionExecutedEvent SUCCESSFUL_ACTION_EXECUTED_EVENT =
       new ActionExecutedEvent(
+          ActionsTestUtil.DUMMY_ARTIFACT.getExecPath(),
           new ActionsTestUtil.NullAction(),
           /* exception= */ null,
           ActionsTestUtil.DUMMY_ARTIFACT.getPath(),
@@ -912,6 +913,7 @@
 
     ActionExecutedEvent failedActionExecutedEvent =
         new ActionExecutedEvent(
+            ActionsTestUtil.DUMMY_ARTIFACT.getExecPath(),
             new ActionsTestUtil.NullAction(),
             new ActionExecutionException("Exception", /* action= */ null, /* catastrophe= */ false),
             ActionsTestUtil.DUMMY_ARTIFACT.getPath(),
@@ -942,6 +944,7 @@
 
     ActionExecutedEvent failedActionExecutedEvent =
         new ActionExecutedEvent(
+            ActionsTestUtil.DUMMY_ARTIFACT.getExecPath(),
             new ActionsTestUtil.NullAction(),
             new ActionExecutionException("Exception", /* action= */ null, /* catastrophe= */ false),
             ActionsTestUtil.DUMMY_ARTIFACT.getPath(),