Skip to content

Commit a77c31b

Browse files
authored
[tests] Use a constant-length timestamp redaction for test determinism (#6511)
Changes our "timestamp redaction" logic to ignore the original length of the input timestamp. Timestamps are not always constant-width -- depending on the amount of trailing zeros, they can have variable stringified lengths. Unfortunately, this variability is at-odds with our goal of deterministic test output. In this PR, we favor the deterministic output, to avoid test flaking: Variable space output logic is removed in favor of a stable redaction marker. Additionally, tests with different timestamp lengths are added to act as a regression against this particular flake. Fixes #6509
1 parent 164aa1a commit a77c31b

File tree

5 files changed

+125
-104
lines changed

5 files changed

+125
-104
lines changed

Cargo.lock

+1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)