test(e2e): drop redundant lifecycle roundtrips, keep only Matter Server (#1414) (#1419)

Closes #1414.

The HAOS addon-lifecycle tier had four near-identical
``test_<addon>_start_stop_restart_roundtrip`` tests (Node-RED, ESPHome,
Matter Server, AppDaemon). All four used the same ``_addon_action``
helper hitting the same MCP wire path —
``ha_call_service(hassio.addon_{start,stop,restart}, data={"addon": slug})`` —
so they were four instances of one test parametrised by addon. From a
code-coverage standpoint one is sufficient to guard the wire path.

Node-RED's roundtrip in particular produced a recurring Supervisor 500
``Server got itself in trouble`` flake on the first ``addon_stop`` call
— four reproductions in <90 min across unrelated PRs, with reruns of
the same commit consistently passing. Node-RED's container teardown
(full Node.js runtime) is slow enough that Supervisor's addon_stop
handler races to a 500 before the response body is built.

Dropping three of the four roundtrips:

* Removes the recurring Node-RED flake at source
* Saves ~37-50s per HAOS lane (Node-RED ~25s, AppDaemon ~14s, ESPHome
  <13s) — based on pytest's slowest-10 across 15 recent runs
* Preserves full addon-lifecycle wire coverage via the Matter Server
  roundtrip, which has never been observed flaking and runs <13s

Matter Server kept because it's the lightest addon to cycle in the
bake; AppDaemon (~14s, sometimes slowest-10) dropped along with the
others.

Addon-specific surface coverage is unchanged for every addon:
``ha_get_addon`` info/options, ``ha_manage_addon`` options writes,
``ha_get_logs`` Supervisor logs, and Node-RED's proxy/ingress tests
(``test_manage_addon_modes.py``) all continue to run against the four
running addons.

Co-authored-by: kingpanther13 <kingpanther13@users.noreply.github.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: kingpanther13

tests/src/e2e/haos_only/test_addon_lifecycle.py

@@ -7,10 +7,17 @@
 Two coverage modes:
 
 1. **Running addons** (Node-RED, ESPHome Device Builder, Matter Server,
-   AppDaemon — all ``start=True`` in the bake): full lifecycle round-trip
-   plus options-get / options-set persistence and a log-shape check.
-   Each test leaves the addon in ``started`` state via an explicit
-   cleanup call so later tests in the same session find it running.
+   AppDaemon — all ``start=True`` in the bake): options-get / options-set
+   persistence and a log-shape check. Each test leaves the addon in
+   ``started`` state via an explicit cleanup call so later tests in the
+   same session find it running.
+
+   The full ``hassio.addon_stop`` → ``addon_start`` → ``addon_restart``
+   lifecycle round-trip is exercised by **Matter Server only**. All four
+   roundtrip variants used the same ``_addon_action`` helper hitting the
+   same MCP wire path, so one is sufficient signal; Matter Server is the
+   lightest addon to cycle (consistently <13s) and Node-RED's roundtrip
+   produced a recurring Supervisor 500 flake — see #1414.
 
 2. **Stopped addons** (Mosquitto, MQTT IO — stay ``start=False`` because
    their schemas require config they don't have in the bake): exercise
@@ -87,9 +94,7 @@ async def _resolve_slug(mcp_client: Any, display_name: str) -> str:
     for entry in payload.get("addons", []):
         if entry.get("name") == display_name:
             slug = entry.get("slug")
-            assert slug, (
-                f"Addon {display_name!r} listed but has no slug field: {entry}"
-            )
+            assert slug, f"Addon {display_name!r} listed but has no slug field: {entry}"
             return str(slug)
 
     # Filter ``None`` names so a future Supervisor addition with a
@@ -116,9 +121,7 @@ async def _get_addon_detail(mcp_client: Any, slug: str) -> dict[str, Any]:
     return detail
 
 
-async def _addon_action(
-    mcp_client: Any, slug: str, action: str
-) -> dict[str, Any]:
+async def _addon_action(mcp_client: Any, slug: str, action: str) -> dict[str, Any]:
     """Invoke ``hassio.addon_{action}`` via ``ha_call_service``.
 
     Returns the parsed result (success or failure). Caller decides how
@@ -209,36 +212,6 @@ async def _ensure_started(mcp_client: Any, slug: str) -> None:
 # ---------------------------------------------------------------------------
 
 
-async def test_nodered_start_stop_restart_roundtrip(mcp_client: Any) -> None:
-    """Stop → start → restart cycles Node-RED via real Supervisor.
-
-    Each transition is verified by re-reading ``ha_get_addon(slug=...)``
-    and asserting the state moved as expected. Final assertion plus the
-    cleanup hook leave the addon ``started`` for downstream tests.
-    """
-    slug = await _resolve_slug(mcp_client, NODERED_NAME)
-    try:
-        stop_result = await _addon_action(mcp_client, slug, "stop")
-        assert stop_result.get("success"), (
-            f"hassio.addon_stop({slug}) failed: {stop_result}"
-        )
-        await _wait_for_state(mcp_client, slug, STOPPED_STATES)
-
-        start_result = await _addon_action(mcp_client, slug, "start")
-        assert start_result.get("success"), (
-            f"hassio.addon_start({slug}) failed: {start_result}"
-        )
-        await _wait_for_state(mcp_client, slug, "started")
-
-        restart_result = await _addon_action(mcp_client, slug, "restart")
-        assert restart_result.get("success"), (
-            f"hassio.addon_restart({slug}) failed: {restart_result}"
-        )
-        await _wait_for_state(mcp_client, slug, "started")
-    finally:
-        await _ensure_started(mcp_client, slug)
-
-
 async def test_nodered_options_get_returns_dict(mcp_client: Any) -> None:
     """`ha_get_addon(slug=...)` exposes the addon's options as a dict."""
     slug = await _resolve_slug(mcp_client, NODERED_NAME)
@@ -295,9 +268,7 @@ async def test_nodered_options_set_persists(mcp_client: Any) -> None:
             ok = write_payload.get("success") is True or (
                 write_payload.get("status") == "pending_restart"
             )
-            assert ok, (
-                f"ha_manage_addon options probe write failed: {write_payload}"
-            )
+            assert ok, f"ha_manage_addon options probe write failed: {write_payload}"
 
             detail_after = await _get_addon_detail(mcp_client, slug)
             options_after = detail_after.get("options") or {}
@@ -344,31 +315,6 @@ async def test_nodered_logs_fetch_shape(mcp_client: Any) -> None:
 # ---------------------------------------------------------------------------
 
 
-async def test_esphome_start_stop_restart_roundtrip(mcp_client: Any) -> None:
-    """Stop → start → restart cycles ESPHome via real Supervisor."""
-    slug = await _resolve_slug(mcp_client, ESPHOME_NAME)
-    try:
-        stop_result = await _addon_action(mcp_client, slug, "stop")
-        assert stop_result.get("success"), (
-            f"hassio.addon_stop({slug}) failed: {stop_result}"
-        )
-        await _wait_for_state(mcp_client, slug, STOPPED_STATES)
-
-        start_result = await _addon_action(mcp_client, slug, "start")
-        assert start_result.get("success"), (
-            f"hassio.addon_start({slug}) failed: {start_result}"
-        )
-        await _wait_for_state(mcp_client, slug, "started")
-
-        restart_result = await _addon_action(mcp_client, slug, "restart")
-        assert restart_result.get("success"), (
-            f"hassio.addon_restart({slug}) failed: {restart_result}"
-        )
-        await _wait_for_state(mcp_client, slug, "started")
-    finally:
-        await _ensure_started(mcp_client, slug)
-
-
 async def test_esphome_options_get_returns_dict(mcp_client: Any) -> None:
     """`ha_get_addon(slug=...)` exposes ESPHome's options as a dict."""
     slug = await _resolve_slug(mcp_client, ESPHOME_NAME)
@@ -417,9 +363,7 @@ async def test_esphome_options_set_persists(mcp_client: Any) -> None:
             ok = write_payload.get("success") is True or (
                 write_payload.get("status") == "pending_restart"
             )
-            assert ok, (
-                f"ha_manage_addon options probe write failed: {write_payload}"
-            )
+            assert ok, f"ha_manage_addon options probe write failed: {write_payload}"
 
             detail_after = await _get_addon_detail(mcp_client, slug)
             options_after = detail_after.get("options") or {}
@@ -540,9 +484,7 @@ async def test_matter_server_options_set_persists(mcp_client: Any) -> None:
             ok = write_payload.get("success") is True or (
                 write_payload.get("status") == "pending_restart"
             )
-            assert ok, (
-                f"ha_manage_addon options probe write failed: {write_payload}"
-            )
+            assert ok, f"ha_manage_addon options probe write failed: {write_payload}"
 
             detail_after = await _get_addon_detail(mcp_client, slug)
             options_after = detail_after.get("options") or {}
@@ -582,31 +524,6 @@ async def test_matter_server_logs_fetch_shape(mcp_client: Any) -> None:
 # ---------------------------------------------------------------------------
 
 
-async def test_appdaemon_start_stop_restart_roundtrip(mcp_client: Any) -> None:
-    """Stop → start → restart cycles AppDaemon via real Supervisor."""
-    slug = await _resolve_slug(mcp_client, APPDAEMON_NAME)
-    try:
-        stop_result = await _addon_action(mcp_client, slug, "stop")
-        assert stop_result.get("success"), (
-            f"hassio.addon_stop({slug}) failed: {stop_result}"
-        )
-        await _wait_for_state(mcp_client, slug, STOPPED_STATES)
-
-        start_result = await _addon_action(mcp_client, slug, "start")
-        assert start_result.get("success"), (
-            f"hassio.addon_start({slug}) failed: {start_result}"
-        )
-        await _wait_for_state(mcp_client, slug, "started")
-
-        restart_result = await _addon_action(mcp_client, slug, "restart")
-        assert restart_result.get("success"), (
-            f"hassio.addon_restart({slug}) failed: {restart_result}"
-        )
-        await _wait_for_state(mcp_client, slug, "started")
-    finally:
-        await _ensure_started(mcp_client, slug)
-
-
 async def test_appdaemon_webui_no_ingress_shape(mcp_client: Any) -> None:
     """`ha_get_addon(slug=...)` reports ``webui`` set + ``ingress=false`` for AppDaemon.