forked from project-chip/connectedhomeip
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit 40e5d1a
[HVAC] Implement TC-TSTAT-2.2 (project-chip#36023)
* Initial generated test
* Fixed and compared against test plan
* Remove unused TC-TSTAT-2.2 yaml
* Apply suggestions from code review
Co-authored-by: fesseha-eve <88329315+fessehaeve@users.noreply.github.com>
* Update TC_TSTAT_2_2.py
fixed below mentioned discrepancies on the code:
Step-3c: The test plan expects a Success response, but the script is checking for a ConstraintError. If the value does not violate any conditions, the response should be Success. In such cases, the test will fail.
Step-4c: If the AUTO feature is enabled, the script writes the UnoccupiedCoolingSetpoint attribute with MaxCoolSetpointLimitValue and checks for a ConstraintError response, which is not required.
Step-5c: The script should check the following condition when the AUTO feature is enabled: Min(MaxHeatSetpointLimit, (UnoccupiedCoolingSetpoint - MinSetpointDeadBand)). However, this condition is missing in the script, and it incorrectly checks for a ConstraintError response.
Step-7a: The write and read verification is missing.
Step-9b: The script incorrectly writes the attribute value using AbsMinCoolSetpointLimitValue instead of MinCoolSetpointLimitValue.
Step-9c: A duplicate read check has been removed.
* Python linter fix for TC_TSTAT_2_2
* Update src/python_testing/TC_TSTAT_2_2.py
Co-authored-by: Nivi Sarkar <55898241+nivi-apple@users.noreply.github.com>
* Restore setpoints to original values at end of test
* Use get_endpoint method to find active endpoint
* Update CI Test Arguments to the apparent new style
---------
Co-authored-by: fesseha-eve <88329315+fessehaeve@users.noreply.github.com>
Co-authored-by: Kishok G <133193761+KishokG@users.noreply.github.com>
Co-authored-by: Nivi Sarkar <55898241+nivi-apple@users.noreply.github.com>1 parent 0237a2b commit 40e5d1aCopy full SHA for 40e5d1a
File tree
2 files changed
+796
-2005
lines changed- src
- app/tests/suites/certification
- python_testing
2 files changed
+796
-2005
lines changed
0 commit comments