Move tests to root directory #74
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
also refactor so they use the pytest runner not using many features of pytest yet, but that should come
Use the new fancy ruff tool for linting and get coverage data from pytest-cov pretty basic at the moment, automatically formatting code feels a bit overkill given the number of contributors.
|
The CI changes included in this are running in my fork and the resulting badge should look like this when merged:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Refactor to use
pytestfor running tests rather than the set of shell scripts and path hacks that it neededPreviously tests were somewhat hidden in the
csg2csgdirectory (leading to things like #69 (comment)) as well as doing apip installwould cause tests to be installed which doesn't seem useful.Am testing with Python 3.7 for compatibility, but would prefer to update to something supported.