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.

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

test: Bump timeout to run tests in travis thread sanitizer #15005

Merged
merged 1 commit into from Dec 21, 2018

Conversation

Projects
None yet
3 participants
@MarcoFalke
Copy link
Member

commented Dec 19, 2018

No need to exclude tests because their timeout was set too strict

@MarcoFalke MarcoFalke added the Tests label Dec 19, 2018

@practicalswift

This comment has been minimized.

Copy link
Member

commented Dec 19, 2018

ACK fa05d52

Very nice find -- I didn't understand that the timeouts were the problem for these two tests.

Now if we only could enable the wallet tests too :-)

@Empact

This comment has been minimized.

Copy link
Member

commented Dec 20, 2018

ACK fa05d52

MarcoFalke added a commit to MarcoFalke/bitcoin that referenced this pull request Dec 21, 2018

Merge bitcoin#15005: test: Bump timeout to run tests in travis thread…
… sanitizer

fa05d52 test: Bump timeout to run tests in travis thread sanitizer (MarcoFalke)

Pull request description:

  No need to exclude tests because their timeout was set too strict

Tree-SHA512: ea7032e674a66456bab40122adf4beebd703fd626a7567ca76d2a4e78041af115564d863dc7f742d39bd4adb249c29751439a1238d5e5c34e474c7221d4fd6e1

@MarcoFalke MarcoFalke merged commit fa05d52 into bitcoin:master Dec 21, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@MarcoFalke MarcoFalke deleted the MarcoFalke:Mf1812-testBumpTimeout branch Dec 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.