kaidokert / rm32 / 25197859786

63%

Build Type

push

github

Committed by web-flow

Commit Message

Make struct fields private, add accessor methods (#24)

* Make struct fields private, add accessor methods

Strip pub/pub(crate) from struct fields across BemfState, DutyState,
ProtectionState, Measurements, TimingState, Commutation, MainState,
and InputState. Add minimal accessor methods where external access
is needed.

Fields kept pub only where rm32_stm32 constructs or directly writes
them (DutyState::minimum/min_startup/startup_max, ProtectionState
bemf_timeout fields, MainState::config/just_armed/cell_count).

The harness binary is a separate crate in Cargo's model, so all
harness field access now goes through public accessor methods.
This forms a stable API while field layout remains free to change.

* Remove &mut accessors, add specific setter methods instead

forward_mut(), measurements_mut(), timing_mut(), pid_mut() returned
mutable references to internal state — fake encapsulation that gives
callers full field access.

Replace with specific setter methods on MainState that delegate to
sub-structs: set_use_current_limit, set_use_speed_control,
set_average_interval, set_last_average_interval, set_battery_voltage,
set_actual_current.

Commutation::forward_mut() replaced with copy-in/copy-out pattern
at the CommandProcessor call site.

Coverage Stats

0 of 175 new or added lines in 5 files covered. (0.0%)

3 existing lines in 1 file now uncovered.

3338 of 5467 relevant lines covered (61.06%)

625.85 hits per line