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 test debugging on CI #1491

Closed
juherr opened this issue Jul 23, 2017 · 5 comments
Closed

Improve test debugging on CI #1491

juherr opened this issue Jul 23, 2017 · 5 comments

Comments

@juherr
Copy link
Member

juherr commented Jul 23, 2017

As @selenareneephillips pointed it on #1487, debugging issues on Travis is a pain.

Last time I checked, I found potential ideas:

@krmahadevan
Copy link
Member

krmahadevan commented Jul 24, 2017

We can perhaps consider uploading the test artifacts to some new repository under https://github.com/testng-team to the gh-pages branch by following this instructions .

We will perhaps need @cbeust to help us with generating and adding a new pair of credentials as detailed here

@juherr
Copy link
Member Author

juherr commented Jul 24, 2017

@krmahadevan Yes, it was the idea behind https://github.com/testng-team/testng-travis-reports but I've never gone further.

@selenareneephillips
Copy link
Contributor

Could I also suggest migrating over to more modern and fully featured logging framework like slf4j + logback? It's significantly less clunky than java.util.logging. It can also capture the thread ID. I discovered that the Java logger cannot do that when I was working on the logging for the paralellization test suite.

@juherr
Copy link
Member Author

juherr commented Jul 24, 2017

@selenareneephillips Where do you want slf4j?
For internal usage, @cbeust doesn't like to add extra dependencies, then we will have to design an optional dependency to slf4j around the current testng logger first if you want to use slf4j out of the box.
If you want it in tests, I think it won't be a problem so feel free to use it as a test dependency.

@juherr
Copy link
Member Author

juherr commented Sep 16, 2017

Fixed with #1537

@juherr juherr closed this as completed Sep 16, 2017
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

3 participants