Revamp unit testing infrastructure #1401
Merged
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1401 +/- ##
========================================
Coverage ? 43.7%
========================================
Files ? 36
Lines ? 2137
Branches ? 0
========================================
Hits ? 934
Misses ? 1203
Partials ? 0Continue to review full report at Codecov.
|
khyberspache
reviewed
Mar 17, 2020
Can you expand on why you switched out unites for Pytest?
…d to remove deprecated calls in py38
privateducky
reviewed
Mar 17, 2020
privateducky
reviewed
Mar 17, 2020
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.


Replaces barebones Travis CI configuration with Tox for more in-depth testing featureset and consistency across multiple dev environments
Previously, you would manually install flake8 and use unittest manually to run tests. Tox builds entire virtualenvs with the required dependencies to run tests to mitigate the potential differences between developer environments.
Switch out python
unittestforpytestAdd development requirements file
Add code coverage with
coverage.pythrough https://codecov.io/gh/mitre/calderaAdd
pre-commitconfiguration file that can be used by developers to catch potential errors before committed. It currently only runs flake8 and flake8-bugbear, but it will be incorporating the followingpre-commithooks in the future:The hooks do the following:
check-byte-order-markerForbid files which have a UTF-8 byte-order marker
trailing-whitespaceTrims trailing whitespace.
end-of-file-fixerMakes sure files end in a newline and only a newline.
The text was updated successfully, but these errors were encountered: