From bb107e97efc1b2d04ecb315d550b29d247c6a1c6 Mon Sep 17 00:00:00 2001 From: Cecille Freeman Date: Tue, 26 Nov 2024 21:25:47 -0500 Subject: [PATCH] Fix up test steps --- src/python_testing/TC_OPCREDS_3_5.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/python_testing/TC_OPCREDS_3_5.py b/src/python_testing/TC_OPCREDS_3_5.py index 81abe6e89b0249..65c622a17b0900 100644 --- a/src/python_testing/TC_OPCREDS_3_5.py +++ b/src/python_testing/TC_OPCREDS_3_5.py @@ -101,8 +101,8 @@ def steps_TC_OPCREDS_3_5(self): "Verify that DUT sends ArmFailSafeResponse with the ErrorCode set to OK"), TestStep(18, "TH1 reads the NOCs attribute from the Node Operational Credentials cluster using a fabric-filtered read", "Verify that the returned list has a single entry. Verify that the NOC field matches noc_update2 and the ICAC field matches icac_update2"), - # TODO: Gibran - Please rephrase using standard cleanup language add this to the test plan - TestStep(19, "remove fabric for TH2") + TestStep(19, "TH1 reads its fabric index from the CurrentFabricIndex attribute and saves as fabric_idx"), + TestStep(20, "TH0 sends the RemoveFabric command with the fabric index set to fabric_idx") ] @async_test_body @@ -238,6 +238,8 @@ async def test_TC_OPCREDS_3_5(self): self.step(19) fabric_idx = resp = await self.read_single_attribute_check_success(dev_ctrl=th1, node_id=self.dut_node_id, cluster=opcreds, attribute=opcreds.Attributes.CurrentFabricIndex) + + self.step(20) cmd = Clusters.OperationalCredentials.Commands.RemoveFabric(fabricIndex=fabric_idx) await self.send_single_cmd(dev_ctrl=self.default_controller, node_id=self.dut_node_id, cmd=cmd)