Thermostat Test Scripts (#21068) (#21088)

* Yaml File updates Updated scripts - Correct attribute * REGEN * Restyled by whitespace * Restyled by prettier-yaml Co-authored-by: Restyled.io <[email protected]> Co-authored-by: chrisbeach59 <[email protected]> Co-authored-by: Restyled.io <[email protected]>
project-chip · Jul 22, 2022 · cf37a25 · cf37a25
1 parent b9871ef
commit cf37a25
Show file tree

Hide file tree

Showing 7 changed files with 1,502 additions and 1,766 deletions.
diff --git a/src/app/tests/suites/certification/PICS.yaml b/src/app/tests/suites/certification/PICS.yaml
@@ -608,38 +608,23 @@ PICS:
       id: DGSW.S.C00.Rsp
 
       # Thermostat cluster
-    - label: "Does the device implement the Heating feature"
+    - label: "Thermostat is capable of managing a heating device"
       id: TSTAT.S.F00
 
-    - label: "Does the device implement the Cooling feature"
+    - label: "Thermostat is capable of managing a cooling device"
       id: TSTAT.S.F01
 
-    - label: "Does the device implement the Occupancy feature"
+    - label: "Supports Occupied and Unoccupied setpoints"
       id: TSTAT.S.F02
 
-    - label: "Does the device implement the Auto feature"
-      id: TSTAT.S.F05
-
-    - label: "Does the device implement the MinHeatSetpointLimit attribute"
-      id: A_MINHEATSETPOINTLIMIT
-
-    - label: "Does the device implement the MaxHeatSetpointLimit attribute"
-      id: A_MAXHEATSETPOINTLIMIT
-
-    - label: "Does the device implement the MinCoolSetpointLimit attribute"
-      id: A_MINCOOLSETPOINTLIMIT
-
-    - label: "Does the device implement the MaxCoolSetpointLimit attribute"
-      id: A_MAXCOOLSETPOINTLIMIT
-
-    - label: "Does the device implement the Tolerance attribute?"
-      id: A_TEMPERATURE_TOLERANCE
+    - label: "Supports a weekly schedule of setpoint transitions(Schedule)"
+      id: TSTAT.S.F03
 
-    - label: "Does the device implement the OccupiedCoolingSetpoint attribute?"
-      id: A_OCCUPIEDCOOLINGSETPOINT
+    - label: "Supports configurable setback (or span)"
+      id: TSTAT.S.F04
 
-    - label: "Does the device implement the OccupiedHeatingSetpoint attribute?"
-      id: A_OCCUPIEDHEATINGSETPOINT
+    - label: "Supports a System Mode of Auto"
+      id: TSTAT.S.F05
 
     - label: "Does the device implement the LocalTemperature attribute?"
       id: TSTAT.S.A0000
@@ -819,24 +804,6 @@ PICS:
     - label: "Does the device implement the GetRelayStatusLog command?"
       id: TSTAT.S.C04.Rsp
 
-    - label: "Thermostat is capable of managing a heating device"
-      id: TSTAT.S.F00
-
-    - label: "Thermostat is capable of managing a cooling device"
-      id: TSTAT.S.F01
-
-    - label: "Supports Occupied and Unoccupied setpoints"
-      id: TSTAT.S.F02
-
-    - label: "Supports a weekly schedule of setpoint transitions"
-      id: TSTAT.S.F03
-
-    - label: "Supports configurable setback (or span)"
-      id: TSTAT.S.F04
-
-    - label: "Supports a System Mode of Auto"
-      id: TSTAT.S.F05
-
     #Client Commands
     - label: "Does the device implement sending the SetpointRaiseLower command?"
       id: TSTAT.C.C00.Tx

diff --git a/src/app/tests/suites/certification/Test_TC_TSTAT_1_1.yaml b/src/app/tests/suites/certification/Test_TC_TSTAT_1_1.yaml
@@ -45,7 +45,7 @@ tests:
       response:
           constraints:
               type: map32
-              minValue: 0
+              minValue: 1
               maxValue: 63
 
     - label: "Read the global attribute: AttributeList"
@@ -56,6 +56,16 @@ tests:
               type: list
               contains: [0, 27, 28, 65528, 65529, 65531, 65532, 65533]
 
+    - label:
+          "Read the Feature dependent(TSTAT.S.F00(HEAT)) attribute in
+          AttributeList"
+      PICS: TSTAT.S.F00
+      command: "readAttribute"
+      attribute: "AttributeList"
+      response:
+          constraints:
+              type: list
+              contains: [18]
     - label:
           "Read the Feature dependent(TSTAT.S.F01(COOL)) attribute in
           AttributeList"
@@ -79,15 +89,15 @@ tests:
               contains: [2]
 
     - label:
-          "Read the Feature dependent(TSTAT.S.F00(HEAT)) attribute in
-          AttributeList"
-      PICS: TSTAT.S.F00
+          "Read the Feature dependent(TSTAT.S.F00(HEAT) & TSTAT.S.F02(OCC))
+          attribute in AttributeList"
+      PICS: TSTAT.S.F00 && TSTAT.S.F02
       command: "readAttribute"
       attribute: "AttributeList"
       response:
           constraints:
               type: list
-              contains: [18]
+              contains: [20]
 
     - label:
           "Read the Feature dependent(TSTAT.S.F01(COOL) & TSTAT.S.F02(OCC))
@@ -100,17 +110,6 @@ tests:
               type: list
               contains: [19]
 
-    - label:
-          "Read the Feature dependent(TSTAT.S.F00(HEAT) & TSTAT.S.F02(OCC))
-          attribute in AttributeList"
-      PICS: TSTAT.S.F00 && TSTAT.S.F02
-      command: "readAttribute"
-      attribute: "AttributeList"
-      response:
-          constraints:
-              type: list
-              contains: [20]
-
     - label:
           "Read the Feature dependent(TSTAT.S.F05(AUTO)) attribute in
           AttributeList"
@@ -120,7 +119,7 @@ tests:
       response:
           constraints:
               type: list
-              contains: [25]
+              contains: [17, 18, 25]
 
     - label:
           "Read the Feature dependent(TSTAT.S.F03(SCH)) attribute in
@@ -182,6 +181,7 @@ tests:
 
     #issue #15011 disabled steps below Global attributes missing from YAML framework
     - label: "Read the global attribute: EventList"
+      disabled: true
       verification: |
           Not implemented in chip-tool
       cluster: "LogCommands"