Merge branch 'esp8266-native-ninja-emission' into esp8266-arduino-toolchain

# Conflicts:
#	esphome/components/esp8266/__init__.py

tests/unit_tests/components/esp8266/test_boards.py

@@ -1,11 +1,7 @@
 """Tests for the per-board linker-script rule."""
 
-import pytest
-
 from esphome.components.esp8266 import _choose_ld_script
 from esphome.components.esp8266.boards import BOARDS, board_ld_script
-import esphome.config_validation as cv
-from esphome.core import EsphomeError
 
 
 def test_d1_wroom_02_keeps_its_shipped_layout() -> None:
@@ -21,13 +17,6 @@ def test_default_boards_use_the_flash_size_layout() -> None:
 
 
 def test_choose_ld_script_paths() -> None:
-    """Old cores get the size default, overriding boards hard-error there
-    (a substituted layout would wipe flash-backed state), modern cores
-    honor the override."""
-    assert _choose_ld_script("nodemcuv2", cv.Version(2, 3, 0)) is None
-    assert _choose_ld_script("nodemcuv2", cv.Version(2, 4, 2)) == "eagle.flash.4m.ld"
-    assert _choose_ld_script("d1_wroom_02", cv.Version(2, 7, 4)) == (
-        "eagle.flash.2m64.ld"
-    )
-    with pytest.raises(EsphomeError, match="cannot honor"):
-        _choose_ld_script("d1_wroom_02", cv.Version(2, 4, 2))
+    """Default boards get the size layout, overriding boards keep theirs."""
+    assert _choose_ld_script("nodemcuv2") == "eagle.flash.4m.ld"
+    assert _choose_ld_script("d1_wroom_02") == "eagle.flash.2m64.ld"

tests/unit_tests/components/esp8266/test_framework_version.py

@@ -0,0 +1,23 @@
+"""Tests for the Arduino framework version floor."""
+
+import pytest
+
+from esphome.components.esp8266 import _arduino_check_versions
+import esphome.config_validation as cv
+from esphome.const import CONF_PLATFORM_VERSION, CONF_VERSION
+
+
+def test_versions_before_3_are_rejected() -> None:
+    with pytest.raises(cv.Invalid, match="no longer supported") as excinfo:
+        _arduino_check_versions({CONF_VERSION: "2.7.4"})
+    assert excinfo.value.path == [CONF_VERSION]
+
+
+def test_supported_versions_pass() -> None:
+    value = _arduino_check_versions({CONF_VERSION: "3.0.2"})
+    assert value[CONF_VERSION] == "3.0.2"
+    assert "espressif8266@3.2.0" in value[CONF_PLATFORM_VERSION]
+
+    value = _arduino_check_versions({CONF_VERSION: "recommended"})
+    assert value[CONF_VERSION] == "3.1.2"
+    assert "espressif8266@4.2.1" in value[CONF_PLATFORM_VERSION]

tests/unit_tests/components/esp8266/test_toolchain_validation.py

@@ -63,8 +63,10 @@ def test_valid_config_passes() -> None:
 
 
 def test_platformio_toolchain_skips_checks() -> None:
+    # 3.0.2 is pio-legal (>= the global 3.0.0 floor) but below the native
+    # toolchain's own 3.1.1 floor; the bogus board only the native path checks
     CORE.toolchain = Toolchain.PLATFORMIO
-    config = _config(board="not_a_board", version="2.7.4")
+    config = _config(board="not_a_board", version="3.0.2")
     assert _validate_native_toolchain(config) is config