Skip to content

travis: Run property based testing #14171

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
Closed
maflcko wants to merge 1 commit into from
Closed

travis: Run property based testing #14171

maflcko wants to merge 1 commit into from

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Sep 8, 2018

No description provided.

@maflcko maflcko added the Tests label Sep 8, 2018
@DrahtBot
Copy link
Contributor

DrahtBot commented Sep 8, 2018 •
edited
Loading

No more conflicts as of last run.

@maflcko
Copy link
Member Author

maflcko commented Sep 9, 2018 

Running tests: key_properties from test/key_properties.cpp
Running 4 test cases...
Test cases order is shuffled using seed: 1426192395
Entering test module "Bitcoin Test Suite"
test/key_properties.cpp(22): Entering test suite "key_properties"
test/key_properties.cpp(46): Entering test case "key_sign_symmetry"
Using configuration: seed=8254597269899563210
unknown location(0): fatal error: in "key_properties/key_sign_symmetry": memory access violation at address: 0x00000020: no mapping at fault address
test/key_properties.cpp(46): last checkpoint: "key_sign_symmetry" entry.
Test is aborted
test/key_properties.cpp(46): Leaving test case "key_sign_symmetry"; testing time: 18160us
Test is aborted
test/key_properties.cpp(22): Leaving test suite "key_properties"; testing time: 18222us
Test is aborted
Leaving test module "Bitcoin Test Suite"; testing time: 18410us
*** 1 failure is detected in the test module "Bitcoin Test Suite"
Makefile:10713: recipe for target 'test/key_properties.cpp.test' failed
make[3]: *** [test/key_properties.cpp.test] Error 1

@DrahtBot DrahtBot mentioned this pull request Sep 9, 2018
Merged
@practicalswift
Copy link
Contributor

practicalswift commented Sep 9, 2018

This is so great to see in actions! Thanks a lot @Christewart for making this happen. Hope to see more contributions from you! :-)

Strong concept ACK

@maflcko maflcko force-pushed the Mf1809-qaRapidCheck branch 6 times, most recently from 163b2db to fa546dc Compare September 11, 2018 21:41
@maflcko maflcko closed this Sep 11, 2018
@maflcko maflcko deleted the Mf1809-qaRapidCheck branch September 11, 2018 21:41
@practicalswift
Copy link
Contributor

practicalswift commented Sep 11, 2018

Why the close? :-)

@maflcko
Copy link
Member Author

maflcko commented Sep 13, 2018

@practicalswift This is up for grabs. Figuring out what caused this probably needs someone to reproduce locally in a docker container that is set up the same way as the one on travis. Or run with valgrind/asan on travis (remote debugging)

@Christewart
Copy link
Contributor

Christewart commented Sep 17, 2018

@fanquake I know you expressed interest in picking this up too ^.

@Christewart Christewart mentioned this pull request Oct 8, 2018
Closed
@fanquake fanquake mentioned this pull request Dec 1, 2018
Merged
@fanquake
Copy link
Member

fanquake commented Dec 9, 2018

Removed "Up for grabs" as this is being re-tried in #14853.

@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

Tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@maflcko @DrahtBot @practicalswift @Christewart @fanquake