kaidokert / rm32 / 25203213512

63%

push

github

Encapsulate DutyState and ProtectionState, remove last pub fields from state.rs (#28)

* Encapsulate DutyState and ProtectionState, remove last pub fields

DutyState: make minimum/min_startup/startup_max pub(crate), add
set_duty_limits() and apply_dead_time_override() methods. Firmware
and harness now use these instead of direct field writes.

ProtectionState: make bemf_timeout_happened/bemf_timeout pub(crate),
add getters and setters. Firmware reads via getters, harness injects
via setters.

Zero pub fields remain in control/state.rs — all access is through
methods or pub(crate).

* Apply dead_time_override in harness, add regression test

The harness load_eeprom path was missing apply_dead_time_override(),
so driving_brake_strength configs never shifted duty thresholds.
This was an untested firmware path.

Add dead_time_override.txt blackbox test that verifies
driving_brake_strength=5 raises the duty setpoint floor from ~153
to >250 via the dead-time override.

* Add unit tests for DutyState and ProtectionState methods

Cover set_duty_limits, apply_dead_time_override (including zero noop),
and ProtectionState getter/setter round-trip. Addresses codecov
coverage regression from adding untested methods.

53 of 61 new or added lines in 2 files covered. (86.89%)

3393 of 5538 relevant lines covered (61.27%)

617.85 hits per line