tree d81d9f8dead913c05e2144cc2cb28db930dd55fb
parent 4186fc3ef661fe0ecf3d97a3874a838251eef78e
author brandjon <brandjon@google.com> 1591637436 -0700
committer Copybara-Service <copybara-worker@google.com> 1591637548 -0700

Improve error handling for builtins injection

BuiltinsFailedException gets wrapped as a BzlLoadFailedException in BzlLoadFunction, and vice versa in StarlarkBuiltinsFunction. This makes for more verbose errors, but generally users who are not Bazel developers shouldn't see these errors.

The tests assert against the long form of the error rather than a substring to demonstrate the message in context.

Also slightly refactored the way errors in loaded bzls are reported, for uniformity between inlining and non-inlining code paths.

Work toward #11437.

RELNOTES: None
PiperOrigin-RevId: 315303094