[wifi] Tighten the LN882x roaming comment and test

This commit is contained in:
J. Nick Koston
2026-10-04 20:00:56 -05:00
parent 3314e1953a
commit eb1d73fea3
2 changed files with 3 additions and 4 deletions
+1 -3
View File
@@ -542,9 +542,7 @@ CONFIG_SCHEMA = cv.All(
),
cv.Optional(CONF_PASSIVE_SCAN, default=False): cv.boolean,
cv.Optional(CONF_ENABLE_ON_BOOT, default=True): cv.boolean,
# An LN882H scan takes the radio off the AP long enough to drop
# the association, so the post-connect roam check costs a
# disconnect there.
# An LN882H scan drops the association, so roaming costs a disconnect there.
cv.SplitDefault(
CONF_POST_CONNECT_ROAMING,
esp8266=True,
@@ -23,6 +23,7 @@ def test_default_values_are_not_emitted(
assert "set_reboot_timeout(" not in main_cpp
assert "set_power_save_mode(" not in main_cpp
assert "set_min_auth_mode(" not in main_cpp
assert "set_post_connect_roaming(" not in main_cpp
def test_custom_values_are_emitted(
@@ -43,7 +44,7 @@ def test_ln882x_skips_post_connect_roaming(
generate_main: Callable[[str | Path], str],
component_config_path: Callable[[str], Path],
) -> None:
"""An LN882H scan drops the association, so LN882x does not roam by default."""
"""LN882x defaults post_connect_roaming off."""
main_cpp = generate_main(component_config_path("bare_ln882x.yaml"))
assert "set_post_connect_roaming(false);" in main_cpp