fix(glances): cache dedicated httpx client and add timeout regression test

Address PR review feedback:

- Cache the per-verify_ssl httpx client in hass.data so config entry
  reloads reuse the same client instead of leaking a new one (and a new
  EVENT_HOMEASSISTANT_CLOSE handler) on every reload.
- Add a regression test that asserts setup constructs the client with
  the configured DEFAULT_TIMEOUT and that a reload hits the cache
  (no second client created).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: serl

homeassistant/components/glances/__init__.py

@@ -9,6 +9,7 @@
     GlancesApiError,
     GlancesApiNoDataAvailable,
 )
+import httpx
 
 from homeassistant.const import (
     CONF_HOST,
@@ -19,18 +20,23 @@
     CONF_VERIFY_SSL,
     Platform,
 )
-from homeassistant.core import HomeAssistant
+from homeassistant.core import HomeAssistant, callback
 from homeassistant.exceptions import (
     ConfigEntryAuthFailed,
     ConfigEntryError,
     ConfigEntryNotReady,
     HomeAssistantError,
 )
 from homeassistant.helpers.httpx_client import create_async_httpx_client
+from homeassistant.util.hass_dict import HassKey
 
 from .const import DEFAULT_TIMEOUT
 from .coordinator import GlancesConfigEntry, GlancesDataUpdateCoordinator
 
+DATA_HTTPX_CLIENT: HassKey[dict[bool, httpx.AsyncClient]] = HassKey(
+    "glances_httpx_client"
+)
+
 PLATFORMS = [Platform.SENSOR]
 
 
@@ -64,11 +70,25 @@ async def async_unload_entry(hass: HomeAssistant, entry: GlancesConfigEntry) ->
     return await hass.config_entries.async_unload_platforms(entry, PLATFORMS)
 
 
+@callback
+def _async_get_httpx_client(hass: HomeAssistant, verify_ssl: bool) -> httpx.AsyncClient:
+    """Return a cached Glances httpx client (one per verify_ssl value).
+
+    The shared httpx client cannot be used because it has a 5-second timeout
+    that is too short for slow Glances hosts. Caching here ensures entry
+    reloads reuse the same client instead of leaking one on every reload.
+    """
+    clients = hass.data.setdefault(DATA_HTTPX_CLIENT, {})
+    if (client := clients.get(verify_ssl)) is None:
+        client = clients[verify_ssl] = create_async_httpx_client(
+            hass, verify_ssl=verify_ssl, timeout=DEFAULT_TIMEOUT
+        )
+    return client
+
+
 async def get_api(hass: HomeAssistant, entry_data: dict[str, Any]) -> Glances:
     """Return the api from glances_api."""
-    httpx_client = create_async_httpx_client(
-        hass, verify_ssl=entry_data[CONF_VERIFY_SSL], timeout=DEFAULT_TIMEOUT
-    )
+    httpx_client = _async_get_httpx_client(hass, entry_data[CONF_VERIFY_SSL])
     for version in (4, 3):
         api = Glances(
             host=entry_data[CONF_HOST],

tests/components/glances/test_init.py

@@ -1,6 +1,6 @@
 """Tests for Glances integration."""
 
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, patch
 
 from freezegun.api import FrozenDateTimeFactory
 from glances_api.exceptions import (
@@ -10,7 +10,11 @@
 )
 import pytest
 
-from homeassistant.components.glances.const import DEFAULT_SCAN_INTERVAL, DOMAIN
+from homeassistant.components.glances.const import (
+    DEFAULT_SCAN_INTERVAL,
+    DEFAULT_TIMEOUT,
+    DOMAIN,
+)
 from homeassistant.config_entries import ConfigEntryState
 from homeassistant.core import HomeAssistant
 from homeassistant.helpers.update_coordinator import UpdateFailed
@@ -83,6 +87,30 @@ async def test_update_error_includes_message(
     )
 
 
+async def test_dedicated_httpx_client_uses_timeout_and_is_cached(
+    hass: HomeAssistant,
+) -> None:
+    """The integration's dedicated httpx client uses DEFAULT_TIMEOUT and is reused on reload."""
+    entry = MockConfigEntry(domain=DOMAIN, data=MOCK_USER_INPUT)
+    entry.add_to_hass(hass)
+
+    with patch(
+        "homeassistant.components.glances.create_async_httpx_client"
+    ) as mock_create:
+        await hass.config_entries.async_setup(entry.entry_id)
+        await hass.async_block_till_done()
+        assert entry.state is ConfigEntryState.LOADED
+
+        assert mock_create.call_count == 1
+        kwargs = mock_create.call_args.kwargs
+        assert kwargs["timeout"] == DEFAULT_TIMEOUT
+        assert kwargs["verify_ssl"] == MOCK_USER_INPUT["verify_ssl"]
+
+        assert await hass.config_entries.async_reload(entry.entry_id)
+        await hass.async_block_till_done()
+        assert mock_create.call_count == 1
+
+
 async def test_unload_entry(hass: HomeAssistant) -> None:
     """Test removing Glances."""
     entry = MockConfigEntry(domain=DOMAIN, data=MOCK_USER_INPUT)