dangernoodle-io / breadboard / 28502756441

100%

push

github

refactor(bb_http): tolerate handler=NULL in register_described_route, unify paths (B1-452) (#633)

- bb_http_register_described_route now skips the httpd/dispatch wiring
  step when route->handler is NULL, adding only the descriptor to the
  OpenAPI registry (equivalent to register_route_descriptor_only, but
  reachable through the single unified entry point)
- migrate bb_openapi, bb_manifest, bb_ota_pull, bb_ota_push, and
  bb_event_routes (/api/events) off the split
  register_route + register_route_descriptor_only pair: each descriptor
  now carries its real handler and registers via a single
  bb_http_register_described_route call
- add a defensive 501 guard in api_dispatch_handler for a NULL-handler
  dispatch hit (unreachable today — bb_api_dispatch_add is never fed a
  NULL handler — but guards the invariant)
- add host tests for the new NULL-handler branch: descriptor added
  without wiring a dispatch entry, and the underlying-registration
  failure hook has no effect on that path
- bb_websocket keeps register_route_descriptor_only: its handler type
  (bb_websocket_handler_fn) is incompatible with bb_route_t.handler, so
  it is not part of this migration

1848 of 1848 branches covered (100.0%)

Branch coverage included in aggregate %.

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

3186 of 3186 relevant lines covered (100.0%)

1253.52 hits per line