wiliot-certificate 4.4.3__py3-none-any.whl → 4.5.0a1__py3-none-any.whl

certificate/tests/energy2400/signal_indicator_ble5_test/signal_indicator_ble5_test.py

@@ -0,0 +1,378 @@
+from certificate.cert_prints import *
+from certificate.cert_defines import *
+from certificate.wlt_types import *
+import certificate.cert_mqtt as cert_mqtt
+import certificate.cert_common as cert_common
+import certificate.cert_config as cert_config
+
+# Test Description:
+#   This test is to verify the functionality of both signal indicator tx (tx_brg) and rx (rx_brg) at BRG level.
+#   We will configure several signal indicator params during the test, and check the functionality of the signal indicator logic
+#   for each of them.
+#   It is important to execute the test with several setups: 2 Fanstel BRG's, 2 Minew BRG's and 1 Fanstel and 1 Minew BRG.
+#   At first, we will configure several tx signal indicator params and check for ack's, to verify all indicated params were
+#   received at the cloud.
+#   Then, we will examine the signal indicator end-2-end logic with both transmitter and receiver:
+#   phase 1 - One BRG will be configured as signal indicator tx, and the other as signal indicator rx, and we expect to see
+#   signal indicator packets only from the tx BRG, and according to the tx params (to check the repetition and cycle params).
+#   phase 2 - Same as phase 1, but with different tx params configured.
+#   phase 3 - One rx BRG without any tx BRG. We don't expect to see any signal indicator packets. This phase is to verify the
+#   brg module logic is working properly, and no tag packet is accidentally being treated as signal indicator packet.
+#   phase 4 - Both BRG's will be configured to be transmitters and receivers, with different tx params for each one. we expect
+#   to see signal indicator packets from both BRG's, according to the tx params.
+#   phase 5 - One BRG will be configured as signal indicator tx, but no rx, so we don't expect to receive signal indicatopr packets.
+#   that way we can assure the logic within the receiver is not confused by the signal indicator uuid as external sensor.
+
+
+# Test MACROS #
+NUM_OF_SCANNING_CYCLE = 5
+DEFAULT_SCAN_TIME = 30
+SCAN_DELAY_TIME = 3
+BLE5_MAX_DURATION_SEC = ag.BLE5_PARAM_PRIMARY_CHANNEL_SCAN_CYCLE // 1000 + 1
+
+
+def test_rssi_threshold(test):
+    cycle, rep = 5, 4
+    dut = cert_config.get_brg_by_target(test, DUT)
+    tester = cert_config.get_brg_by_target(test, TESTER)
+    rssi_threshold = -25
+
+    # configuring receiver #
+    utPrint(f"Configuring DUT BRG {dut.id_str} as Signal Indicator Receiver with RSSI Threshold of {rssi_threshold}", "BOLD")
+    test = cert_config.brg_configure(test=test, module=dut.sensors,
+                                     fields=[BRG_SENSOR0, BRG_RSSI_THRESHOLD],
+                                     values=[ag.EXTERNAL_SENSORS_SIGNAL_INDICATOR, rssi_threshold],
+                                     ble5=test.dut_is_bridge(), wait=test.dut_is_combo())[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason(f"BRG {dut.id_str}: didn't receive signal indicator receiver configuration!")
+        return test
+    if test.dut_is_bridge():
+        wait_time_n_print(BLE5_MAX_DURATION_SEC)  # BLE5 configuration can take up to BLE5_MAX_DURATION_SEC
+    # configuring tester as a transmitter #
+    utPrint(f"Configuring TESTER BRG {tester.id_str} as Signal Indicator Transmitter", "BOLD")
+    test = cert_config.brg_configure(test=test, module=tester.energy2400, fields=[BRG_SIGNAL_INDICATOR_CYCLE, BRG_SIGNAL_INDICATOR_REP],
+                                     values=[cycle, rep], target=TESTER)[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason(f"BRG {tester.id_str}: didn't receive signal indicator transmitter configuration!")
+        return test
+    utPrint(f"BRG {tester.id_str} configured to be transmitter - cycle = {cycle} repetition = {rep}", "BOLD")
+    # phase analysis #
+    cert_mqtt.mqtt_scan_n_create_log_file(test, (NUM_OF_SCANNING_CYCLE * cycle) + SCAN_DELAY_TIME, "rssi_threshold")
+    rec_sig_ind_pkts = cert_common.get_all_sig_ind_pkts(test=test, rx_brg=dut, tx_brg=tester)
+    for p in rec_sig_ind_pkts:
+        print(f"rssi value: {p[UNIFIED_SENSOR_PKT].pkt.rssi}")
+    rssi_threshold_viloation_pkts = [p for p in rec_sig_ind_pkts if p[UNIFIED_SENSOR_PKT].pkt.rssi >= -1 * rssi_threshold]
+    if rssi_threshold_viloation_pkts:
+        test.rc = TEST_FAILED
+        test.add_reason(f"rssi_threshold phase failed - BRG {dut.id_str} echoed"
+                        f" {len(rssi_threshold_viloation_pkts)} signal indicator packets\n with RSSI weaker than {rssi_threshold}")
+    return test
+
+
+def test_rx(test):
+
+    dut = cert_config.get_brg_by_target(test, DUT)
+    tester = cert_config.get_brg_by_target(test, TESTER)
+    cycle, rep = 8, 4
+
+    utPrint(f"TESTER BRG with cycle = {cycle}, repetition = {rep}\n", "BLUE")
+    # configuring receiver #
+    utPrint(f"Configuring DUT BRG {dut.id_str} as Signal Indicator Receiver", "BOLD")
+    test = cert_config.brg_configure(test=test, module=dut.sensors, fields=[BRG_SENSOR0],
+                                     values=[ag.EXTERNAL_SENSORS_SIGNAL_INDICATOR],
+                                     ble5=test.dut_is_bridge(), wait=test.dut_is_combo(),
+                                     target=DUT)[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason(f"BRG {dut.id_str}: didn't receive signal indicator receiver configuration!")
+        return test
+    if test.dut_is_bridge():
+        wait_time_n_print(BLE5_MAX_DURATION_SEC)  # BLE5 configuration can take up to BLE5_MAX_DURATION_SEC
+
+    # configuring transmitter #
+    utPrint(f"Configuring TESTER BRG {tester.id_str} as Signal Indicator Transmitter", "BOLD")
+    test = cert_config.brg_configure(test=test, module=tester.energy2400,
+                                     fields=[BRG_SIGNAL_INDICATOR_CYCLE, BRG_SIGNAL_INDICATOR_REP],
+                                     values=[cycle, rep], target=TESTER)[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason(f"BRG {tester.id_str}: didn't receive signal indicator transmitter configuration!")
+        return test
+    utPrint(f"TESTER {tester.id_str} configured to be transmitter - cycle={cycle}, repetition={rep}", "BOLD")
+
+    # phase analysis #
+    cert_mqtt.mqtt_scan_n_create_log_file(test, (NUM_OF_SCANNING_CYCLE * cycle) + SCAN_DELAY_TIME, f"rx_{cycle}_{rep}")
+    rec_sig_ind_pkts = cert_common.get_all_sig_ind_pkts(test=test, rx_brg=dut, tx_brg=tester)
+
+    if cert_common.sig_ind_pkts_fail_analysis(tx_brg=tester, rx_brg=dut, cycles=NUM_OF_SCANNING_CYCLE, received_pkts=rec_sig_ind_pkts):
+        test.rc = TEST_FAILED
+        expected_signal_ind_pkts = cert_common.exp_sig_ind_pkts2(tx_brg=tester, rx_brg=dut, cycles=NUM_OF_SCANNING_CYCLE)
+        test.add_reason(f"rx phase failed - BRG {dut.id_str} received wrong number of "
+                        f"signal indicator packets\nreceived {len(rec_sig_ind_pkts)} packets, "
+                        f"expected {expected_signal_ind_pkts} packets")
+        print(rec_sig_ind_pkts)  # TODO: logging print(debug)
+        print([[p[TIMESTAMP], p[UNIFIED_SENSOR_PKT].pkt.signal_indicator_payload.rx_antenna] for p in rec_sig_ind_pkts])
+
+    test = cert_common.rx_tx_antenna_check(test, rec_sig_ind_pkts, tester, dut, NUM_OF_SCANNING_CYCLE)
+    test = cert_common.output_power_check(test, rec_sig_ind_pkts, tester)
+    test = cert_common.rssi_check(test, rec_sig_ind_pkts)
+
+    return test
+
+
+def test_tx(test):
+
+    dut = cert_config.get_brg_by_target(test, DUT)
+    tester = cert_config.get_brg_by_target(test, TESTER)
+    cycle, rep = 8, 4
+
+    # configuring transmitter #
+    utPrint(f"Configuring DUT BRG {dut.id_str} as Signal Indicator Transmitter (cycle={cycle}, repetition={rep})", "BOLD")
+    test = cert_config.brg_configure(test=test, module=dut.energy2400,
+                                     fields=[BRG_SIGNAL_INDICATOR_CYCLE, BRG_SIGNAL_INDICATOR_REP],
+                                     values=[cycle, rep], ble5=test.dut_is_bridge(), wait=test.dut_is_combo())[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason(f"DUT BRG {dut.id_str}: didn't receive signal indicator transmitter configuration!")
+        return test
+    if test.dut_is_bridge():
+        wait_time_n_print(BLE5_MAX_DURATION_SEC)  # BLE5 configuration can take up to BLE5_MAX_DURATION_SEC
+
+    # configuring receiver #
+    utPrint(f"Configuring TESTER BRG {tester.id_str} as Signal Indicator Receiver", "BOLD")
+    test = cert_config.brg_configure(test=test, module=tester.sensors, fields=[BRG_SENSOR0],
+                                     values=[ag.EXTERNAL_SENSORS_SIGNAL_INDICATOR], target=TESTER)[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason(f"TESTER BRG {tester.id_str}: didn't receive signal indicator receiver configuration!")
+        return test
+
+    # phase analysis #
+    cert_mqtt.mqtt_scan_n_create_log_file(test, (NUM_OF_SCANNING_CYCLE * cycle) + SCAN_DELAY_TIME,
+                                          f"tx_{cycle}_{rep}")
+    rec_sig_ind_pkts = cert_common.get_all_sig_ind_pkts(test=test, rx_brg=tester, tx_brg=dut)
+    if cert_common.sig_ind_pkts_fail_analysis(tx_brg=dut, rx_brg=tester, cycles=NUM_OF_SCANNING_CYCLE, received_pkts=rec_sig_ind_pkts):
+        test.rc = TEST_FAILED
+        expected_signal_ind_pkts = cert_common.exp_sig_ind_pkts2(tester, dut, NUM_OF_SCANNING_CYCLE)
+        test.add_reason(f"tx phase failed - BRG {dut.id_str} received wrong number of "
+                        f"signal indicator packets\nreceived {len(rec_sig_ind_pkts)} packets, "
+                        f"expected {expected_signal_ind_pkts} packets")
+        print(rec_sig_ind_pkts)  # TODO: logging print(debug)
+        print([[p[TIMESTAMP], p[UNIFIED_SENSOR_PKT].pkt.signal_indicator_payload.rx_antenna] for p in rec_sig_ind_pkts])
+
+    test = cert_common.rx_tx_antenna_check(test, rec_sig_ind_pkts, dut, tester, NUM_OF_SCANNING_CYCLE)
+    test = cert_common.output_power_check(test, rec_sig_ind_pkts, dut)
+    test = cert_common.rssi_check(test, rec_sig_ind_pkts)
+
+    return test
+
+
+def test_disable_tx(test):
+    cycle, rep = ag.BRG_DEFAULT_SIGNAL_INDICATOR_CYCLE, ag.BRG_DEFAULT_SIGNAL_INDICATOR_REP
+    tester = cert_config.get_brg_by_target(test, TESTER)
+    dut = cert_config.get_brg_by_target(test, DUT)
+    utPrint(f"DUT BRG with TX disabled - cycle = {cycle}, repetition = {rep}\n", "BLUE")
+    # configuring tester as a receiver, the dut should be with defaults which means TX is disabled
+    utPrint(f"Configuring TESTER BRG {tester.id_str} as Signal Indicator Receiver", "BOLD")
+    test = cert_config.brg_configure(test=test, module=tester.sensors,
+                                     fields=[BRG_SENSOR0],
+                                     values=[ag.EXTERNAL_SENSORS_SIGNAL_INDICATOR],
+                                     target=TESTER)[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason(f"BRG {tester.id_str}: didn't receive signal indicator receiver configuration!")
+        return test
+    utPrint(f"TESTER BRG {tester.id_str} successfully configured as Signal Indicator Receiver\n", "BOLD")
+
+    # phase analysis #
+    cert_mqtt.mqtt_scan_n_create_log_file(test, DEFAULT_SCAN_TIME, "disable_tx")
+    expected_signal_ind_pkts = [0]
+    rec_sig_ind_pkts = cert_common.get_all_sig_ind_pkts(test=test, rx_brg=tester, tx_brg=dut)
+    if len(rec_sig_ind_pkts) not in expected_signal_ind_pkts:
+        test.rc = TEST_FAILED
+        test.add_reason(f"disable_tx phase failed - DUT BRG {dut.id_str} sent signal indicator packets\n"
+                        f"received {len(rec_sig_ind_pkts)} packets, "
+                        f"expected {expected_signal_ind_pkts} packets")
+
+    return test
+
+
+def test_disable_rx(test):
+    # DUT BRG without rx. just waiting for packets to be sent from the transmitter and verify
+    # The receiver does not receive any signal indicator packets.
+    cycle, rep = 4, 3
+    dut = cert_config.get_brg_by_target(test, DUT)
+    tester = cert_config.get_brg_by_target(test, TESTER)
+    utPrint("DUT BRG with RX disabled\n", "BLUE")
+    # configuring tester as transmitter, dut should be with defaults which means rx disabled
+    utPrint(f"Configuring TESTER BRG {tester.id_str} as Signal Indicator Transmitter", "BOLD")
+    test = cert_config.brg_configure(test=test, module=tester.energy2400,
+                                     fields=[BRG_SIGNAL_INDICATOR_CYCLE, BRG_SIGNAL_INDICATOR_REP],
+                                     values=[cycle, rep], target=TESTER)[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason(f"TESTER BRG {tester.id_str}: didn't receive signal indicator transmitter configuration!")
+        return test
+
+    # phase analysis #
+    cert_mqtt.mqtt_scan_n_create_log_file(test, (NUM_OF_SCANNING_CYCLE * cycle) + SCAN_DELAY_TIME, "disable_rx")
+    expected_signal_ind_pkts = [0]
+    rec_sig_ind_pkts = cert_common.get_all_sig_ind_pkts(test=test, rx_brg=dut, tx_brg=tester)
+    if len(rec_sig_ind_pkts) not in expected_signal_ind_pkts:
+        test.rc = TEST_FAILED
+        test.add_reason(f"disable_rx phase failed - DUT {dut.id_str} received signal indicator packets\n"
+                        f"received {len(rec_sig_ind_pkts)} packets, expected 0 packets")
+
+    return test
+
+
+def test_rx_tx(test):
+    tx_cycle, tx_rep = 5, 4
+    rx_cycle, rx_rep = 5, 4
+    dut = cert_config.get_brg_by_target(test, DUT)
+    tester = cert_config.get_brg_by_target(test, TESTER)
+    utPrint("Both TESTER and DUT are transmitters and receivers\n", "BLUE")
+    # configuring dut brg as receiver
+    utPrint(f"Configuring DUT BRG {dut.id_str} as Signal Indicator Receiver", "BOLD")
+    test = cert_config.brg_configure(test=test, module=dut.sensors, fields=[BRG_SENSOR0],
+                                     values=[ag.EXTERNAL_SENSORS_SIGNAL_INDICATOR],
+                                     ble5=test.dut_is_bridge(), wait=test.dut_is_combo())[0]
+    if test.rc == TEST_FAILED and test.exit_on_param_failure:
+        test.add_reason(f"BRG {dut.id_str}: didn't receive signal indicator receiver configuration!")
+        return test
+    if test.dut_is_bridge():
+        wait_time_n_print(BLE5_MAX_DURATION_SEC)  # BLE5 configuration can take up to BLE5_MAX_DURATION_SEC
+    utPrint(f"BRG {dut.id_str} successfully configured as Signal Indicator Receiver\n", "BOLD")
+    # configuring dut brg as transmitter
+    utPrint(f"Configuring DUT BRG {dut.id_str} as Signal Indicator Transmitter", "BOLD")
+    test = cert_config.brg_configure(test=test, module=dut.energy2400, fields=[BRG_SIGNAL_INDICATOR_CYCLE, BRG_SIGNAL_INDICATOR_REP],
+                                     values=[tx_cycle, tx_rep], ble5=test.dut_is_bridge(), wait=test.dut_is_combo())[0]
+    if test.rc == TEST_FAILED and test.exit_on_param_failure:
+        test.add_reason(f"BRG {dut.id_str}: didn't receive signal indicator transmitter configuration!")
+        return test
+    if test.dut_is_bridge():
+        wait_time_n_print(BLE5_MAX_DURATION_SEC)  # BLE5 configuration can take up to BLE5_MAX_DURATION_SEC
+    utPrint(f"BRG {dut.id_str} configured to be transmitter - cycle={tx_cycle}, repetition={tx_rep}", "BOLD")
+
+    # configuring tester brg as receiver
+    utPrint(f"Configuring TESTER BRG {tester.id_str} as Signal Indicator Receiver", "BOLD")
+    test = cert_config.brg_configure(test=test, module=tester.sensors,
+                                     fields=[BRG_SENSOR0],
+                                     values=[ag.EXTERNAL_SENSORS_SIGNAL_INDICATOR],
+                                     target=TESTER)[0]
+    if test.rc == TEST_FAILED and test.exit_on_param_failure:
+        test.add_reason(f"BRG {tester.id_str}: didn't receive signal indicator receiver configuration!")
+        return test
+    utPrint(f"BRG {tester.id_str} successfully configured as Signal Indicator Receiver\n", "BOLD")
+
+    # configuring tester brg as transmitter (already configured as rx)
+    utPrint(f"Configuring TESTER BRG {tester.id_str} as Signal Indicator Transmitter", "BOLD")
+    test = cert_config.brg_configure(test=test, module=tester.energy2400,
+                                     fields=[BRG_SIGNAL_INDICATOR_CYCLE, BRG_SIGNAL_INDICATOR_REP],
+                                     values=[rx_cycle, rx_rep],
+                                     target=TESTER)[0]
+    if test.rc == TEST_FAILED and test.exit_on_param_failure:
+        test.add_reason(f"BRG {tester.id_str}: didn't receive signal indicator transmitter configuration!")
+        return test
+    utPrint(f"BRG {tester.id_str} configured to be transmitter - cycle={rx_cycle}, repetition={rx_rep}")
+
+    # phase analysis #
+    cert_mqtt.mqtt_scan_n_create_log_file(test, NUM_OF_SCANNING_CYCLE * max(tx_cycle, rx_cycle) + SCAN_DELAY_TIME, "rx_tx")
+
+    # Analyzing dut performance as receiver
+    utPrint(f"Analyzing DUT {dut.id_str} performance as a Receiver\n", "BOLD")
+    rec_sig_ind_pkts = cert_common.get_all_sig_ind_pkts(test=test, rx_brg=dut, tx_brg=tester)
+    if cert_common.sig_ind_pkts_fail_analysis(tx_brg=tester, rx_brg=dut, cycles=NUM_OF_SCANNING_CYCLE, received_pkts=rec_sig_ind_pkts):
+        test.rc = TEST_FAILED
+        expected_signal_ind_pkts = cert_common.exp_sig_ind_pkts2(dut, tester, NUM_OF_SCANNING_CYCLE)
+        test.add_reason(f"rx_tx phase failed - BRG {dut.id_str} received wrong number of "
+                        f"signal indicator packets\nreceived {len(rec_sig_ind_pkts)} packets, "
+                        f"expected {expected_signal_ind_pkts} packets")
+        print(rec_sig_ind_pkts)  # TODO: logging print(debug)
+
+    # Analyzing tester performance as receiver
+    utPrint(f"Analyzing DUT {tester.id_str} performance as a Transmitter\n", "BOLD")
+    rec_sig_ind_pkts = cert_common.get_all_sig_ind_pkts(test=test, rx_brg=tester, tx_brg=dut)
+    if cert_common.sig_ind_pkts_fail_analysis(tx_brg=dut, rx_brg=tester, cycles=NUM_OF_SCANNING_CYCLE, received_pkts=rec_sig_ind_pkts):
+        test.rc = TEST_FAILED
+        expected_signal_ind_pkts = cert_common.exp_sig_ind_pkts2(dut, tester, NUM_OF_SCANNING_CYCLE)
+        test.add_reason(f"rx_tx phase failed - BRG {tester.id_str} received wrong number of "
+                        f"signal indicator packets\n received {len(rec_sig_ind_pkts)} packets, "
+                        f"expected {expected_signal_ind_pkts} packets")
+        print(rec_sig_ind_pkts)  # TODO: logging print(debug)
+    # NOTE: We skipped the antenna and output power checks for this phase
+    return test
+
+
+SIGNAL_INDICATOR_TEST_MAP = {"rssi_threshold": test_rssi_threshold,
+                             "rx": test_rx, "tx": test_tx, "rx_tx": test_rx_tx,
+                             "disable_tx": test_disable_tx, "disable_rx": test_disable_rx}
+
+
+def run(test):
+    # Test prolog
+    test = cert_common.test_prolog(test)
+
+    dut = cert_config.get_brg_by_target(test, DUT)
+    tester = cert_config.get_brg_by_target(test, TESTER)
+
+    # Configure DUT to scan ble5 channel during all the test, if dut is a bridge only,
+    # configure the pattern to extended advertising for beeing able to receive ble5 packets on the tester
+    if test.dut_is_bridge():
+        fields = [BRG_RX_CHANNEL, BRG_PATTERN]
+        values = [ag.RX_CHANNEL_10_500K, ag.DATAPATH_PATTERN_EXTENDED_ADV_CH_10]
+    else:
+        fields = [BRG_RX_CHANNEL]
+        values = [ag.RX_CHANNEL_10_500K]
+    test = cert_config.brg_configure(test, fields=fields, values=values,
+                                     module=dut.datapath, ble5=test.dut_is_bridge(), wait=test.dut_is_combo())[0]
+    if test.rc == TEST_FAILED:
+        return cert_common.test_epilog(test, revert_brgs=True,
+                                       modules=[dut.datapath],
+                                       ble5=test.dut_is_bridge())
+    # Configure TESTER to scan ble5 channel during all the test
+    test = cert_config.brg_configure(test, fields=[BRG_RX_CHANNEL],
+                                     values=[ag.RX_CHANNEL_10_500K],
+                                     module=tester.datapath, target=TESTER)[0]
+    if test.rc == TEST_FAILED:
+        return cert_common.test_epilog(test, revert_brgs=True, modules=[dut.datapath],
+                                       brg1_modules=[tester.datapath], ble5=test.dut_is_bridge())
+    for param in test.params:
+        phase_run_print(param.name)
+        test = SIGNAL_INDICATOR_TEST_MAP[param.value](test)
+        cert_mqtt.generate_log_file(test, param.name)
+        field_functionality_pass_fail_print(test, param.name)
+        test.set_phase_rc(param.name, test.rc)
+        test.add_phase_reason(param.name, test.reason)
+        if test.rc == TEST_FAILED and test.exit_on_param_failure:
+            break
+        else:
+            test.reset_result()
+            # Reset DUT to defaults after every phase (don't fail the phase on that)
+            test = cert_config.config_brg_defaults(test,
+                                                   modules=[dut.energy2400, dut.sensors],
+                                                   ble5=test.dut_is_bridge(), wait=test.dut_is_combo())[0]
+        if test.dut_is_bridge():
+            # BLE5 configuration can take up to BLE5_MAX_DURATION_SEC, we configured 2 modules here
+            wait_time_n_print(2 * BLE5_MAX_DURATION_SEC)
+        if test.rc == TEST_FAILED:
+            test.add_reason("Failed to restore DUT to defaults")
+        else:
+            # Reset TESTER to defaults after every phase (don't fail the phase on that)
+            test = cert_config.config_brg_defaults(test,
+                                                   modules=[tester.energy2400, tester.sensors],
+                                                   target=TESTER)[0]
+            if test.rc == TEST_FAILED:
+                test.add_reason("Failed to restore brg0 to defaults")
+        if test.rc == TEST_FAILED:
+            test.set_phase_rc(param.name, test.rc)
+            test.add_phase_reason(param.name, test.reason)
+            if test.exit_on_param_failure:
+                break
+        else:
+            test.reset_result()
+
+    # Revert DUT to defaults here and not in epilog
+    test = cert_config.config_brg_defaults(test, modules=[dut.datapath], ble5=test.dut_is_bridge(), wait=test.dut_is_combo())[0]
+    if test.rc == TEST_FAILED:
+        test.add_reason("Failed to revert brg0 datapath to defaults")
+    else:
+        test = cert_config.config_brg_defaults(test, modules=[tester.datapath], target=TESTER)[0]
+        if test.rc == TEST_FAILED:
+            test.add_reason("Failed to revert brg1 datapath to defaults")
+
+    return cert_common.test_epilog(test, revert_brgs=False)

brg_certificate/tests/energy2400/signal_indicator_ble5_10_500k_test/signal_indicator_ble5_10_500k_test.json → certificate/tests/energy2400/signal_indicator_ext_adv_test/signal_indicator_ext_adv_test.json

@@ -1,7 +1,7 @@
 {
-    "name": "Signal Indicator BLE5 E4 Pixels",
+    "name": "Signal Indicator Extended Advertising",
     "module": "Energizer 2.4GHz",
-    "purpose": "verify Energizer 2.4GHz signal indicator functionality for BLE_5.0 in CHANNEL_10 for E4 pixels modulation",
+    "purpose": "verify Signal Indicator functionality when using bridge with extended advertising pattern",
     "documentation": "https://community.wiliot.com/customers/s/article/Bridge-to-Bridge-Signal-Indicator",
     "initialCondition": "Bridge configured to defaults",
     "procedure": ["Test prolog",
@@ -15,6 +15,6 @@
     "mandatory": 0,
     "multiBridgeTest": 1,
     "gwOnlyTest": 0,
-    "internalBridge": 1,
+    "dataSimOnlyTest": 0,
     "allSupportedValues": ["rssi_threshold", "brg0_tx_brg1_rx", "brg0_none_brg1_rx", "brg0_rxtx_brg1_rxtx", "brg0_tx_brg1_none"]
 }