dangernoodle-io / breadboard / 30188140451

97%

push

github

fix: degrade doc-only schema-compose failures in mqtt and temp health sections (#1099)

B1-1231 sweep, lane 2 of 3 (B1-1240 + B1-1241): a doc-only OpenAPI fragment compose failure aborted registration of the mqtt/temp /api/health sections entirely, dropping them from the response body over a documentation-only failure.

- components/bb_mqtt_client/src/bb_mqtt_client_health_section.c: bb_mqtt_client_health_register() no longer bails on a compose failure; logs a warning (with rc) and registers the section with schema_props left NULL (omitted by bb_health_schema's pointer-NULL gate).

- platform/espidf/bb_temp/bb_temp.c + platform/host/bb_temp/bb_temp.c: same degrade-and-continue fix applied identically to bb_temp_register_info() in both platform twins (shared block stays byte-identical, verified by diff after edit); the espidf copy ships on device.

- test/test_host_openapi_meta_on/test_bb_mqtt_client_health_schema_wiring.c + test_bb_temp_health_schema_wiring.c: both sites are host-compiled via native_openapi_runtime_meta, so the existing offline-on-compose-failure tests are rewritten (renamed *_degrades_on_compose_failure) to assert the section still registers with schema_props == NULL.

- test/test_host_openapi_meta_on/test_main.c: RUN_TEST/forward-decl renames to match.

- test/test_host/test_bb_health_section.c + test_main.c: adds test_bb_health_section_test_find_returns_null_when_not_registered -- a direct, producer-independent regression test for bb_health_section_test_find()'s not-found (NULL) branch, orphaned by the two rewrites above (this test file runs in every merged host env, no fault-injection or native_openapi_runtime_meta dependency needed).

No CMakeLists REQUIRES changes needed -- bb_log was already a PRIV_REQUIRES dep of both bb_mqtt_client and bb_temp.

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

7798 of 8401 branches covered (92.82%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 2 files covered. (100.0%)

13249 of 13404 relevant lines covered (98.84%)

2620.36 hits per line