Make BRBINFO_4_1 require commissioning (#35017)

* Make BRVINFO_4_1 require commissioning

* Also add icd registration

---------

Co-authored-by: Andrei Litvin <[email protected]>

src/python_testing/TC_BRBINFO_4_1.py

@@ -57,7 +57,8 @@ def desc_TC_BRBINFO_4_1(self) -> str:
 
     def steps_TC_BRBINFO_4_1(self) -> list[TestStep]:
         steps = [
-            TestStep("0", "Preconditions"),
+            TestStep("0",  "DUT commissioned", is_commissioning=True),
+            TestStep("0a", "Preconditions"),
             TestStep("1a", "TH reads from the ICD the A_IDLE_MODE_DURATION, A_ACTIVE_MODE_DURATION, and ACTIVE_MODE_THRESHOLD attributes"),
             TestStep("1b", "Simple KeepActive command w/ subscription. ActiveChanged event received by TH contains PromisedActiveDuration"),
             TestStep("2", "Sends 3x KeepActive commands w/ subscription. ActiveChanged event received ONCE and contains PromisedActiveDuration"),
@@ -73,7 +74,7 @@ def _ask_for_vendor_commissioniong_ux_operation(self, discriminator, setupPinCod
             f"- setupQRCode: {setupQRCode}\n"
             f"- setupManualcode: {setupManualCode}\n"
             f"If using FabricSync Admin test app, you may type:\n"
-            f">>> pairing onnetwork 111 {setupPinCode}")
+            f">>> pairing onnetwork 111 {setupPinCode} --icd-registration true")
 
     async def _send_keep_active_command(self, duration, endpoint_id) -> int:
         logging.info("Sending keep active command")
@@ -164,9 +165,11 @@ async def test_TC_BRBINFO_4_1(self):
         dynamic_endpoint_id = await self._get_dynamic_endpoint()
         logging.info(f"Dynamic endpoint is {dynamic_endpoint_id}")
 
-        # Preconditions
         self.step("0")
 
+        # Preconditions
+        self.step("0a")
+
         logging.info("Ensuring DUT is commissioned to TH")
 
         # Confirms commissioning of DUT on TH as it reads its fature map