Use pytest-forked module for Linux CI #294
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
Due to apparent bugs in pyparsing, this can significantly speed up unit test execution. Also: - Removed use of pexpect in unit tests
- The python-forked module doesn't work on Windows
Codecov Report
@@ Coverage Diff @@
## master #294 +/- ##
==========================================
+ Coverage 91.35% 91.42% +0.06%
==========================================
Files 1 1
Lines 1481 1481
==========================================
+ Hits 1353 1354 +1
+ Misses 128 127 -1
Continue to review full report at Codecov.
|
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.
Now the Travis CI Linux builds make use of the
pytest-forkedmodule to speed up unit tests.The reason this helps is due to an apparent bug in
pyparsing. See Issue #287 for more information.Also:
pexpectin unit test along with the single unit test which used it and was skipped on all current CI platforms because it didn't work there.This closes #293