Skip to content

Shows the name of the current test in the maven logs - #158

Closed
tresf wants to merge 3 commits into
masterfrom
show-test-methods
Closed

tresf wants to merge 3 commits into
masterfrom
show-test-methods

Conversation

@tresf

@tresf tresf commented Nov 30, 2023 •

Copy link
Copy Markdown

Purpose:

  • Adds a few utility classes to customize the styling from within JUnit to match that of maven
  • Adds the method name of the current test being run in green

Closes #156

Output from the GitHub runner:

[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running jssc.bootpath.ManualBootLibraryPathFailedTest
[INFO] --- surefire:testBootPathOverride @ jssc
#                   ^----------------------------------- HERE

Output from a console that supports color:
junit_method_names2

@tresf
tresf requested a review from pietrygamat November 30, 2023 19:59
@pietrygamat

pietrygamat commented Dec 1, 2023 •

Copy link
Copy Markdown
Collaborator

Output from a console that supports color:

What about consoles without color support? Are the control characters invisible, or printed junk?

@tresf

tresf commented Dec 1, 2023

Copy link
Copy Markdown
Author

Output from a console that supports color:

What about consoles without color support? Are the control characters invisible, or printed junk?

I've provided a colorless example from the GitHub actions. I haven't used an interactive terminal that didn't support colors in over 10 years.

@tresf tresf mentioned this pull request Dec 1, 2023
3 tasks done
@pietrygamat pietrygamat added this to the 2.9.6 milestone Dec 1, 2023
@tresf

tresf commented Dec 8, 2023

Copy link
Copy Markdown
Author

Closing, superseded by #159.

@tresf tresf closed this Dec 8, 2023
@tresf
tresf deleted the show-test-methods branch December 8, 2023 22:45
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.

Show method names in unit tests

2 participants