divviup / divviup-api / 24151341698

69%

Build Type

push

github

Committed by web-flow

Commit Message

Replace panics with error propagation in DP strategy representation (#2181)

`representation_histogram()` and `representation_sumvec()` contained
`panic!()` and `.expect()` calls for invalid DP strategy states and
malformed epsilon arrays. While upstream validation currently prevents
these paths from being reached via untrusted input, these are
defense-in-depth changes that return `ValidationErrors` instead of
panicking.

Also converts the `chunk_length` missing panic in
`Histogram::representation_for_protocol` to return an error.

Coverage Stats

33 of 33 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

3896 of 6957 relevant lines covered (56.0%)

60.35 hits per line