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

Travis: fix stalled build #270

Merged
merged 1 commit into from
Mar 3, 2016
Merged

Travis: fix stalled build #270

merged 1 commit into from
Mar 3, 2016

Conversation

ricardopereira
Copy link
Contributor

#143 - Use travis_wait when a long running command or compile step regularly takes longer than 10 minutes without producing any output. It writes a short line to the build log every minute for 20 minutes, extending the amount of time your command has to finish.

 - Use `travis_wait` when a long running command or compile step
     regularly takes longer than 10 minutes without producing any
output. It writes a short line to the build log every minute for 20
minutes, extending the amount of time your command has to finish. Prefix
`travis_wait` with a greater number.
@ricardopereira
Copy link
Contributor Author

It's working 😁

So, the test suite will only output the log when it's done.
If the test suite fails then the command will exit with error, like it used to be.
The summary of failures is generated by xcodebuild. One test failed with:

[11:38:08]: ▸ Failing tests:
[11:38:08]: ▸ -[RestClient RestClient__should_request_another_token_after_current_one_is_no_longer_valid()]
[11:38:08]: ▸ -[RealtimeClientConnection Connection__ACK_and_NACK__should_trigger_the_failure_callback_for_the_remaining_pending_messages_if__connection_is_closed()]
[11:38:08]: ▸ ** TEST FAILED **

@ricardopereira
Copy link
Contributor Author

@mattheworiordan @tcard Can I merge?

@tcard
Copy link
Contributor

tcard commented Mar 3, 2016

Cool! LGTM

@mattheworiordan
Copy link
Member

Nice :)

ricardopereira added a commit that referenced this pull request Mar 3, 2016
@ricardopereira ricardopereira merged commit fa0479e into master Mar 3, 2016
@ricardopereira ricardopereira deleted the travis-enhance branch March 4, 2016 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants