File: crates/storage/src/role.rs:30 + missing test in crates/storage/src/role.rs mod tests
Severity: robustness — coverage
Obvious? no
pub fn set_default_server(&mut self, id: String) mutates worker config but no test asserts the setter writes/persists or that subsequent reads return the new value. Pure setter — straightforward unit test addition; without it a regression renaming the field or changing the assignment site goes unnoticed.
Fix: add mod tests covering set → get round-trip + persistence assertion.
Filed by /general-audit @ b901575 (2026-05-02). master: #513.
File:
crates/storage/src/role.rs:30+ missing test incrates/storage/src/role.rsmod testsSeverity: robustness — coverage
Obvious? no
pub fn set_default_server(&mut self, id: String)mutates worker config but no test asserts the setter writes/persists or that subsequent reads return the new value. Pure setter — straightforward unit test addition; without it a regression renaming the field or changing the assignment site goes unnoticed.Fix: add
mod testscovering set → get round-trip + persistence assertion.Filed by
/general-audit@b901575(2026-05-02). master: #513.