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

Enhancement: Allow for lazily-constructed messages #185

Open
stdedos opened this issue Jun 5, 2024 · 2 comments
Open

Enhancement: Allow for lazily-constructed messages #185

stdedos opened this issue Jun 5, 2024 · 2 comments

Comments

@stdedos
Copy link

stdedos commented Jun 5, 2024

https://junit.org/junit5/docs/snapshot/user-guide/#writing-tests

        assertTrue('a' < 'b', () -> "Assertion messages can be lazily evaluated -- "
                + "to avoid constructing complex messages unnecessarily.");

Maybe you can pass the library's result.getMessage() as an optional parameter (so the user can use the message themselves, or not)

Maybe a tangent to #183

@carterpage
Copy link
Member

Are you having performance problems with the library?

@stdedos
Copy link
Author

stdedos commented Jul 27, 2024

No, but "maybe" I could do something about #183 (comment)

I think pre-computing all tests' diffs by myself will kill performance

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

No branches or pull requests

2 participants