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

Provide more information on a timeout error #148

Open
justinbmeyer opened this issue Jul 14, 2017 · 2 comments
Open

Provide more information on a timeout error #148

justinbmeyer opened this issue Jul 14, 2017 · 2 comments

Comments

@justinbmeyer
Copy link
Member

We see build errors pretty often like:

https://travis-ci.org/canjs/canjs/builds/253644821#L3694

     Error: Browser timed out within 120 seconds
      at Timeout.testTimedOut [as _onTimeout] (/home/travis/build/[secure]/[secure]/node_modules/testee/lib/runner.js:47:27)
      at ontimeout (timers.js:386:14)
      at tryOnTimeout (timers.js:250:5)
      at Timer.listOnTimeout (timers.js:214:5)

I'm not sure if this is testee, or our build. One way to tell is if testee encounters a timeout error, it should scrape the HTML content of the page and print it (or even a subset of it) for the user. And also may forward any caught window.onerror messages.

@andrejewski
Copy link
Contributor

This would be great. Needs coordination between testee and testee-client but I think its feasible.

@andrejewski
Copy link
Contributor

I am thinking of adding a preflight browser check where Testee starts a browser, points it at a simple HTTP server and once a request is received we shutdown the server and continue the normal test flow. The value here is that we can provide a much better error about a browser launch failure as opposed to someone's own test errors.

This is just one idea.

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

No branches or pull requests

2 participants