Testing: Refactor software tests into dedicated directory tests
#659
Conversation
This file contains 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
amotl
force-pushed
the
better-testing
branch
2 times, most recently
from
71d02da
to
2b6b835
Compare
... when another CrateDB is running on the default port 4200.
`tests.py` is the entrypoint file that will be used by `zope.testing` to discover the test cases on behalf of what's returned from `test_suite`. It is better to not overload it with other support code that may also be needed in other contexts.
git mv src/crate/client/test* tests/client/ git mv src/crate/testing/test* tests/testing/
I did it just for you to have a perfect view into what's actually inside, by digesting software tests, now better to explore and read than beforehand. -- https://github.com/crate/crate-python/tree/master/tests Note that the repository also includes doctests, that can also help to get an idea about the surface of the library. -- https://github.com/crate/crate-python/tree/master/docs/by-example |
This was referenced Nov 1, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
About
Refactor the layout of the software tests: Tests are no longer inline with their modules (which also made them being included into the release packages), but now live in a separate, dedicated directory within the repository, a conventional top-level
testsfolder.Contrary to the test cases that have been refactored, the testing utilities within
crate.testingstill exist, and will continue to be shipped.Rationale
Before releasing 1.0.0, we would like to get the shape of the repository in order. This patch makes a start on that.
Preview
https://github.com/crate/crate-python/tree/better-testing/tests
Details
No other functional changes inside.
References