Skip to content

test(code_language): suppress large fixture output and consolidate tests#3806

Open
ceberam wants to merge 1 commit into
mainfrom
test/consolidate-code
Open

test(code_language): suppress large fixture output and consolidate tests#3806
ceberam wants to merge 1 commit into
mainfrom
test/consolidate-code

Conversation

@ceberam

@ceberam ceberam commented Jul 14, 2026

Copy link
Copy Markdown
Member

Summary

Cleans up tests/test_code_language_detection.py without changing any
test logic or coverage:

  • Suppress verbose fixture output — the four large-input cases in
    test_detection_stays_linear_on_large_input were previously parametrized
    with raw string values (~200 k characters each). Under pytest -v these
    are printed verbatim as test IDs, flooding CI logs. Each case now uses
    pytest.param(..., id="<short-name>") so the output stays terse
    ([repeated-from-keywords], [repeated-select-lines], etc.).

  • Consolidate hint-interaction teststest_hint_takes_precedence_over_content
    and test_unrecognized_hint_falls_back_to_content tested the same
    integration boundary (hint vs. content resolution in
    detect_code_language). They are merged into a single parametrized
    test_detect_with_hint, consistent with the style used throughout
    the module.

No test cases were added or removed; all 55 tests continue to pass.

Checklist:

  • Documentation has been updated, if necessary.
  • Examples have been added, if necessary.
  • Tests have been added, if necessary.

…nt tests

Signed-off-by: Cesar Berrospi Ramis <ceb@zurich.ibm.com>
@github-actions

Copy link
Copy Markdown
Contributor

DCO Check Passed

Thanks @ceberam, all your commits are properly signed off. 🎉

@mergify

mergify Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant