trickstercache / trickster / 39747709189

94%

main: 94%

Build Type

Pull #1212

github

Committed by thinker0

Commit Message

fix(alb,tsm): propagate error winner body and status when all fanouts fail

Address PR #1212 review feedback:
- Propagate error status code, headers, and body when all fanouts fail
  (no 2xx members), preferring lowest 4xx user error over 5xx.
- Avoid attempting accumulator timeseries merge on non-2xx responses.
- Attach spy TSUnmarshaler directly to child fanout resources in unit
  tests to reliably verify that timeseries decoding is skipped.
- Add regression test for mixed error responses (400 vs 502/504).

Signed-off-by: thinker0 <thinker0@gmail.com>
Co-authored-by: Gemini <noreply@google.com>

Pull Request Pull Request #1212: fix(alb,tsm): skip timeseries unmarshaling for non-2xx gather responses

Coverage Stats

71 of 84 new or added lines in 3 files covered. (84.52%)

4 existing lines in 2 files now uncovered.

89846 of 95866 relevant lines covered (93.72%)

17012.53 hits per line