dangernoodle-io / breadboard / 28491360281

100%

push

github

fix(bb_wifi): align egress histogram, cover top-reason, dedup + rssi cleanup (#622)

- EVT_RECOVERY_REQUEST handler now increments reason_histogram[WIFI_REASON_BB_EGRESS_DEAD]
  to keep the histogram consistent with egress_dead_count (histogram/counter divergence fix)
- extract wifi_hist_top_reason() into platform/host/bb_wifi/wifi_hist_priv.h as a static
  inline; both gather_histogram (bb_pub_wifi.c) and bb_wifi_emit_section (bb_wifi_emit.c)
  now call the shared helper instead of hand-rolling the same loop
- add bb_wifi_test_set_reason_histogram() host stub (bb_wifi_host.c + bb_wifi_test.h) to
  inject arbitrary histogram state in tests; bb_wifi_get_reason_histogram returns injected
  values under BB_WIFI_TESTING
- add tests: test_bb_wifi_reason_histogram_inject_top_reason (test_bb_wifi.c),
  test_bb_pub_wifi_reason_histogram_top_reason_injected (test_bb_pub_wifi.c), and
  test_bb_wifi_emit_top_reason_injected (test_bb_pub_telemetry_fidelity.c) — each injects
  reason 3 count 5 and asserts top_reason_code:3 / top_reason_count:5 in the output
- remove redundant s_last_rssi from bb_wifi.c (mirrored s_cached_rssi exactly); disconnect
  handler now snapshots s_cached_rssi directly under s_ap_mux
- init s_disconnect_rssi to INT8_MIN (sentinel = no disconnect yet vs. genuine 0 dBm);
  host stub default changed from 0 to INT8_MIN; bb_wifi.h comment updated accordingly
- rename test_bb_wifi_disconnect_rssi_default_zero to _default_sentinel to match new value

1850 of 1850 branches covered (100.0%)

Branch coverage included in aggregate %.

3190 of 3190 relevant lines covered (100.0%)

1229.94 hits per line