Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

unittest-statistics.py mixes up test suite and test case statistics #17

Closed
lucash-dev opened this issue Apr 16, 2018 · 0 comments
Closed

unittest-statistics.py mixes up test suite and test case statistics #17

lucash-dev opened this issue Apr 16, 2018 · 0 comments

Comments

@lucash-dev
Copy link
Contributor

The python script is mixing up test suite statistics and test case statistics. This is likely caused by a change in bitcoin to the output of test_bitcoin.

Steps to reproduce issue: run the script, and notice that there's a line for "Bitcoin Test Suite" with the total time of all tests.

I have code to fix this issue and will submit a PR.
This was referenced Apr 16, 2018
Merged
Closed
laanwj added a commit that referenced this issue May 9, 2018
@laanwj
Merge #18: fixed issue that mixed up suite and test case statistics.
72a842c 
d98d1e3 fixed issue that mixed up suite and test case statistics. (lucash-dev)

Pull request description:

  This PR fixes issue [17](#17).

Tree-SHA512: 8ef8377e0063ef25983607a569f1d28753a0e3c336caf986aa0756885413b81f2f8df7839e60680cece9d5fa076799a1f30d6f36afbb1bd253ac649a3693d2ac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lucash-dev and others