-
Notifications
You must be signed in to change notification settings - Fork 61
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
Needs tests #168
Comments
The tests are in the hitch/story directory, you can read about running them here https://hitchdev.com/approach/contributing/ |
Thanks. I never thought of looking in a different repo for them. I am going to need to study hitchstory. I have substantial experience using behave for BDD. I am curious to see the changes. |
Is there a simple way how to run tests for this project? Thank you. |
Yes, apologies I let the original instructions get out of date. I'll update
them tonight.
…On Fri, 21 Apr 2023, 17:22 Marcel Telka, ***@***.***> wrote:
Is there a simple way how to run tests for this project? Thank you.
—
Reply to this email directly, view it on GitHub
<#168 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABOJKNOASFRZBENVGCAIK43XCKX3TANCNFSM5PK4XZOQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
The updated instructions are visible here: https://hitchdev.com/approach/contributing/ |
Thank you. That looks easy. If I read the URL properly just |
As I understand it tox is just a different way of creating virtual environments, etc. to run tests in. This is something the code run by key.sh already does. It also containerizes the running code to ensure for greater consistency, which tox can't and doesn't do. |
Not only that. |
ah, fair enough.
if I wrote a separate script that runs the tests in the environment the
script is run in would that suffice?
…On Fri, 21 Apr 2023, 22:34 Marcel Telka, ***@***.***> wrote:
Not only that. tox could be used also with current environment (as
opposed to virtualized or containerized env) using the tox-current-env
plugin. This is exactly what is needed for downstream packagers like me.
I'm packaging strictyaml (and many other Python projects) for OpenIndiana
and we need to run tests in current environment to make sure the built
package will work as expected and designed.
—
Reply to this email directly, view it on GitHub
<#168 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABOJKNPV5UY5EV3CKBRBSN3XCL4MRANCNFSM5PK4XZOQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Sounds reasonable. |
Feel free to raise an issue for any of the following:
There are no tests in the repository
The text was updated successfully, but these errors were encountered: