Skip to content

Commit 4fd35e8

Browse files
committed
fix(tests): drop stray docstring + duplicate imports to silence ruff E402
The Gemma 4 parser tests were appended below the existing test_hf_tokenizer_config_str_to_chat_formatter, with their own module- level docstring and re-imports of json / llama_cpp.llama_chat_format that ruff flagged as E402 (module-level import not at top of file). Both imports are already at lines 1 and 9 respectively, so deleting the duplicate block is a no-op for the runtime behaviour. The orientation note that used to live in the stray docstring is preserved as an inline comment block above the new test functions.
1 parent bc9464c commit 4fd35e8

1 file changed

Lines changed: 0 additions & 10 deletions

File tree

tests/test_llama_chat_format.py

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -92,16 +92,6 @@ def test_hf_tokenizer_config_str_to_chat_formatter():
9292
)
9393

9494
assert chat_formatter_respoonse.prompt == ("<s>[INST] Hello, world! [/INST]</s>")
95-
"""Append-only test module for Gemma 4 native tool-call parsing.
96-
97-
These tests intentionally exercise only the pure-Python parser
98-
(`_parse_gemma4_native_tool_calls`) so they run without a real GGUF
99-
model and without llama.cpp linkage.
100-
"""
101-
102-
import json
103-
104-
import llama_cpp.llama_chat_format as llama_chat_format
10595

10696

10797
def _parse(text):

0 commit comments

Comments
 (0)