TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOpe…

…ration to be TRUE or FALSE (#28523) * TC-DRLK-2.2/2.3/2.12: Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE - Modify flow to allow for RequirePINforRemoteOperation to be TRUE or FALSE - Reorder steps 7-9 to ensure subsequent test of WrongCodeEntryLimit doesn't already have existing failed entry attempts - Fix incorrect PICS (use .Write PICS) for writing WrongCodeEntryLimit and UserCodeTemporaryDisableTime Fixes #3263 Fixes #3265 Fixes #3266 * Restyled by autopep8 * Exclude .yaml tests from CI due to use of EqualityTests pseudo-cluster * Add zap-regen'd files * Exclude .yaml tests from CI linux due to use of EqualityTests pseudo-cluster --------- Co-authored-by: Restyled.io <[email protected]>
project-chip · Aug 10, 2023 · 1b6ea10 · 1b6ea10
1 parent cdcadfa
commit 1b6ea10
Show file tree

Hide file tree

Showing 7 changed files with 2,541 additions and 8,134 deletions.
diff --git a/scripts/tests/chiptest/__init__.py b/scripts/tests/chiptest/__init__.py
@@ -170,6 +170,9 @@ def _GetChipReplUnsupportedTests() -> Set[str]:
         "Test_TC_DRLK_2_8.yaml",   # Test fails only in chip-repl: Refer--> https://github.com/project-chip/connectedhomeip/pull/27011#issuecomment-1593339855
         "Test_TC_ACE_1_6.yaml",    # Test fails only in chip-repl: Refer--> https://github.com/project-chip/connectedhomeip/pull/27910#issuecomment-1632485584
         "Test_TC_IDM_1_2.yaml",              # chip-repl does not support AnyCommands (19/07/2023)
+        "Test_TC_DRLK_2_2.yaml",             # chip-repl does not support EqualityCommands pseudo-cluster (08/04/2023)
+        "Test_TC_DRLK_2_3.yaml",             # chip-repl does not support EqualityCommands pseudo-cluster (08/04/2023)
+        "Test_TC_DRLK_2_12.yaml",            # chip-repl does not support EqualityCommands pseudo-cluster (08/04/2023)
         "TestGroupKeyManagementCluster.yaml",  # chip-repl does not support EqualityCommands (2023-08-04)
         "Test_TC_S_2_2.yaml",              # chip-repl does not support scenes cluster commands
         "Test_TC_S_2_3.yaml",              # chip-repl does not support scenes cluster commands

diff --git a/src/app/tests/suites/certification/Test_TC_DRLK_2_12.yaml b/src/app/tests/suites/certification/Test_TC_DRLK_2_12.yaml
@@ -21,6 +21,9 @@ config:
     nodeId: 0x12344321
     cluster: "Door Lock"
     endpoint: 1
+    requirePINIsTrue:
+        type: BOOLEAN
+        defaultValue: false
 
 tests:
     - label: "Wait for the commissioned device to be retrieved"
@@ -152,13 +155,61 @@ tests:
       command: "readAttribute"
       attribute: "RequirePINforRemoteOperation"
       response:
-          value: false
+          saveAs: requirePINIsTrue
+          constraints:
+              type: boolean
+              minValue: 0
+              maxValue: 1
+
+    - label:
+          "Step 2a: TH verifies the RequirePINforRemoteOperation attribute value
+          is now False on the DUT"
+      PICS: DRLK.S.F07 && DRLK.S.F00 && DRLK.S.A0033.Write
+      cluster: "EqualityCommands"
+      command: "BooleanEquals"
+      arguments:
+          values:
+              - name: "Value1"
+                value: requirePINIsTrue
+              - name: "Value2"
+                value: false
+      response:
+          - values:
+                - name: "Equals"
+                  value: true
+
+    - label: "Step 2b: Compute RequirePINforRemoteOperation"
+      cluster: "EqualityCommands"
+      command: "BooleanEquals"
+      arguments:
+          values:
+              - name: "Value1"
+                value: requirePINIsTrue
+              - name: "Value2"
+                value: false
+      response:
+          - values:
+                - name: "Equals"
+                  saveAs: requirePINIsFalse
+                  constraints:
+                      type: boolean
+                      minValue: 0
+                      maxValue: 1
 
     - label: "Step 3: TH sends Ubolt Door Command to the DUT without PINCode"
+      runIf: requirePINIsFalse
       PICS: DRLK.S.C27.Rsp
       command: "UnboltDoor"
       timedInteractionTimeoutMs: 1000
 
+    - label: "Step 3: TH sends Ubolt Door Command to the DUT without PINCode"
+      runIf: requirePINIsTrue
+      PICS: DRLK.S.C27.Rsp
+      command: "UnboltDoor"
+      timedInteractionTimeoutMs: 1000
+      response:
+          error: FAILURE
+
     - label: "Step 4: TH sends Ubolt Door Command to the DUT with PINCode"
       PICS: DRLK.S.C27.Rsp
       command: "UnboltDoor"
@@ -195,19 +246,49 @@ tests:
       command: "readAttribute"
       attribute: "RequirePINforRemoteOperation"
       response:
-          value: true
+          saveAs: requirePINIsTrue
+          constraints:
+              type: boolean
+              minValue: 0
+              maxValue: 1
 
-    - label: "Step 7: TH sends Ubolt Door Command to the DUT with valid PINCode"
-      PICS: DRLK.S.C27.Rsp
-      command: "UnboltDoor"
-      timedInteractionTimeoutMs: 1000
+    - label:
+          "Step 6a: TH verifies the RequirePINforRemoteOperation attribute value
+          is now True on the DUT"
+      PICS: DRLK.S.F07 && DRLK.S.F00 && DRLK.S.A0033.Write
+      cluster: "EqualityCommands"
+      command: "BooleanEquals"
       arguments:
           values:
-              - name: "PINCode"
-                value: "123456"
+              - name: "Value1"
+                value: requirePINIsTrue
+              - name: "Value2"
+                value: true
+      response:
+          - values:
+                - name: "Equals"
+                  value: true
+
+    - label: "Step 6b: Compute RequirePINforRemoteOperation"
+      cluster: "EqualityCommands"
+      command: "BooleanEquals"
+      arguments:
+          values:
+              - name: "Value1"
+                value: requirePINIsTrue
+              - name: "Value2"
+                value: false
+      response:
+          - values:
+                - name: "Equals"
+                  saveAs: requirePINIsFalse
+                  constraints:
+                      type: boolean
+                      minValue: 0
+                      maxValue: 1
 
     - label:
-          "Step 8: TH sends Unbolt Door Command to the DUT with Invalid PINCode"
+          "Step 7: TH sends Unbolt Door Command to the DUT with Invalid PINCode"
       PICS: DRLK.S.C27.Rsp
       command: "UnboltDoor"
       timedInteractionTimeoutMs: 1000
@@ -218,13 +299,29 @@ tests:
       response:
           error: FAILURE
 
-    - label: "Step 9: TH sends Unbolt Door Command to the DUT without PINCode"
+    - label: "Step 8: TH sends Unbolt Door Command to the DUT without PINCode"
+      runIf: requirePINIsTrue
       PICS: DRLK.S.C27.Rsp
       command: "UnboltDoor"
       timedInteractionTimeoutMs: 1000
       response:
           error: FAILURE
 
+    - label: "Step 8: TH sends Unbolt Door Command to the DUT without PINCode"
+      runIf: requirePINIsFalse
+      PICS: DRLK.S.C27.Rsp
+      command: "UnboltDoor"
+      timedInteractionTimeoutMs: 1000
+
+    - label: "Step 9: TH sends Ubolt Door Command to the DUT with valid PINCode"
+      PICS: DRLK.S.C27.Rsp
+      command: "UnboltDoor"
+      timedInteractionTimeoutMs: 1000
+      arguments:
+          values:
+              - name: "PINCode"
+                value: "123456"
+
     - label:
           "Step 10: TH writes WrongCodeEntryLimit attribute value as 3 on the
           DUT"