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

[test] beautify logfile output #585

Merged
merged 1 commit into from
Jan 8, 2025
Merged

[test] beautify logfile output #585

merged 1 commit into from
Jan 8, 2025

Conversation

jukzi
Copy link
Contributor

@jukzi jukzi commented Jan 8, 2025

Also explicitly log the testclass to identify the cause of "OpenFromClipboardTests does not exist"

#584

Also explicitly log the testclass to identify the cause of
"OpenFromClipboardTests does not exist"

eclipse-jdt#584
@jukzi
Copy link
Contributor Author

jukzi commented Jan 8, 2025

before:
image
after:
image

@akurtakov
Copy link
Contributor

I really wonder how is this output helpful at all?

@iloveeclipse
Copy link
Member

I really wonder how is this output helpful at all?

If tests hang or report errors without stack etc you will see who was that

@jukzi
Copy link
Contributor Author

jukzi commented Jan 8, 2025

when there is some exception/logging during a test it is clear to which tests it belongs. Especially if a exception does happen in another thread that does not have the test in the stacktrace

@jukzi jukzi merged commit 78045e3 into eclipse-jdt:master Jan 8, 2025
11 checks passed
@jukzi jukzi deleted the logfile branch January 8, 2025 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants