Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Storage.HTTPCancelMultiple test occasionally fails #3163

Closed
jfirebaugh opened this issue Dec 2, 2015 · 5 comments
Closed

Storage.HTTPCancelMultiple test occasionally fails #3163

jfirebaugh opened this issue Dec 2, 2015 · 5 comments
Assignees
Labels

Comments

@jfirebaugh
Copy link
Contributor

https://travis-ci.org/mapbox/mapbox-gl-native/jobs/94300803#L1439

../../test/storage/http_cancel.cpp:43: Failure
Value of: res.error
  Actual: 8-byte object <60-43 24-01 00-00 00-00>
Expected: nullptr
Which is: 8-byte object <00-00 00-00 00-00 00-00>
../../test/storage/http_cancel.cpp:45: Failure
Value of: res.data.get()
  Actual: false
Expected: true
@jfirebaugh
Copy link
Contributor Author

https://travis-ci.org/mapbox/mapbox-gl-native/jobs/100170656

The message from the unexpected error is "Couldn't connect to server: couldn't connect to host".

@1ec5
Copy link
Contributor

1ec5 commented Jan 4, 2016

This test is failing for me at least as often on Travis as the iOS KIF tests do on Bitrise.

@jfirebaugh
Copy link
Contributor Author

The test server is initialized with gtest's Shared Test Resource functionality -- it's started once, used for all Storage tests, then shut down. But while it's active, it will shut itself down if there's no activity for 5 seconds. As of e6dc4fe this is quite common. When that happens, any subsequent tests that try to connect receive an error.

@jfirebaugh jfirebaugh self-assigned this Jan 5, 2016
jfirebaugh added a commit that referenced this issue Jan 5, 2016
It caused #3163.

Attempts at having the child exit automatically when the parent exits were stymied
by the fact that unix sucks: http://stackoverflow.com/q/284325/52207.
@jfirebaugh
Copy link
Contributor Author

Fixed in 79368a0. Implementing proper process control is up for grabs.

@kkaefer
Copy link
Member

kkaefer commented Jan 5, 2016

#3432

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants