moonbitlang / core / 5817

90%

push

github

fix(quickcheck): omit empty categories from ObservationSummary debug rendering (#3992)

* fix(quickcheck): omit empty categories from ObservationSummary debug rendering

With snapshot tests as the primary way to consume reports, the Debug
rendering is effectively API surface. A classes-only summary previously
rendered a noisy labels: {} field (and vice versa); empty categories are
now omitted, so classify-based coverage snapshots read as
observations={ classes: {...} }.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* test: make the classes-only rendering snapshot distribution-independent

Review feedback: classify unconditionally so the class count always
equals the test count and cannot drift if Int generation or the default
seed changes; the test only exercises the rendering shape.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* style: build the summary record with conditional spreads

Review feedback: construct the field map declaratively in the literal
instead of mutating it. Repr::record is kept (rather than Repr on a
Map) so keys keep the unquoted record rendering that snapshots rely on.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix: qualify Repr inside the spread literal

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

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

16231 of 17970 relevant lines covered (90.32%)

168728.94 hits per line