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

build: add nox automatic tests #7

Merged
merged 11 commits into from
Apr 27, 2023
Merged

Conversation

12rambau
Copy link
Contributor

structure of a simple nox test suite.
The idea behind nox is to run specific tests in isolated envoronment based on special sub-set of requirements, usually test and doc.

It's coded in python so easier to debug (for me) than a tox.ini file and can be reused in github actions to avoid dealing with deps management and test definition in the yaml.

in your local computer, it's as easy as:

nox

As there is nothing to test yet, I assumed that the pre-commit linter will eventually be merged and I'll be able to build test and doc structure from here afterward.

@12rambau 12rambau mentioned this pull request Apr 24, 2023
@12rambau
Copy link
Contributor Author

@santiagobasulto deeply sorry for the noise but I was trying to make it work against the Github actions.

Now it's ready to merge.

we have 2 nox sessions here "lint" that simply apply all pre-commits hooks and "test" that run the pytest in an isolated env.

The env are based on poetry groups so pytest deps are not needed in the dev suit anymore but just in the test one. Note that at the moment it look like nothing but sphinx build, coverage and sphinx autosummary can (will) also be included so the commands will beecome more and more powerfull.

I use the same session to run the test in github actions.

I'll wait for this one to be merged before adding more things in the doc as it's easier to build the site locally with nox session rather than custom venvs

@12rambau 12rambau mentioned this pull request Apr 27, 2023
@santiagobasulto
Copy link
Owner

This is perfect. Sorry for the delay, I have a pretty tough week :/ I'll merge it now.

@santiagobasulto santiagobasulto merged commit eab7ed3 into santiagobasulto:master Apr 27, 2023
@santiagobasulto
Copy link
Owner

@12rambau can you hit me on email? I assume you're in European timezone as me. Let's connect outside this repo: santiago.basulto@gmail.com

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