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

Improve logs for Sauce labs runs on Travis #10530

Merged
merged 10 commits into from Jul 19, 2017
Merged

Improve logs for Sauce labs runs on Travis #10530

merged 10 commits into from Jul 19, 2017

Conversation

rsimha
Copy link
Contributor

@rsimha rsimha commented Jul 19, 2017

This PR does the following:

  • Exposes the actual number of integration tests that pass, fail, or are skipped using the super-dots reporter.
  • Prints a human readable summary of the tests run, with failure reasons, using the mocha reporter in minimal mode.
  • In case of test failures, no longer prints a meaningless call stack, but instead, cleanly exits the test runner with a process.exit(exitCode).

Fixes #10482

@rsimha rsimha requested a review from dreamofabear July 19, 2017 16:47
@rsimha rsimha self-assigned this Jul 19, 2017
@rsimha
Copy link
Contributor Author

rsimha commented Jul 19, 2017

/to @choumx

@rsimha rsimha requested a review from erwinmombay July 19, 2017 17:53
@rsimha
Copy link
Contributor Author

rsimha commented Jul 19, 2017

@choumx, @erwinmombay, this is now ready for review.

See https://travis-ci.org/ampproject/amphtml/jobs/255352838 for an example of what the sauce labs logs will look like in case of a failure.

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

Successfully merging this pull request may close these issues.

None yet

4 participants