From 97ffb2db249fdc4e3067cbfcc0952a45ebe7367e Mon Sep 17 00:00:00 2001 From: luoxu Date: Tue, 21 Jul 2026 15:33:27 +0800 Subject: [PATCH] fix(bt): reorder test imports to satisfy pre-commit --- .../extension/log_compression/tests/test_arg_size_types.py | 1 - .../ble_log/extension/log_compression/tests/test_db_manager.py | 1 - .../extension/log_compression/tests/test_format_parser.py | 1 - .../extension/log_compression/tests/test_incremental.py | 1 - .../extension/log_compression/tests/test_log_extraction.py | 1 - .../extension/log_compression/tests/test_macro_generation.py | 1 - .../extension/log_compression/tests/test_pipeline_e2e.py | 3 +-- .../ble_log/extension/log_compression/tests/test_utils.py | 1 - .../ble_log/extension/log_compression/tests/update_golden.py | 1 - 9 files changed, 1 insertion(+), 10 deletions(-) diff --git a/components/bt/common/ble_log/extension/log_compression/tests/test_arg_size_types.py b/components/bt/common/ble_log/extension/log_compression/tests/test_arg_size_types.py index 97870d54d8a..d07e67e1732 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/test_arg_size_types.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/test_arg_size_types.py @@ -5,7 +5,6 @@ Verifies that FormatToken.conv_char and FormatToken.length are used correctly to determine ARG_SIZE_TYPE (the fix for the off-by-one bug). """ - import re import shutil import tempfile diff --git a/components/bt/common/ble_log/extension/log_compression/tests/test_db_manager.py b/components/bt/common/ble_log/extension/log_compression/tests/test_db_manager.py index bd49ce62fab..c16a8268995 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/test_db_manager.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/test_db_manager.py @@ -1,7 +1,6 @@ # SPDX-FileCopyrightText: 2026 Espressif Systems (Shanghai) CO LTD # SPDX-License-Identifier: Apache-2.0 """Tests for LogDBManager: state persistence, incremental logic, file tracking.""" - import shutil import tempfile import unittest diff --git a/components/bt/common/ble_log/extension/log_compression/tests/test_format_parser.py b/components/bt/common/ble_log/extension/log_compression/tests/test_format_parser.py index b76ff5692df..15517c57d9b 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/test_format_parser.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/test_format_parser.py @@ -1,7 +1,6 @@ # SPDX-FileCopyrightText: 2026 Espressif Systems (Shanghai) CO LTD # SPDX-License-Identifier: Apache-2.0 """Tests for c_format_parse.py: FormatToken NamedTuple and parse_format_string.""" - import unittest import test_utils # noqa: F401 — must be first to set up sys.path diff --git a/components/bt/common/ble_log/extension/log_compression/tests/test_incremental.py b/components/bt/common/ble_log/extension/log_compression/tests/test_incremental.py index b429fcab891..9bc24460e1a 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/test_incremental.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/test_incremental.py @@ -1,7 +1,6 @@ # SPDX-FileCopyrightText: 2026 Espressif Systems (Shanghai) CO LTD # SPDX-License-Identifier: Apache-2.0 """Tests for incremental compression: caching, re-runs, config changes.""" - import shutil import tempfile import unittest diff --git a/components/bt/common/ble_log/extension/log_compression/tests/test_log_extraction.py b/components/bt/common/ble_log/extension/log_compression/tests/test_log_extraction.py index 30022999323..7540f92f167 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/test_log_extraction.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/test_log_extraction.py @@ -1,7 +1,6 @@ # SPDX-FileCopyrightText: 2026 Espressif Systems (Shanghai) CO LTD # SPDX-License-Identifier: Apache-2.0 """Tests for LogCompressor.extract_log_calls: tree-sitter AST extraction.""" - import unittest import test_utils # noqa: F401 — must be first to set up sys.path diff --git a/components/bt/common/ble_log/extension/log_compression/tests/test_macro_generation.py b/components/bt/common/ble_log/extension/log_compression/tests/test_macro_generation.py index ff133716bfd..ee66660c443 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/test_macro_generation.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/test_macro_generation.py @@ -1,7 +1,6 @@ # SPDX-FileCopyrightText: 2026 Espressif Systems (Shanghai) CO LTD # SPDX-License-Identifier: Apache-2.0 """Tests for compressed macro generation (bluedroid and mesh module scripts).""" - import sys import types import unittest diff --git a/components/bt/common/ble_log/extension/log_compression/tests/test_pipeline_e2e.py b/components/bt/common/ble_log/extension/log_compression/tests/test_pipeline_e2e.py index df9bb132078..4ed9a69462f 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/test_pipeline_e2e.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/test_pipeline_e2e.py @@ -1,15 +1,14 @@ # SPDX-FileCopyrightText: 2026 Espressif Systems (Shanghai) CO LTD # SPDX-License-Identifier: Apache-2.0 """End-to-end pipeline tests: .c input -> .h output with golden file comparison.""" - import re import shutil import tempfile import unittest +from test_utils import assert_header_matches_golden from test_utils import FIXTURE_DIR from test_utils import PipelineContext -from test_utils import assert_header_matches_golden class _E2ETestCase(unittest.TestCase): diff --git a/components/bt/common/ble_log/extension/log_compression/tests/test_utils.py b/components/bt/common/ble_log/extension/log_compression/tests/test_utils.py index 6ef002afe2d..ea35f01f65c 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/test_utils.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/test_utils.py @@ -1,7 +1,6 @@ # SPDX-FileCopyrightText: 2026 Espressif Systems (Shanghai) CO LTD # SPDX-License-Identifier: Apache-2.0 """Shared utilities for BLE log compression tests (unittest-based, no third-party deps).""" - from __future__ import annotations import difflib diff --git a/components/bt/common/ble_log/extension/log_compression/tests/update_golden.py b/components/bt/common/ble_log/extension/log_compression/tests/update_golden.py index 2ecf02bf45b..503d1e0c067 100644 --- a/components/bt/common/ble_log/extension/log_compression/tests/update_golden.py +++ b/components/bt/common/ble_log/extension/log_compression/tests/update_golden.py @@ -11,7 +11,6 @@ This runs the compression pipeline on each fixture C source file and copies the generated header to fixtures/expected/. The output should be reviewed for correctness before committing. """ - import shutil import sys import tempfile