-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
Integration Module with ApprovalTests.Java #420
Comments
Could be interesting for you https://github.com/mmerdes. |
So essentially, all this is is, is an This is usually done manually in JUnit tests when you expect a certain output - I can see how this could be useful. Question is though, can it be integrated with property tests? From where I am looking it mainly seems useful with |
I tend to think so, because it could simplify and empower the The standard |
Yeah ... It could be pretty useful - it won't work with that library though from what I can tell? There doesn't seem to be any way to save any userdata on invocations (so you can keep track of things like invocation # and seed #). I see there is something for parameterized tests in there (although, the userdata in this case is just a string and it outputs additional files for each parameterized version... which isn't very useful when you have 1000's of invocations): Rather than modifying that library, a native solution with an Sorry if I'm rambling... I just started thinking out loud... ;) |
Testing Problem
Approval testing is an interesting approach, especially when dealing with legacy code.
Combining it with (deterministic) test generation as done by jqwik could give it real additional value.
ApprovalTests.Java is the framework of choice for doing AT in Java.
It sort of works with jqwik, but not in a very convenient way.
Suggested Solution
Create a jqwik module which builds on the ideas in this issue for ApprovalTests.Java
Discussion
There may be changes warranted in ApprovalTests.Java to simplify the implementation of a jqwik-approvaltests module. Maybe there could be native support for jqwik contributed as a PR.
The different options should be researched in a POC first before deciding on the best way to go.
The text was updated successfully, but these errors were encountered: