TBRM Tests scripts consistency with te2 fixes

examples/chip-tool/py_matter_chip_tool_adapter/matter_chip_tool_adapter/encoder.py

@@ -130,6 +130,14 @@
                 'has_destination': False,
                 'has_endpoint': False,
             },
+            'EstablishPASESession': {
+                'alias': 'code-paseonly',
+                'arguments': {
+                    'nodeId': 'node-id'
+                },
+                'has_destination': False,
+                'has_endpoint': False,
+            },
             'GetCommissionerNodeId': {
                 'has_destination': False,
                 'has_endpoint': False,

scripts/py_matter_yamltests/matter_yamltests/pseudo_clusters/clusters/commissioner_commands.py

@@ -56,6 +56,11 @@
       <arg name="RCAC" type="octet_string"/>
       <arg name="IPK" type="octet_string"/>
     </command>
+
+    <command source="client" code="5" name="EstablishPASESession">
+      <arg name="nodeId" type="node_id"/>
+      <arg name="payload" type="char_string"/>
+    </command>
 
 </cluster>
 </configurator>

src/app/tests/suites/certification/Test_TC_TBRM_2_2.yaml

@@ -41,19 +41,21 @@ tests:
           values:
               - name: nodeId
                 value: nodeId
-
+    # Step 1
     - label: "TH reads the ActiveDatasetTimestamp attribute from the DUT"
       command: readAttribute
       attribute: ActiveDatasetTimestamp
       response:
           value: null
 
+    # Step 2
     - label: "TH reads the PendingDatasetTimestamp attribute from the DUT"
       command: readAttribute
       attribute: PendingDatasetTimestamp
       response:
           value: null
 
+    # Step 3
     - label:
           "TH sends a valid ActiveDatasetRequest command to the DUT without
           having armed the fail-safe"
@@ -65,6 +67,7 @@ tests:
       response:
           error: FAILSAFE_REQUIRED
 
+    # Step 4
     - label: "TH sends ArmFailSafe command to the DUT"
       cluster: General Commissioning
       command: ArmFailSafe
@@ -76,6 +79,7 @@ tests:
               - name: Breadcrumb
                 value: 1
 
+    # Step 5
     - label: "TH sends an invalid ActiveDatasetRequest command to the DUT"
       command: SetActiveDatasetRequest
       arguments:
@@ -85,19 +89,32 @@ tests:
       response:
           error: INVALID_COMMAND
 
+    # Step 6
     - label: "TH sends a valid ActiveDatasetRequest command to the DUT"
       command: SetActiveDatasetRequest
       arguments:
           values:
               - name: ActiveDataset
                 value: PIXIT.TBRM.THREAD_ACTIVE_DATASET
 
+    # Step 7
+    - label: "TH sends CommissioningComplete command to the DUT"
+      cluster: General Commissioning
+      command: CommissioningComplete
+      endpoint: 0
+      response:
+          values:
+              - name: "ErrorCode"
+                value: 0 # SUCCESS
+
+    # Step 8
     - label: "TH reads the InterfaceEnabled attribute from the DUT"
       command: readAttribute
       attribute: InterfaceEnabled
       response:
           value: true
 
+    # Step 9
     - label: "TH reads the ActiveDatasetTimestamp attribute from the DUT"
       command: readAttribute
       attribute: ActiveDatasetTimestamp
@@ -106,7 +123,8 @@ tests:
           constraints:
               type: int64u
 
-    - label: "TH sends a valid GetActiveDatasetRequest command to the DUT"
+    # Step 10
+    - label: "TH send a GetActiveDatasetRequest command to the DUT"
       command: GetActiveDatasetRequest
       response:
           values:

src/app/tests/suites/certification/Test_TC_TBRM_2_3.yaml

@@ -49,7 +49,7 @@ tests:
           constraints:
               type: int64u
 
-    - label: "If the ActiveDatasetTimestamp attribute not null, go to step 4"
+    - label: "If the ActiveDatasetTimestamp attribute not null, go to step 5"
       cluster: EqualityCommands
       command: UnsignedNumberEquals
       arguments:
@@ -86,6 +86,17 @@ tests:
                 value: PIXIT.TBRM.THREAD_ACTIVE_DATASET
 
     # Step 4
+    - label: "TH sends CommissioningComplete command to the DUT"
+      runIf: noActiveDataset
+      cluster: General Commissioning
+      command: CommissioningComplete
+      endpoint: 0
+      response:
+          values:
+              - name: "ErrorCode"
+                value: 0 # SUCCESS
+
+    # Step 5
     - label: "TH reads the PendingDatasetTimestamp attribute from the DUT"
       command: readAttribute
       attribute: PendingDatasetTimestamp
@@ -94,15 +105,15 @@ tests:
           constraints:
               type: int64u
 
-    # Step 5
+    # Step 6
     - label: "TH sends a SetPendingDatasetRequest command to the DUT"
       command: SetPendingDatasetRequest
       arguments:
           values:
               - name: PendingDataset
                 value: PIXIT.TBRM.THREAD_PENDING_DATASET
 
-    # Step 6
+    # Step 7
     - label: "TH sends a GetPendingDatasetRequest command to the DUT"
       command: GetPendingDatasetRequest
       response:
@@ -112,7 +123,7 @@ tests:
                     type: octet_string
                     # TODO: This should be PIXIT.TBRM.THREAD_PENDING_DATASET but ignoring the Delay Timer element if present
 
-    # Step 7
+    # Step 8
     - label: "TH reads the PendingDatasetTimestamp attribute from the DUT"
       command: readAttribute
       attribute: PendingDatasetTimestamp
@@ -121,7 +132,7 @@ tests:
               type: int64u
               notValue: initialPendingTimestamp
 
-    # Step 8
+    # Step 9
     - label:
           "TH subscribes to the ActiveDatasetTimestamp attribute from the DUT"
       command: subscribeAttribute
@@ -141,15 +152,15 @@ tests:
           constraints:
               hasValue: true
 
-    # Step 9
+    # Step 10
     - label: "TH reads the PendingDatasetTimestamp attribute from the DUT"
       command: readAttribute
       attribute: PendingDatasetTimestamp
       response:
           value: null
 
-    # Step 10
-    - label: "TH sends a valid GetActiveDatasetRequest command to the DUT"
+    # Step 11
+    - label: "TH sends a GetActiveDatasetRequest command to the DUT"
       command: GetActiveDatasetRequest
       response:
           values:
@@ -158,7 +169,7 @@ tests:
                 constraints:
                     type: octet_string
 
-    # Step 11
+    # Step 12
     - label: "TH reads the InterfaceEnabled attribute from the DUT"
       command: readAttribute
       attribute: InterfaceEnabled

src/app/tests/suites/certification/Test_TC_TBRM_2_4.yaml

@@ -0,0 +1,41 @@
+# Copyright (c) 2024 Project CHIP Authors
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+name: "[TC-TBRM-2.4] Verify that getting Active or Pending Dataset in the PASE session results in unsupported access"
+
+PICS:
+    - TBRM.S
+
+config:
+    nodeId: 0x12344321
+    payload: MT:Y.K9042C00KA0648G00
+
+tests:
+    - label: "TH EstablishPASESession"
+      cluster: CommissionerCommands
+      endpoint: 0
+      command: EstablishPASESession
+      arguments:
+          values:
+              - name: nodeId
+                value: nodeId
+              - name: payload
+                value: payload
+
+    - label: "TH send GetActiveDatasetRequest command to the DUT in PASE session"
+      cluster: Thread Border Router Management
+      endpoint: 1
+      command: GetActiveDatasetRequest
+      response:
+          error: UNSUPPORTED_ACCESS

src/app/tests/suites/certification/Test_TC_TBRM_3_1_Simulated.yaml

@@ -0,0 +1,44 @@
+# Copyright (c) 2024 Project CHIP Authors
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+name: "[TC-TBRM-3.1] Functionality with client as DUT"
+
+PICS:
+    - TBRM.C
+
+config:
+    nodeId: 0x12344321
+    cluster: Thread Border Router Management
+    endpoint: 1
+
+tests:
+    # Step 1
+    - label: "DUT send SetActiveDatasetRequest to TH"
+      PICS: TBRM.C.C03.Tx
+      wait: "SetActiveDatasetRequest"
+
+    # Step 2
+    - label: "DUT send SetPendingDatasetRequest to TH"
+      PICS: TBRM.C.C04.Tx
+      wait: "SetPendingDatasetRequest"
+
+    # Step 3
+    - label: "DUT send GetActiveDatasetRequest to TH"
+      PICS: TBRM.C.C00.Tx
+      wait: "GetActiveDatasetRequest"
+
+    # Step 4
+    - label: "DUT send GetPendingDatasetRequest to TH"
+      PICS: TBRM.C.C01.Tx
+      wait: "GetPendingDatasetRequest"