fix: Einstellungen liessen sich gar nicht speichern
In ha-dev brach jeder Versuch, den Abfrageabstand oder die HA-Adresse zu
sichern, mit einem 500er ab. Im Protokoll stand der Grund:
ValueError: Config entry update listeners should not be used with
OptionsFlowWithReload
Der Eintrag meldete zusaetzlich einen add_update_listener an, der beim
Aendern neu laedt - genau das macht OptionsFlowWithReload aber schon
selbst, und Home Assistant weist die Kombination rundheraus ab. Der
Listener faellt weg.
Die Tests in test_config_flow.py konnten das nicht sehen: dort ist
async_setup_entry durchweg ersetzt, der Eintrag wird also nie wirklich
aufgesetzt und meldet auch keinen Listener an. Der neue Test in
test_init.py laeuft am eingerichteten Eintrag und faellt, wenn man den
Listener wieder einbaut - gegengeprueft.
This commit is contained in:
+28
-1
@@ -6,11 +6,12 @@ from unittest.mock import MagicMock
|
||||
|
||||
from homeassistant.config_entries import ConfigEntryState
|
||||
from homeassistant.core import HomeAssistant
|
||||
from homeassistant.data_entry_flow import FlowResultType
|
||||
from homeassistant.helpers import device_registry as dr
|
||||
from pytest_homeassistant_custom_component.common import MockConfigEntry
|
||||
|
||||
from custom_components.ninstrip.api import NinstripAuthError, NinstripConnectionError
|
||||
from custom_components.ninstrip.const import DOMAIN
|
||||
from custom_components.ninstrip.const import CONF_DEVICE_BASE_URL, CONF_SCAN_INTERVAL, DOMAIN
|
||||
from tests_unit.samples import MAC_K1Y
|
||||
|
||||
|
||||
@@ -57,3 +58,29 @@ async def test_abgelehntes_token_verlangt_neue_anmeldung(
|
||||
assert any(
|
||||
flow["context"]["source"] == "reauth" for flow in hass.config_entries.flow.async_progress()
|
||||
)
|
||||
|
||||
|
||||
async def test_einstellungen_lassen_sich_am_laufenden_eintrag_sichern(
|
||||
hass: HomeAssistant, init_integration: MockConfigEntry
|
||||
) -> None:
|
||||
"""Am wirklich eingerichteten Eintrag - hier klemmte es.
|
||||
|
||||
Der Eintrag hatte zusaetzlich einen add_update_listener zum Neuladen, und
|
||||
zusammen mit einem OptionsFlowWithReload weist Home Assistant das
|
||||
Speichern rundheraus ab ("Config entry update listeners should not be used
|
||||
with OptionsFlowWithReload"). In ha-dev liess sich dadurch keine einzige
|
||||
Einstellung sichern. Die Tests in test_config_flow.py sehen das nicht:
|
||||
dort ist async_setup_entry durchweg ersetzt, der Eintrag wird also nie
|
||||
wirklich aufgesetzt und meldet auch keinen Listener an.
|
||||
"""
|
||||
ergebnis = await hass.config_entries.options.async_init(init_integration.entry_id)
|
||||
ergebnis = await hass.config_entries.options.async_configure(
|
||||
ergebnis["flow_id"],
|
||||
{CONF_SCAN_INTERVAL: 45, CONF_DEVICE_BASE_URL: "http://192.168.1.131:8124"},
|
||||
)
|
||||
await hass.async_block_till_done()
|
||||
assert ergebnis["type"] is FlowResultType.CREATE_ENTRY
|
||||
assert init_integration.options == {
|
||||
CONF_SCAN_INTERVAL: 45,
|
||||
CONF_DEVICE_BASE_URL: "http://192.168.1.131:8124",
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user