tree e9523b3758ffe993ed06f762ecbd29fdd0bb2fd1
parent 56a29d3e33f6271f3bd23f0c52a2c48e7bb1f2c5
author janakr <janakr@google.com> 1617653026 -0700
committer Copybara-Service <copybara-worker@google.com> 1617653136 -0700

Thread BlazeRuntime#bugReporter through to SkyframeExecutor -> ActionExecutionFunction and friends, allowing us to make better assertions in tests.

I couldn't reproduce my claims of actually crashing in these tests, so added the additional coverage for my sins.

This may be overkill since in the end I might just remove these bug reports anyway, but the overall changes seem worth it for future testability at the cost of one more straw on the camel's back of SkyframeExecutor constructor argument length.

PiperOrigin-RevId: 366857978