tree 06c4192db59a382bdfd7e622f3f1b43f79827b3e
parent 13a1c3d93991f0ea7f3c0be7eee29422cb5e68da
author Nate Bosch <nbosch@google.com> 1616443778 -0700
committer GitHub <noreply@github.com> 1616443778 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJgWPmCCRBK7hj4Ov3rIwAAdHIIAI+b6x4pjuttwXrc0y+rlRlv
 nK+fjSO2W+b2oY8BPlWSKvXUwK6VbmA17Qrn7mV0+BQo4ioQJQobHIo+80DuA/B3
 RzDhtKWoQelhSCBz+LWbkQAxmwFKiQr+zJrGhGvp2eiyO91VbHMTOrKC0MXAdvZI
 3tPCmImbP8+I+0AFuV43zz5UyMvthhqoEt2ET9lFsnVZpxuBIdaKUV4IPCxyNqVY
 fcBbPiJqKBf7Ek+TUWpFx8/BHpoAzpymX7OCFtowPSWHg7hQFwXVN7mUHsQeQhdf
 4v+tZuglsunS0Qs8GUeqidm20ovFnPMjZBQ0IrU1MSKdvW9y8qqfb4vRzMafpt0=
 =bJw3
 -----END PGP SIGNATURE-----
 

Disable stack trace chains by default - for real (#1470)

Changing the default in the arg parser did not change the runner
behavior, and the test that changed was a false positive for the
behavior difference.

Change the default for `chainStackTraces` in the `Metadata` class which
is the real owner of the defaults. Enable stack trace chaining by
default when using `--verbose-trace`, otherwise default to off.

Remove the extra failure output about enabling stack trace chaining for
load "test" failures since they often have no stack trace.

The output is still very verbose - this commit does not reduce the
output to only once at the end of the tests, but that is something we
should consider doing.

Enable stack trace chaining in some tests where the output is checked in
the test.