[TC-FAN] Update Python test to validate lowestOff fallback value in FanControl Step command
@FrancoLionti ci sta già lavorando.
Dal 4/5/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Summary Title:
Update Python test to validate lowestOff fallback value in FanControl Step command
Description:
Following up on a PR review comment from @soares-sergio, it was identified that the fallback value for lowestOff in the FanControl cluster's Step command should be true according to the spec, rather than false.
Link to the comment
Original implementation:
bool lowestOffValue = commandData.lowestOff.ValueOr(false);
// Suggested/Spec-compliant change:
bool lowestOffValue = commandData.lowestOff.ValueOr(true);
While we have existing Python tests for the FanControl cluster, they do not currently validate this specific omitted-argument fallback behavior. This issue is to track the necessary update to the existing test scripts to ensure this spec requirement is explicitly covered and not missed in future regressions.
Steps to reproduce:
Locate the existing Python test script that covers the FanControl cluster's Step command within src/python_testing/ (Target files to check: TC_FAN_3_1.py, TC_FAN_3_2.py, TC_FAN_3_3.py, TC_FAN_3_4.py, TC_FAN_3_5.py, or TC_FAN_4_1.py).
Implement a new test step/case that sends a Step command where the lowestOff field is intentionally omitted from the payload.
Assert that the device processes the command using the default fallback value of true, exactly matching the spec requirements.
Logs:
N/A - This is a test coverage enhancement request originating from a code review.
- Lingua principale
- Python
- Stelle
- 19
- Fork
- 7
- Merge medio
- 3g 4h
- PR unite (30g)
- 2
Preparare l'ambiente
Non abbiamo ancora controllato i file di configurazione di questo progetto. 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 project-chip/matter-test-scripts
-
[TC-TSTAT-2.1] Yaml script has to be updated by adding the newly added steps as per the Test plan.Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
project-chip/matter-test-scripts#794 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
Tutte le issue di project-chip/matter-test-scripts
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
letsencrypt/cp-cps#353 ·
-
Marble Madness II is missingAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
PedestrianDynamics/pyFDS-Evac#394 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
DOI-USGS/pywatershed#421 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
python-pillow/Pillow#10087 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno