Skip to content

fix(discovery): ignore empty test files#143

Merged
Khabib73 merged 1 commit into
Fast-Python:mainfrom
innovationty:bugfix/ignore-empty-test-files
Jul 15, 2026
Merged

fix(discovery): ignore empty test files#143
Khabib73 merged 1 commit into
Fast-Python:mainfrom
innovationty:bugfix/ignore-empty-test-files

Conversation

@innovationty

Copy link
Copy Markdown
Contributor

Summary

  • ignore .yatl.yaml and .yatl.yml files that contain only whitespace or comments during discovery
  • prevent empty test files from being counted or reported as passing
  • cover directory discovery, direct file discovery, and CLI output

Why

Empty test files were included in the discovered file count and then executed as empty test specifications. Filtering them during discovery keeps the reported count and output aligned with the tests that actually contain content.

Fixes #95

Validation

  • pytest tests/test_run.py tests/test_cli/test_command.py::test_empty_test_files_are_ignored -q (12 passed)
  • pytest --ignore=tests/test_cli/test_command.py -q (68 passed)
  • ruff check .
  • ruff format --check .

@innovationty
innovationty marked this pull request as ready for review July 15, 2026 08:39
@Khabib73

Copy link
Copy Markdown
Member

@innovationty Thank you so much for your excellent contribution! Your PR helps the project a lot, and I am happy to accept it. 😊 If you enjoyed working on backus (YATL), I would appreciate it if you supported us with a star on GitHub! ⭐

@Khabib73
Khabib73 merged commit 19f914d into Fast-Python:main Jul 15, 2026
2 checks passed
@innovationty

Copy link
Copy Markdown
Contributor Author

Thank you for the kind words and I’m glad the change is helpful.I’ve starred the repository and would be happy to contribute again in the future.

@Khabib73

Copy link
Copy Markdown
Member

welcome :)

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.

2 participants