From ea169e2cb4232e3b4c7f3003234e8fb14ffc060e Mon Sep 17 00:00:00 2001 From: morris Date: Fri, 2 Jan 2026 23:41:41 +0800 Subject: [PATCH] refactor(tests): add missing sdkconfig files in the driver test --- .../esp_driver_i2s/test_apps/lp_i2s/pytest_lp_i2s.py | 7 +++++++ .../{sdkconfig.ci.defaults => sdkconfig.ci.default} | 0 components/esp_driver_sdio/src/sdio_slave.c | 2 +- .../temperature_sensor/pytest_temperature_sensor.py | 9 ++++++++- .../spi_flash/test_apps/esp_flash/pytest_esp_flash.py | 4 ++-- 5 files changed, 18 insertions(+), 4 deletions(-) rename components/esp_driver_i2s/test_apps/lp_i2s/{sdkconfig.ci.defaults => sdkconfig.ci.default} (100%) diff --git a/components/esp_driver_i2s/test_apps/lp_i2s/pytest_lp_i2s.py b/components/esp_driver_i2s/test_apps/lp_i2s/pytest_lp_i2s.py index 2c7e4ffa8a..25b21b0be9 100644 --- a/components/esp_driver_i2s/test_apps/lp_i2s/pytest_lp_i2s.py +++ b/components/esp_driver_i2s/test_apps/lp_i2s/pytest_lp_i2s.py @@ -6,5 +6,12 @@ from pytest_embedded import Dut @pytest.mark.esp32p4 @pytest.mark.lp_i2s +@pytest.mark.parametrize( + 'config', + [ + 'default', + ], + indirect=True, +) def test_lp_i2s(dut: Dut) -> None: dut.run_all_single_board_cases(timeout=200) diff --git a/components/esp_driver_i2s/test_apps/lp_i2s/sdkconfig.ci.defaults b/components/esp_driver_i2s/test_apps/lp_i2s/sdkconfig.ci.default similarity index 100% rename from components/esp_driver_i2s/test_apps/lp_i2s/sdkconfig.ci.defaults rename to components/esp_driver_i2s/test_apps/lp_i2s/sdkconfig.ci.default diff --git a/components/esp_driver_sdio/src/sdio_slave.c b/components/esp_driver_sdio/src/sdio_slave.c index 779f0ec109..18a46c338a 100644 --- a/components/esp_driver_sdio/src/sdio_slave.c +++ b/components/esp_driver_sdio/src/sdio_slave.c @@ -613,7 +613,7 @@ static void sdio_intr_send(void *arg) esp_err_t sdio_slave_send_queue(uint8_t *addr, size_t len, void *arg, TickType_t wait) { - SDIO_SLAVE_CHECK(len > 0, "len <= 0", ESP_ERR_INVALID_ARG); + SDIO_SLAVE_CHECK(len > 0 && len <= 4092, "length out of range: (0, 4092]", ESP_ERR_INVALID_ARG); SDIO_SLAVE_CHECK(esp_ptr_dma_capable(addr) && (uint32_t)addr % 4 == 0, "buffer to send should be DMA capable and 32-bit aligned", ESP_ERR_INVALID_ARG); diff --git a/components/esp_driver_tsens/test_apps/temperature_sensor/pytest_temperature_sensor.py b/components/esp_driver_tsens/test_apps/temperature_sensor/pytest_temperature_sensor.py index d6e2808a97..998cb08097 100644 --- a/components/esp_driver_tsens/test_apps/temperature_sensor/pytest_temperature_sensor.py +++ b/components/esp_driver_tsens/test_apps/temperature_sensor/pytest_temperature_sensor.py @@ -40,7 +40,14 @@ def test_temperature_sensor_cbs(dut: Dut) -> None: @pytest.mark.esp32c6 @pytest.mark.two_duts @pytest.mark.parametrize('count', [2], indirect=True) +@pytest.mark.parametrize( + 'config', + [ + 'release', + ], + indirect=True, +) def test_temperature_phy_cases(case_tester: CaseTester) -> None: # type: ignore for case in case_tester.test_menu: if case.attributes.get('test_env', 'two_duts') == 'two_duts': - case_tester.run_all_multi_dev_cases(case=case, reset=True) + case_tester.run_multi_dev_case(case=case, reset=True) diff --git a/components/spi_flash/test_apps/esp_flash/pytest_esp_flash.py b/components/spi_flash/test_apps/esp_flash/pytest_esp_flash.py index 8d1ae43715..c4db18d729 100644 --- a/components/spi_flash/test_apps/esp_flash/pytest_esp_flash.py +++ b/components/spi_flash/test_apps/esp_flash/pytest_esp_flash.py @@ -60,8 +60,8 @@ def test_esp_flash_multi(dut: Dut) -> None: @pytest.mark.parametrize( 'config, baud', [ - ('esp32c2_xtal26m', '74880'), - ('esp32c2_xtal26m_rom', '74880'), + ('c2_xtal26m', '74880'), + ('c2_xtal26m_rom', '74880'), ], indirect=True, )