Adds doctesting to the VCS test suite. #2151
Open
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
Currently only queries test is implemented. Once CDAT/vcs#114 is merged, this test will pass. Tests for other modules/classes will be very similar. Already did some setup for running with vcs.Canvas. A few of the steps in this script aren't necessary for queries, but will be needed for others (the cleanup step is mainly for Canvas and manageElements). If docstring contains '.. pragma: skip-doctest', the doctest for that docstring will not be run. If docstring is empty and not private, and has not been skipped, it will be printed in a "NO DOCUMENTATION" list after the tests so we can view it on cdash, but it will not cause a failure.
ghost
requested a review
from 
ghost
mentioned this pull request
* Canvas.py * boxfill.py * colormap.py * displayplot.py * fillarea.py * isofill.py * isoline.py * line.py * manageElements.py * marker.py * meshfill.py * projection.py * taylordiagram.py * template.py * textcombined.py * textorientation.py * texttable.py
|
@doutriaux1 @chaosphere2112 I got the bulk of the modules/classes in VCS. Tests that break:
Every test will break until CDAT/vcs#114 is merged. The above tests, and some tests I have not added yet, will break until #2144 and CDAT/vcs#104 are merged. |
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.
Currently only queries test is implemented.
Once CDAT/vcs#114 is merged, this test
will pass. Tests for other modules/classes will be very similar.
Already did some setup for running with vcs.Canvas. A few of the
steps in this script aren't necessary for queries, but will be
needed for others (the cleanup step is mainly for Canvas and
manageElements).
If docstring contains '.. pragma: skip-doctest', the doctest
for that docstring will not be run.
If docstring is empty and not private, and has not been skipped,
it will be printed in a "NO DOCUMENTATION" list after the tests
so we can view it on cdash, but it will not cause a failure.