Skip to content

test: drop unnecessary lambda flagged by CodeQL - #53

Merged
mrebbert merged 1 commit into
mainfrom
fix/code-scanning-unnecessary-lambda
Oct 16, 2026
Merged

mrebbert merged 1 commit into
mainfrom
fix/code-scanning-unnecessary-lambda

Conversation

@mrebbert

Copy link
Copy Markdown
Owner

Summary

CodeQL `py/unnecessary-lambda` Alert #19 adressiert: der `lambda: make_system_info_data()` in `tests/test_models_properties.py:117` ist ein nulter-Argument-Wrapper ohne Zweck. Die Fixture hat selbst eine signatur mit ausschließlich default-Werten, also passt `make_system_info_data` direkt an die erwartete `Callable[[], dict[str, Any]]`-Signatur.

Nach Merge schließt CodeQL den Alert automatisch beim nächsten Scan.

Test plan

  • `uv run pytest tests/test_models_properties.py` → 33 passed
  • `uv run ruff check tests/test_models_properties.py` → clean

🤖 Generated with Claude Code

https://claude.ai/code/session_01RgkYaQACLxezeceA1hrXQs

CodeQL py/unnecessary-lambda alert #19: the lambda wrapping the
zero-arg fixture adds nothing, pass the function directly.

Co-Authored-By: Claude Opus 4.7 <[email protected]>
Claude-Session: https://claude.ai/code/session_01RgkYaQACLxezeceA1hrXQs
@mrebbert
mrebbert merged commit 4ab2932 into main Oct 16, 2026
8 checks passed
@mrebbert
mrebbert deleted the fix/code-scanning-unnecessary-lambda branch October 16, 2026 05:22
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