CyclopsMC / IntegratedCrafting / 33432430269

64%

master-1.12: 27%

Pull #221

github

Cover the attuned recipe gui container with game tests

Coveralls flagged the previous commit for dropping coverage: the recipe list
container was the largest new class and had nothing exercising it, since this
repo has no precedent for testing containers at all.

Two game tests now drive it the way a player does. The first writes the part's
gui data and constructs the container from that buffer, which covers the whole
client-sync encoding: recipes sent by id and resolved back through the recipe
manager, the disabled keys, the search filter, and the server index that bulk
actions are keyed on. The second feeds the container the value-notifier payloads
the gui sends, covering single toggles, all three bulk actions, a repeated
identical toggle, and the two cases the server refuses: indexes from a stale
recipe list and indexes out of range. Opening the container through the part's
own menu provider is covered too.

The value ids of both payloads are exposed so a test can address them, and the
gui-side recipe reader is public for the same reason.

The screen itself stays untested, as every other screen in this repo is.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T9xTc8kXNRDppvXktL24aR

1005 of 1831 branches covered (54.89%)

Branch coverage included in aggregate %.

3509 of 5071 relevant lines covered (69.2%)

3.56 hits per line