Pytest Plugin #41
Merged
Pytest Plugin #41
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
* Set up the project to be used as a pytest plugin. * Created the `snappiershot.plugins.tracker.Tracker` object for tracking the status of snapshots throughout an entire run. Other Notes: * Updated the config object, removing the "significant figures" configuration and adding a "full diff" configuration. * Removed the pytest-cov dependency. Unfortunately if does not work well with testing coverage of pytest coverage. More information can be found here: https://pytest-cov.readthedocs.io/en/latest/plugins.html * Created the run_tests.sh bash script for running tests.
bbonenfant
added
enhancement
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
| float_absolute_tolerance = 1e-6 | ||
| float_relative_tolerance = 0.001 | ||
| full_diff = false |
There was a problem hiding this comment.
perhaps a more clear variable name show_full_diff? also its a bit unclear what a non-full diff implies... Should be more clear/verbose about this throughout
There was a problem hiding this comment.
perhaps out of scope, but may want to consider for later: do we want levels of verbosity? (rather than binary)
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
| @@ -0,0 +1,69 @@ | |||
| """ Tests for snappiershot/plugins/pytest.py """ | |||
There was a problem hiding this comment.
ah yes the wonderful meta testing :)
lenabradley
reviewed
Oct 7, 2020
lenabradley
reviewed
Oct 7, 2020
lenabradley
approved these changes
Oct 7, 2020
|
Update the README to include a pytest usage example |
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.
This PR sets up the repository to be used as a
pytestplugin. This includes the following:snappiershot/plugins/pytest.py.pytest11plugin within thepyproject.tomlfile.pytesterplugin andtesterfixture to test the plugin. More information can be found hereIn addition, the following objects and functionality was added:
snappiershot.plugins.tracker.Trackerobjectfor tracking the status of snapshots throughout an entire run.
snappiershot.snapshot.SnapshotStatusenum.configuration and adding a "full diff" configuration.
Finally, the following infrastructure change occurred:
well with testing coverage of pytest coverage. More information can
be found here
Closes #26