dangernoodle-io / breadboard / 27518525631

100%

push

github

feat(bb_nv,bb_diag): network NVS delete via DELETE /api/nvs with JSON payload (B1-290) (#433)

- single DELETE /api/nvs endpoint replaces four separate routes
  (removes DELETE /api/mqtt, /api/httppub, /api/telemetry and the
  old query-param form of /api/nvs)
- body: namespace (string or array-of-strings), optional key (string),
  confirm (bool, required true), wipe_wifi (bool, required when
  namespace is/includes bb_cfg)
- 412 when confirm is absent or false; 412 when bb_cfg targeted
  without wipe_wifi:true (includes array case)
- 400 when namespace is absent or key is combined with array namespace
- 200 {"deleted":[...]} lists each cleared namespace; "key" present
  when a single key was erased
- array namespace erases each in turn — covers the reset-telemetry
  case: ["bb_mqtt","bb_sink_http","bb_pub"]
- body read uses the same bb_http_req_body_len / bb_http_req_recv /
  bb_json_parse pattern as PATCH /api/telemetry
- fix(bb_http): add 412 and 201 to bb_http_resp_set_status status
  map on espidf; 412 was missing, causing set_status(412) to silently
  return BB_ERR_INVALID_ARG and leave the response status at 200 —
  DELETE /api/nvs without confirm sent HTTP 200 on device despite the
  handler calling send_412() (B1-290 dispatch status bug)
- fix(bb_http_host): host mock bb_http_resp_set_status now validates
  status codes against the same set as the espidf backend; previously
  the mock accepted any integer, masking the 412-missing bug in CI —
  the existing 412 tests would now FAIL against the unfixed espidf code
- BB_DIAG_ROUTE_COUNT corrected: 10 → 7 (was 4 nv-delete routes,
  now 1)
- tests rewritten for body-based endpoint: 12 new handler tests + 5
  bb_nv_erase_namespace unit tests retained; per-resource tests dropped

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

1667 of 1667 branches covered (100.0%)

Branch coverage included in aggregate %.

2786 of 2786 relevant lines covered (100.0%)

1120.04 hits per line