github/esphome
2
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-10-13 17:52:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

[ci] Reset sdkconfig after ESP-IDF component discovery (#20511)

Browse Source 
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Clyde Stubbs
2026-10-12 11:46:44 +13:00
committed by Jesse Hills
co-authored by Claude Opus 5.5
parent ec58e864f5
commit 41a9fca881
2 changed files with 37 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/unit_tests/test_espidf_toolchain.py
+29
View File
@@ -608,6 +608,35 @@ def test_run_compile_cache_miss_discovers_and_saves(setup_core: Path) -> None:
]
def test_run_compile_discovery_resets_sdkconfig(setup_core: Path) -> None:
"""Options the discovery configure forced on (e.g. esp-tls selecting TLS)
are dropped before the configure with the real component list."""
_setup_build(setup_core)
internal = CORE.relative_build_path("sdkconfig.test.esphomeinternal")
internal.parent.mkdir(parents=True)
internal.write_text("CONFIG_MBEDTLS_TLS_ENABLED=n\n")
sdkconfig = CORE.relative_build_path("sdkconfig.test")
seen: list[str] = []
def record_write(minimal: bool = False, builtin_components=None) -> None:
if minimal:
sdkconfig.write_text("CONFIG_MBEDTLS_TLS_ENABLED=y\n")
else:
seen.append(sdkconfig.read_text())
with (
patch.object(toolchain, "load_cached_builtin_components", return_value=None),
patch.object(toolchain, "save_cached_builtin_components"),
patch(
"esphome.build_gen.espidf.get_available_components", return_value=["lwip"]
),
patch("esphome.build_gen.espidf.write_project", side_effect=record_write),
patch.object(toolchain, "run_reconfigure", return_value=0),
):
assert toolchain._configure_project() == 0
assert seen == ["CONFIG_MBEDTLS_TLS_ENABLED=n\n"]
def test_run_compile_discovery_failure_stops_before_full_write(
setup_core: Path,
) -> None: