Simplify display name in new parameterized tests for console#2651
Simplify display name in new parameterized tests for console#2651HeikoKlare merged 1 commit intoeclipse-platform:masterfrom
Conversation
|
I just further investigated the logs and saw that it also (even more likely) be the line breaks in the test name causing the issue. I am sure if it's a good idea to have such a complex test name representation. What about assigning proper descriptive names for the parameterization than just repeating all arguments? |
This change adjusts display names in new parameterized tests, for console line wrapping. This hopefully avoids confusing GitHub jobs. In addition, emojis in IOConsoleTests are replaced with unicode codepoints. See: eclipse-platform#2650
|
Descriptive will be difficult, but distinctive is possible. How about now? |
Test Results 54 files ± 0 54 suites ±0 35m 35s ⏱️ -49s Results for commit dbe03f0. ± Comparison against base commit ca6b952. This pull request removes 387 and adds 92 tests. Note that renamed tests count towards both. |
HeikoKlare
left a comment
There was a problem hiding this comment.
Descriptive will be difficult, but distinctive is possible. How about now?
Sounds good. I like the proposal, even in case it finally not fixes the issue with the test report. Let's give this a try. Thank you for quickly taking action!
This change adjusts display names in new parameterized tests, for console line wrapping. This hopefully avoids confusing GitHub jobs.
In addition, emojis in
IOConsoleTestsare replaced with unicode codepoints.See: #2650