-
-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🧪 tests #18
Comments
Mm I'll probably not pitch in at all regarding tests. It's a bit much for me to try to understand right now. I was actually thinking something like literally running |
Yeah this is a tough one to test - I'm going to do my best to try to do some of that bash style testing inside the pytest framework. Honestly just using the plugin to install the testing and linting environments in its own CI/CD is an integration test on its own. hatch-pip-compile/.github/workflows/tests.yaml Lines 50 to 59 in 6cbb581
|
🎉 This issue has been resolved in version 1.8.3 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
This project needs unit (and integration?) tests.
Acceptance Criteria
New tests. I will keep this issue open until I feel like there's sufficient coverage though.
Notes
Some elements of this project can be tough to test without an actual hatch CLI running - hopefully I can make use of the
click.CliRunner
for this.The text was updated successfully, but these errors were encountered: