Skip to content
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

Open
julie777 opened this issue Feb 25, 2022 · 10 comments
Open

Needs tests #168

julie777 opened this issue Feb 25, 2022 · 10 comments

Comments

@julie777
Copy link

Feel free to raise an issue for any of the following:

  • Feature request

There are no tests in the repository

@jnichols0
Copy link
Contributor

jnichols0 commented Mar 4, 2022

The tests are in the hitch/story directory, you can read about running them here https://hitchdev.com/approach/contributing/

@julie777
Copy link
Author

julie777 commented Mar 4, 2022

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.

@mtelka
Copy link

mtelka commented Apr 21, 2023

Is there a simple way how to run tests for this project? Thank you.

@crdoconnor
Copy link
Owner

crdoconnor commented Apr 21, 2023 via email

@crdoconnor
Copy link
Owner

The updated instructions are visible here: https://hitchdev.com/approach/contributing/

@mtelka
Copy link

mtelka commented Apr 21, 2023

Thank you. That looks easy. If I read the URL properly just ./key.sh make followed by ./key.sh regression should be run to test strictyaml. If that's the case then I think you could create simple tox.ini file to have both commands run by tox which is currently the preferred way how to run tests uniformly for most Python projects.

@crdoconnor
Copy link
Owner

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.

@mtelka
Copy link

mtelka commented Apr 21, 2023

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.

@crdoconnor
Copy link
Owner

crdoconnor commented Apr 21, 2023 via email

@mtelka
Copy link

mtelka commented Apr 21, 2023

if I wrote a separate script that runs the tests in the environment the script is run in would that suffice?

Sounds reasonable.

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

No branches or pull requests

4 participants