No unit tests for the Frozen scheduling / manual-override logic or the alarm-window math
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 58/100
Direzione di ricerca
Start by reading crates/podd-core/src/frozen/manager.rs around get_next_command, wanted_for, override expiry, temperature conversion, and LED deadband; then inspect sensor/manager.rs around in_alarm_window. Add table-driven unit tests for the listed logic, including midnight wrap and override expiry, and run the relevant Rust tests. Done when these safety-critical scheduling paths have regression coverage.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The tap detector (sensor/tap.rs) and alarm-message parsing (sensor/state.rs) have good tests, but the safety-critical thermostat scheduling has none: get_next_command / wanted_for / ManualOverride expiry (crates/podd-core/src/frozen/manager.rs:331-409), f_to_centi_c conversion (manager.rs:256-258), led_thermal_state deadband (manager.rs:431-456), and in_alarm_window (sensor/manager.rs:651-658) are all untested. Given the midnight-wrap and override-expiry subtleties (see the alarm-window issue), table-driven tests here would catch regressions in the code that actually decides bed temperature and when alarms run. Related: #19 (CI).
Filed from a repo-wide code audit (working tree at bcad746).
- Lingua principale
- Rust
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di GoogleBot42/podd
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
GoogleBot42/podd#63 ·
-
documentation
Difficoltà 1/5 1-3 ore Idoneità per principianti 78/100
GoogleBot42/podd#62 ·
-
documentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 69/100
GoogleBot42/podd#60 ·
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
GoogleBot42/podd#61 ·
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
GoogleBot42/podd#58 ·
Tutte le issue di GoogleBot42/podd
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
rubys/roundhouse#1917 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 2 giorni
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
[cosmic-applet-battery] Division by zero displays NaN% when UPower reports max_brightness = 0Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
pop-os/cosmic-applets#1619 ·
I maintainer di solito rispondono entro 3 giorni
-
binding:python bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno