[rp2_ble_tracker] Use register_apply_action for stop_scan (#19524)

esphome/components/rp2_ble_tracker/__init__.py

@@ -42,7 +42,6 @@ RP2BLETracker = rp2_ble_tracker_ns.class_(
 )
 
 StartScanAction = rp2_ble_tracker_ns.class_("StartScanAction", automation.Action)
-StopScanAction = rp2_ble_tracker_ns.class_("StopScanAction", automation.Action)
 
 ESPBTAdvertiseTrigger = ble_automation.ESPBTAdvertiseTrigger
 BLEServiceDataAdvertiseTrigger = ble_automation.BLEServiceDataAdvertiseTrigger
@@ -156,9 +155,8 @@ async def start_scan_action_to_code(
     return var
 
 
-@automation.register_action(
+automation.register_apply_action(
     "rp2_ble_tracker.stop_scan",
-    StopScanAction,
     automation.maybe_simple_id(
         cv.Schema(
             {
@@ -166,14 +164,5 @@ async def start_scan_action_to_code(
             }
         )
     ),
-    synchronous=True,
+    automation.ApplyCall("stop_scan()"),
 )
-async def stop_scan_action_to_code(
-    config: ConfigType,
-    action_id: ID,
-    template_arg: cg.TemplateArguments,
-    args: list,
-) -> cg.MockObj:
-    var = cg.new_Pvariable(action_id, template_arg)
-    await cg.register_parented(var, config[CONF_ID])
-    return var

esphome/components/rp2_ble_tracker/automation.h

@@ -37,11 +37,6 @@ template<typename... Ts> class StartScanAction final : public Action<Ts...>, pub
   }
 };
 
-template<typename... Ts> class StopScanAction final : public Action<Ts...>, public Parented<RP2BLETracker> {
- public:
-  void play(const Ts &...x) override { this->parent_->stop_scan(); }
-};
-
 }  // namespace esphome::rp2_ble_tracker
 
 #endif  // USE_RP2

tests/component_tests/rp2_ble_tracker/test_automations_codegen.py

@@ -37,12 +37,13 @@ def test_trigger_codegen(
         "set_manufacturer_uuid128((uint8_t*)(const uint8_t[16]){0xCD,0xAB,0xCD,0xAB,"
         "0xCD,0xAB,0xCD,0xAB,0xCD,0xAB,0xCD,0xAB,0xCD,0xAB,0xCD,0xAB})" in main_cpp
     )
-    # scan-control actions: templatable continuous lambda + parented actions.
+    # scan-control actions: templatable continuous lambda + stop_scan forwarded
+    # straight to the tracker (register_apply_action, no action class).
     # Exactly one set_continuous: the bare start_scan emits none, pinning the
     # restore-configured-mode divergence from esp32 against a future default=.
     assert main_cpp.count("->set_continuous(") == 1
     assert "startscanaction_id->set_continuous(" in main_cpp
-    assert "stopscanaction_id->set_parent(" in main_cpp
+    assert main_cpp.count("->stop_scan();") == 1
     # scan_parameters continuous: false reaches the YAML-mode setter, not the
     # runtime override.
     assert "->set_configured_continuous(false)" in main_cpp