Skip to content

test: cover non-Python CLI targets - #14

Open
PuneetRana7 wants to merge 2 commits into
Johnkothapalli:mainfrom
PuneetRana7:test-empty-non-python-targets
Open

test: cover non-Python CLI targets#14
PuneetRana7 wants to merge 2 commits into
Johnkothapalli:mainfrom
PuneetRana7:test-empty-non-python-targets

Conversation

@PuneetRana7

@PuneetRana7 PuneetRana7 commented Aug 13, 2026

Copy link
Copy Markdown

Summary

  • Add CLI tests for an empty directory.
  • Add CLI tests for a directory containing only non-Python files.
  • Add CLI tests for a direct non-Python file target.
  • Assert file count, score, text output, and exit code for each case.

Testing

  • python -m pytest
  • ruff check .
  • mypy

No production behavior was changed because the existing behavior was already consistent across all three cases.
Closes #5

@Johnkothapalli Johnkothapalli left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The three CLI behavior tests match #5 and pass locally, along with Ruff lint and mypy. One required gate still fails: ruff format --check . reports tests/test_cli.py would be reformatted (missing blank separation between top-level tests, a multiline signature normalization, trailing whitespace, and no final newline).

Please run ruff format tests/test_cli.py, verify ruff format --check ., and add Closes #5 to the PR description so the accepted work is linked to its assigned issue. No production change is needed.

@PuneetRana7

Copy link
Copy Markdown
Author

Ok i ll let u know.

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.

Add edge-case tests for empty and non-Python scan targets

2 participants