From 58b3c60f598be49310ee44a1f458f73d61d89eb3 Mon Sep 17 00:00:00 2001 From: Marek Fiala Date: Tue, 15 Apr 2025 15:05:32 +0200 Subject: [PATCH] fix(tools): Update ruff-format changes --- tools/test_build_system/test_cmake.py | 96 ++++++++++++++++++--------- 1 file changed, 63 insertions(+), 33 deletions(-) diff --git a/tools/test_build_system/test_cmake.py b/tools/test_build_system/test_cmake.py index 90dc3797af..3ca5bd8200 100644 --- a/tools/test_build_system/test_cmake.py +++ b/tools/test_build_system/test_cmake.py @@ -8,10 +8,10 @@ import sys from pathlib import Path import pytest -from test_build_system_helpers import append_to_file from test_build_system_helpers import EnvDict -from test_build_system_helpers import file_contains from test_build_system_helpers import IdfPyFunc +from test_build_system_helpers import append_to_file +from test_build_system_helpers import file_contains from test_build_system_helpers import run_cmake from test_build_system_helpers import run_cmake_and_build @@ -19,61 +19,91 @@ from test_build_system_helpers import run_cmake_and_build def test_build_custom_cmake_project(test_app_copy: Path) -> None: # Test is compatible with any target. Random targets in the list are selected for performance reasons idf_path = Path(os.environ['IDF_PATH']) - for target in ['esp32','esp32c2','esp32c3','esp32c6','esp32h2','esp32p4','esp32s2','esp32s3']: + for target in ['esp32', 'esp32c2', 'esp32c3', 'esp32c6', 'esp32h2', 'esp32p4', 'esp32s2', 'esp32s3']: logging.info(f'Test build ESP-IDF as a library to a custom CMake projects for {target}') - run_cmake_and_build(str(idf_path / 'examples' / 'build_system' / 'cmake' / 'idf_as_lib'), '-G', 'Ninja', - '-DCMAKE_TOOLCHAIN_FILE={}'.format(idf_path / 'tools' / 'cmake' / f'toolchain-{target}.cmake'), f'-DTARGET={target}') + run_cmake_and_build( + str(idf_path / 'examples' / 'build_system' / 'cmake' / 'idf_as_lib'), + '-G', + 'Ninja', + '-DCMAKE_TOOLCHAIN_FILE={}'.format(idf_path / 'tools' / 'cmake' / f'toolchain-{target}.cmake'), + f'-DTARGET={target}', + ) assert file_contains((test_app_copy / 'build' / 'compile_commands.json'), '"command"') shutil.rmtree(test_app_copy / 'build') -@pytest.mark.skipif(sys.platform == 'win32', reason='On Win project is not buildable without system compiler on the host machine. (Win CI runners)') +@pytest.mark.skipif( + sys.platform == 'win32', + reason='On Win project is not buildable without system compiler on the host machine. (Win CI runners)', +) def test_build_custom_cmake_project_host() -> None: - logging.info(f'Test build ESP-IDF as a library to a custom CMake projects for host') + logging.info('Test build ESP-IDF as a library to a custom CMake projects for host') idf_path = Path(os.environ['IDF_PATH']) run_cmake_and_build(str(idf_path / 'examples' / 'build_system' / 'cmake' / 'idf_as_lib'), '-G', 'Ninja') def test_build_cmake_library_psram_workaround(test_app_copy: Path) -> None: - logging.info('Building a project with CMake library imported and PSRAM workaround, all files compile with workaround') + logging.info( + 'Building a project with CMake library imported and PSRAM workaround, all files compile with workaround' + ) idf_path = Path(os.environ['IDF_PATH']) - (test_app_copy / 'sdkconfig.defaults').write_text('\n'.join(['CONFIG_SPIRAM=y', - 'CONFIG_SPIRAM_CACHE_WORKAROUND=y'])) - run_cmake('-G', 'Ninja', '-DCOMPONENTS=main;esp_psram', - '-DSDKCONFIG_DEFAULTS={}'.format(test_app_copy / 'sdkconfig.defaults'), - str(idf_path / 'examples' / 'build_system' / 'cmake' / 'import_lib')) + (test_app_copy / 'sdkconfig.defaults').write_text( + '\n'.join(['CONFIG_SPIRAM=y', 'CONFIG_SPIRAM_CACHE_WORKAROUND=y']) + ) + run_cmake( + '-G', + 'Ninja', + '-DCOMPONENTS=main;esp_psram', + '-DSDKCONFIG_DEFAULTS={}'.format(test_app_copy / 'sdkconfig.defaults'), + str(idf_path / 'examples' / 'build_system' / 'cmake' / 'import_lib'), + ) with open((test_app_copy / 'build' / 'compile_commands.json'), 'r', encoding='utf-8') as f: data = f.read() res = re.findall(r'.*\"command\".*', data) for r in res: - assert 'mfix-esp32-psram-cache-issue' in r, 'All commands in compile_commands.json should use PSRAM cache workaround' + assert 'mfix-esp32-psram-cache-issue' in r, ( + 'All commands in compile_commands.json should use PSRAM cache workaround' + ) def test_build_cmake_library_psram_strategies(idf_py: IdfPyFunc, test_app_copy: Path) -> None: for strategy in ['DUPLDST', 'NOPS', 'MEMW']: logging.info(f'Test for external libraries in custom CMake projects with PSRAM strategy {strategy}') - (test_app_copy / 'sdkconfig.defaults').write_text('\n'.join(['CONFIG_SPIRAM=y', - f'CONFIG_SPIRAM_CACHE_WORKAROUND_STRATEGY_{strategy}=y', - 'CONFIG_SPIRAM_CACHE_WORKAROUND=y'])) + (test_app_copy / 'sdkconfig.defaults').write_text( + '\n'.join( + [ + 'CONFIG_SPIRAM=y', + f'CONFIG_SPIRAM_CACHE_WORKAROUND_STRATEGY_{strategy}=y', + 'CONFIG_SPIRAM_CACHE_WORKAROUND=y', + ] + ) + ) idf_py('reconfigure') with open((test_app_copy / 'build' / 'compile_commands.json'), 'r', encoding='utf-8') as f: data = f.read() res = re.findall(r'.*\"command\".*', data) for r in res: - assert f'mfix-esp32-psram-cache-strategy={strategy.lower()}' in r, ('All commands in compile_commands.json ' - 'should use PSRAM cache workaround strategy') + assert f'mfix-esp32-psram-cache-strategy={strategy.lower()}' in r, ( + 'All commands in compile_commands.json should use PSRAM cache workaround strategy' + ) (test_app_copy / 'sdkconfig').unlink() def test_defaults_unspecified_build_args(idf_copy: Path) -> None: logging.info('Defaults set properly for unspecified idf_build_process args') idf_as_lib_path = idf_copy / 'examples' / 'build_system' / 'cmake' / 'idf_as_lib' - append_to_file(idf_as_lib_path / 'CMakeLists.txt', '\n'.join(['idf_build_get_property(project_dir PROJECT_DIR)', - 'message("Project directory: ${project_dir}")'])) - ret = run_cmake('..', '-G', 'Ninja', - '-DCMAKE_TOOLCHAIN_FILE={}'.format(str(idf_copy / 'tools' / 'cmake' / 'toolchain-esp32.cmake')), - '-DTARGET=esp32', - workdir=idf_as_lib_path) + append_to_file( + idf_as_lib_path / 'CMakeLists.txt', + '\n'.join(['idf_build_get_property(project_dir PROJECT_DIR)', 'message("Project directory: ${project_dir}")']), + ) + ret = run_cmake( + '..', + '-G', + 'Ninja', + '-DCMAKE_TOOLCHAIN_FILE={}'.format(str(idf_copy / 'tools' / 'cmake' / 'toolchain-esp32.cmake')), + '-DTARGET=esp32', + workdir=idf_as_lib_path, + ) assert 'Project directory: {}'.format(str(idf_as_lib_path.as_posix())) in ret.stderr @@ -83,14 +113,14 @@ def test_build_example_on_host(default_idf_env: EnvDict) -> None: idf_as_lib_path = Path(idf_path, 'examples', 'build_system', 'cmake', 'idf_as_lib') try: target = 'esp32' - run_cmake('..', - f'-DCMAKE_TOOLCHAIN_FILE={idf_path}/tools/cmake/toolchain-{target}.cmake', - f'-DTARGET={target}', - '-GNinja', - workdir=idf_as_lib_path) + run_cmake( + '..', + f'-DCMAKE_TOOLCHAIN_FILE={idf_path}/tools/cmake/toolchain-{target}.cmake', + f'-DTARGET={target}', + '-GNinja', + workdir=idf_as_lib_path, + ) - run_cmake('--build', - '.', - workdir=idf_as_lib_path) + run_cmake('--build', '.', workdir=idf_as_lib_path) finally: shutil.rmtree(idf_as_lib_path / 'build', ignore_errors=True)