Repository navigation
Add pytest and CI baseline - #98
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
All changed files use LF-only endings despite the repository-wide CRLF requirement.
5 open findings
What changed in this PR
Adds an initial pytest suite and GitHub Actions CI baseline.
Changes:
- Adds unit tests for input generation and result parsing.
- Configures pytest and Python 3.10/3.13 CI.
- Adds test dependencies and generated-artifact exclusions.
| File | Description |
|---|---|
tests/test_result_parser.py |
Tests outlier filtering. |
tests/test_create_input.py |
Tests validation, parsing, and dispatch. |
requirements-test.txt |
Defines test dependencies. |
pytest.ini |
Configures test discovery. |
.gitignore |
Ignores test artifacts. |
.github/workflows/tests.yml |
Runs pytest in GitHub Actions. |
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
The z-score threshold test cannot detect regressions in the threshold it claims to verify.
1 open finding
5 resolved since last review
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Approval recommended
The new tests and CI configuration are coherent, correctly scoped, and comply with repository conventions.
0 open findings
1 resolved since last review
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
No description provided.