Add integration tests for build trace masking
Description
Trace masking only has unit tests.
Most integration tests use common.Trace
that does not use trace masking buffer.
We have no tests that setup trace testing in an identical manner to how real job are performed.
Proposal
- For each executor, ensure
SetMasked()
incommon.Trace
works as intended. - Add tests to ensure that each executor is using
SetMasked
correctly and the output of the trace contains masked values.
We may want to additionally explore whether we can improve the differences between common.Trace
and network.Trace
. common.Trace
is mostly used in tests, but has very little functionality in comparison to network.Trace
.