From 796e85f0f6a2b41d516ffb15e086980fbbfacea4 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 +- .../test_apps/sdmmc/sdkconfig.ci.default | 0 .../test_apps/sdspi/sdkconfig.ci.default | 0 .../temperature_sensor/pytest_temperature_sensor.py | 9 ++++++++- .../spi_flash/test_apps/esp_flash/pytest_esp_flash.py | 4 ++-- .../test_apps/flash_suspend/pytest_flash_auto_suspend.py | 1 - .../touch_sens_sleep/pytest_touch_sens_sleep.py | 8 ++++++++ 9 files changed, 26 insertions(+), 5 deletions(-) rename components/esp_driver_i2s/test_apps/lp_i2s/{sdkconfig.ci.defaults => sdkconfig.ci.default} (100%) create mode 100644 components/esp_driver_sdmmc/test_apps/sdmmc/sdkconfig.ci.default create mode 100644 components/esp_driver_sdspi/test_apps/sdspi/sdkconfig.ci.default 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 db2527a9ec..8045374fbb 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,6 +6,13 @@ from pytest_embedded_idf.utils import idf_parametrize @pytest.mark.lp_i2s +@pytest.mark.parametrize( + 'config', + [ + 'default', + ], + indirect=True, +) @idf_parametrize('target', ['esp32p4'], indirect=['target']) 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 1b8ebac311..48f858ebeb 100644 --- a/components/esp_driver_sdio/src/sdio_slave.c +++ b/components/esp_driver_sdio/src/sdio_slave.c @@ -617,7 +617,7 @@ static void sdio_intr_send(void *arg) esp_err_t sdio_slave_send_queue(uint8_t *addr, size_t len, void *arg, uint32_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_sdmmc/test_apps/sdmmc/sdkconfig.ci.default b/components/esp_driver_sdmmc/test_apps/sdmmc/sdkconfig.ci.default new file mode 100644 index 0000000000..e69de29bb2 diff --git a/components/esp_driver_sdspi/test_apps/sdspi/sdkconfig.ci.default b/components/esp_driver_sdspi/test_apps/sdspi/sdkconfig.ci.default new file mode 100644 index 0000000000..e69de29bb2 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 fb7d2fcec8..36fe0c7c29 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 @@ -38,8 +38,15 @@ def test_temperature_sensor_cbs(dut: Dut) -> None: @pytest.mark.two_duts @pytest.mark.parametrize('count', [2], indirect=True) +@pytest.mark.parametrize( + 'config', + [ + 'release', + ], + indirect=True, +) @idf_parametrize('target', ['esp32s2', 'esp32c3', 'esp32s3', 'esp32c2', 'esp32c6', 'esp32c61'], indirect=['target']) 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 423e01c440..bd4087e03c 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 @@ -54,8 +54,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, ) diff --git a/components/spi_flash/test_apps/flash_suspend/pytest_flash_auto_suspend.py b/components/spi_flash/test_apps/flash_suspend/pytest_flash_auto_suspend.py index 09d1bda04d..624898ccae 100644 --- a/components/spi_flash/test_apps/flash_suspend/pytest_flash_auto_suspend.py +++ b/components/spi_flash/test_apps/flash_suspend/pytest_flash_auto_suspend.py @@ -10,7 +10,6 @@ from pytest_embedded_idf.utils import idf_parametrize 'config', [ 'release', - 'i2c_isr_flash', 'text_in_flash_when_suspend', ], indirect=True, diff --git a/examples/peripherals/touch_sensor/touch_sens_sleep/pytest_touch_sens_sleep.py b/examples/peripherals/touch_sensor/touch_sens_sleep/pytest_touch_sens_sleep.py index 6228462e5e..21c40d7ef6 100644 --- a/examples/peripherals/touch_sensor/touch_sens_sleep/pytest_touch_sens_sleep.py +++ b/examples/peripherals/touch_sensor/touch_sens_sleep/pytest_touch_sens_sleep.py @@ -7,6 +7,14 @@ from pytest_embedded_idf.utils import soc_filtered_targets @pytest.mark.generic +@pytest.mark.parametrize( + 'config', + [ + 'deep_sleep', + 'light_sleep', + ], + indirect=True, +) @idf_parametrize( 'target', soc_filtered_targets('SOC_TOUCH_SENSOR_SUPPORTED == 1 and SOC_PM_SUPPORT_TOUCH_SENSOR_WAKEUP == 1'),