dangernoodle-io / breadboard / 30163748982

97%

push

github

test: extend live route-schema walk to bb_log_http routes (B1-1214) (#1078)

Scaffold-hint bb_log_http.c (genuinely portable, no ESP-IDF/FreeRTOS
includes) onto the host build alongside bb_storage_http_routes.c, mirror
its CONFIG_BB_LOG_ROUTES=y Kconfig default in the native test env (host
has no sdkconfig.h), and call bb_log_register_routes_init() from
test_route_schema_literals_live.c's seed so the live-registry walk covers
GET/POST /api/log/level's real schema literals too (route_count 2->4,
schema_count 11->14, zero parse failures).

Assert specific route presence, not just non-zero counts (review finding):
a `>0` floor on route_count/schema_count was already satisfied by the 2
pre-existing storage routes, so if CONFIG_BB_LOG_ROUTES=1 were ever
dropped from platformio.ini, bb_log_register_routes_init() would silently
fall back to its no-op stub (still returns BB_OK) and nothing would fail.
Added an explicit presence check for GET and POST /api/log/level in the
registry -- chosen over exact counts because it's stable across unrelated
future route additions and says what it means, while an exact count would
need bumping on every legitimate new route. Verified the guard actually
fails: with -DCONFIG_BB_LOG_ROUTES=1 removed, the test fails with "GET
/api/log/level missing from registry -- bb_log_register_routes_init()
fell back to its no-op stub".


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

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

7668 of 8265 branches covered (92.78%)

Branch coverage included in aggregate %.

13116 of 13274 relevant lines covered (98.81%)

2640.02 hits per line