-
Notifications
You must be signed in to change notification settings - Fork 74
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
Missing Documentation Pages #342
Comments
I found the documentation about the PackageSettings a little lacking. Specifically I had a hard time figuring out the best configuration for use with IntelliJ running my tests. The differences between the various reporters were unclear and I had to try a bunch. Separately, using For the record, we eventually found:
|
https://github.com/approvals/ApprovalTests.Java/blob/master/approvaltests/docs/README.md "Main concepts for ApprovalTests" |
We created this page: https://github.com/approvals/ApprovalTests.Java/blob/master/approvaltests/docs/explanations/BestConfigurationPractices.md Does it help? |
@ScottBob while that does help explain the order of reporters (and that you could have class or method level reporters) my comment was more about how I learned about how I needed to configure ApprovalTest to make it do what I wanted. It took rather a long time to find that PackageSettings was the way to configure multiple test files in 1 consistent way. Some difficulties I had after I figured out that PackageSettings was a thing included:
|
@Kurru We made a page about FrontLoadedReporters. Would appreciate feedback. |
@Kurru We made a page about the benefits of different reporters. Would appreciate feedback. |
Is there a documentation page that you would like to see?
Add the title of the page below in a comment. Also, please preference it with one of the following four categories:
The text was updated successfully, but these errors were encountered: