Skip to content

add tests for CLI patterns - #188

Merged
persandstrom merged 6 commits into
version-2from
add-cli-test
Jul 23, 2026
Merged

add tests for CLI patterns#188
persandstrom merged 6 commits into
version-2from
add-cli-test

Conversation

@persandstrom

Copy link
Copy Markdown
Owner

Add test for CLI
Add test to workflow

Copilot AI review requested due to automatic review settings July 23, 2026 06:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds automated test coverage for the Click-based CLI option parsing and introduces CI to run those tests across supported Python versions.

Changes:

  • Add pytest-based CLI parsing tests exercising --arm-away, --arm-home, and --arm-state.
  • Add pytest to requirements-dev.txt.
  • Add a GitHub Actions workflow to run the test suite on Python 3.10–3.12.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
tests/test_cli.py New tests validating CLI option parsing and arity handling.
requirements-dev.txt Adds pytest for running the new test suite.
.github/workflows/test.yml New CI workflow to execute tests on pushes and PRs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tests/test_cli.py Outdated
Comment thread tests/test_cli.py Outdated
persandstrom and others added 5 commits July 23, 2026 08:24
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@persandstrom
persandstrom merged commit 7c2e915 into version-2 Jul 23, 2026
6 checks passed
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