Skip to content

Move check_agents to pytest; rename properties.yml.example - #12

Merged
LevonBecker merged 1 commit into
mainfrom
standardize_test_config_levonbecker
Jul 30, 2026
Merged

Move check_agents to pytest; rename properties.yml.example#12
LevonBecker merged 1 commit into
mainfrom
standardize_test_config_levonbecker

Conversation

@LevonBecker

Copy link
Copy Markdown
Owner

Summary

Moves check_agents from a modules/tests/ click CLI module to a pytest test under tests/test_check_agents.py, matching the pattern already used in template_ai_vault and ai_vault. Also renames properties.yml.example to properties.example.yml so editors syntax-highlight it as YAML.

Changes

  • Add pytest/pytest-cov deps; add tests.pytest invoke task and CI job
  • Exclude tests/ from ruff/pylint (checked by pytest instead) via pyproject.toml
  • tests/test_check_agents.py replaces modules/tests/check_agents.py; tasks.tests.check_agents now runs pytest tests/test_check_agents.py
  • Update docs (README.md, .github/instructions/*.md) to reflect the new location and pytest job
  • Rename properties.yml.exampleproperties.example.yml, fix references in docs.prompt.md mirrors and template.ignore.yml

check_agents moves from modules/tests/ (a click CLI module) to
tests/test_check_agents.py (a pytest test), matching the pattern already used
in template_ai_vault and ai_vault. Adds pytest/pytest-cov as dependencies, a
tests.pytest invoke task, a pytest CI job, and excludes tests/ from
ruff/pylint (checked by pytest instead) — same pyproject.toml shape as the
other two repos.

Also renames properties.yml.example to properties.example.yml so editors
recognize and syntax-highlight it as YAML.

Co-Authored-By: Claude Sonnet 5 <[email protected]>
@LevonBecker LevonBecker self-assigned this Jul 30, 2026
@LevonBecker
LevonBecker merged commit 545dcb4 into main Jul 30, 2026
6 checks passed
@LevonBecker
LevonBecker deleted the standardize_test_config_levonbecker branch July 30, 2026 01:42
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