dangernoodle-io / breadboard / 30417839708

97%

push

github

fix: tolerate duplicate route registration in bb_wifi_routes_init() (#1125)

bb_wifi_routes_init() aborted its entire init on any single
route-registration failure, including a benign duplicate (method,path)
outcome (first-registration-wins) shared with bb_wifi_prov's own scan
route. Each registration now goes through register_route_tolerate_dup(),
which downgrades BB_ERR_INVALID_STATE to BB_OK per route while leaving a
genuine failure (e.g. BB_ERR_NO_SPACE) fatal, same posture as
bb_http_register_route()'s own dispatch-table-full handling. The
bb_err_t -> bb_err_t mapping decision is extracted into a pure,
host-testable helper (bb_wifi_http_route_register_outcome) since the
caller itself is ESP-IDF-only.

Addresses B1-1259.


Claude-Session: https://claude.ai/code/session_01KM3KoyubaoM6eoUAsw8GYr

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

7892 of 8496 branches covered (92.89%)

Branch coverage included in aggregate %.

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

13414 of 13569 relevant lines covered (98.86%)

2653.35 hits per line