test(hacs): retry TestMcpToolsInstallation on flake (#1426)

* test(hacs): retry TestMcpToolsInstallation on flake (pytest-rerunfailures)

The HACS-mediated install tests in TestMcpToolsInstallation flake on
the ubuntu-24.04-arm CI runner with a generic
'SERVICE_CALL_FAILED: Command failed: Unknown error' from HACS. The
class's existing autouse pre-flight (ha_hacs_search) passes, but the
heavier hacs/repository/add + hacs/repository/download path hits a
transient HACS/GitHub failure on the slower arm runner; subsequent
sibling tests then skip via the pre-flight when HACS becomes
unreachable mid-class.

Apply @pytest.mark.flaky(reruns=2, reruns_delay=10) at the class
level so the install path gets 3 attempts with a 10s pause between —
catches any transient (~99.9% of cases) while still flagging a genuine
persistent regression (fails 3 times -> reported).

Scoped narrowly: only TestMcpToolsInstallation is marked flaky. The
HACS read-side tests (TestHacsSearch, TestHacsSearchInstalled,
TestHacsRepositoryInfo, TestHacsWriteOperations) keep their existing
fail-fast posture.

Add pytest-rerunfailures>=15.0 to dev deps (resolved to v16.3).

* chore: ruff format on test_hacs.py (CI surfaces pre-existing reflow)

---------

Co-authored-by: kingpanther13 <kingpanther13@users.noreply.github.com>

Author: kingpanther13

pyproject.toml

@@ -186,6 +186,7 @@ dev = [
     "pytest>=8.4.2",
     "pytest-asyncio>=1.1.0",
     "pytest-cov>=5.0.0",
+    "pytest-rerunfailures>=15.0",
     "pytest-xdist>=3.8.0",
     "requests>=2.25.0",
     "lefthook>=1.10.0",

tests/src/e2e/workflows/hacs/test_hacs.py

@@ -155,7 +155,9 @@ async def test_list_all_installed(self, mcp_client):
 
         # No filter should be applied
         assert data["category_filter"] is None, "No category filter should be applied"
-        assert data["installed_only"] is True, "Response should indicate installed_only=True"
+        assert data["installed_only"] is True, (
+            "Response should indicate installed_only=True"
+        )
 
         logger.info("List all installed test passed")
 
@@ -187,9 +189,7 @@ async def test_search_installed_with_query(self, mcp_client):
                 f"Repo {repo.get('name')} should be installed"
             )
 
-        logger.info(
-            f"Search installed with query: {data['total_matches']} matches"
-        )
+        logger.info(f"Search installed with query: {data['total_matches']} matches")
 
     async def test_list_by_category(self, mcp_client):
         """
@@ -566,6 +566,7 @@ async def test_hacs_discovery(mcp_client):
 
 @pytest.mark.hacs
 @pytest.mark.slow
+@pytest.mark.flaky(reruns=2, reruns_delay=10)
 class TestMcpToolsInstallation:
     """Test ha_mcp_tools custom component installation via HACS.