[Travis] Show functional tests progress #943
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
example (added the progress index `n/m`) ``` 1/107 - wallet_hd.py passed, Duration: 27 s ......................................................................................... 2/107 - mining_getblocktemplate_longpoll.py passed, Duration: 72 s .................................................................. 3/107 - feature_maxuploadtarget.py passed, Duration: 78 s ``` - clear dots line ``` $ test/functional/test_runner.py -t can_trash Temporary test directory at can_trash/test_runner_₿_🏃_20181018_220600 1/105 - wallet_hd.py passed, Duration: 21 s 2/105 - mining_getblocktemplate_longpoll.py passed, Duration: 71 s 3/105 - feature_maxuploadtarget.py passed, Duration: 68 s .................. ``` - don't print the `dot` progressive if `--quiet` - done_str - nothing commit to check again travis tests
This helps finding out which tests fail to finish.
Collaborator
|
utACK ae0b4d0 |
Author
|
Added backport of bitcoin#16214 |
Travis was regularly failing because the full test suite with coverage takestoo much time. Bitcoin has some optimisations in there which allows them to give less time to the test part of the process. We don't so until we do, we have to reserve more time. We will probably have to restart manually more travis jobs but that's the only way to have them run to the end.
Author
|
This is good to go now. This should avoid seeing travis build fail at the 50mn mark with no way to get them to succeed. |
Fuzzbawls
approved these changes
Jul 11, 2019
Collaborator
|
ACK 5fbf26c |
Fuzzbawls
added a commit
that referenced
this pull request
Jul 11, 2019
5fbf26c [Travis] Give more time to tests (warrows) 1aa76d0 travis: Use absolute paths for cache dirs (MarcoFalke) 1fa0bf3 travis: Fix caching issues (MarcoFalke) e4945a2 [Travis] Log more info (warrows) ae0b4d0 tests: Print remaining jobs in test_runner.py (Steven Roose) a57939f tests: Print dots by default (Chun Kuan Lee) cde9d13 show the progress of functional test (Isidoro Ghezzi) Pull request description: Backport of bitcoin#14504, bitcoin#14569 and bitcoin#15466. This should help to see which test takes too long or blocks travis when it hangs because of the 50 minutes limit. ACKs for commit 5fbf26: Fuzzbawls: ACK 5fbf26c Tree-SHA512: 0c2bebd8f160e2c6358a598f3cf95c3451068af1b5e269d79366fe890f9609c140555cb182f0f4563dac35a0a433556eaf06e4a45865fd55bdd357130acc1d2c
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
Backport of bitcoin#14504, bitcoin#14569 and bitcoin#15466.
This should help to see which test takes too long or blocks travis when it hangs because of the 50 minutes limit.