-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Description builder #925
Description builder #925
Conversation
…ry methods to allow clients to specify alternative names for the Suites and Tests, without loosing the tied linking between the test result and the class / methods that specifies the test.
…ry methods to allow clients to specify alternative names for the Suites and Tests, without loosing the tied linking between the test result and the class / methods that specifies the test.
…into description-builder
Fantastic. I will add Description.toImmutableDescription() so we can use immutable descriptions in runners that implement Filterable |
When do you think this merged changed will be included in the next (un)official release? |
We were thinking this would be in the next major release after 4.12 (tentatively 5.0). I had some problems with this API when I was working on using it with the filtering code, so I don't think it is quite ready to be merged. |
@marcphilipp I was having some problems using the description builder in the filtering code. Hopefully I'll find time to give it another go in the next month |
@kcooney let me know, if I can support you with something. |
Continue of pull request #890:
As discussed in #890 I rebased the branch on immutable-description and created a new pull request.
All the best
bechte