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

Add new reporting system #183

Closed
wants to merge 17 commits into from
Closed

Add new reporting system #183

wants to merge 17 commits into from

Conversation

jmalloc
Copy link
Member

@jmalloc jmalloc commented Nov 23, 2020

What change does this introduce?

This PR adds the report.Report type and several associated types which represent a human-readable report about the activity within and final result of a test.

What issues does this relate to?

Why make this change?

To allow for a consist report building and rendering experience regardless of whether the test result was determined by an Expectation or by some other occurrence.

Is there anything you are unsure about?

No

@jmalloc jmalloc force-pushed the report-builder branch 2 times, most recently from ef705ac to 7fbb02a Compare November 23, 2020 02:30
report/builder.go Outdated Show resolved Hide resolved
report/suggestion.go Outdated Show resolved Hide resolved
report/step.go Outdated Show resolved Hide resolved
@jmalloc jmalloc changed the title Add Finding and its associated types. Add new reporting system Nov 23, 2020
@jmalloc jmalloc force-pushed the report-builder branch 8 times, most recently from 03c9787 to cf97f8d Compare November 28, 2020 09:15
@jmalloc jmalloc closed this Jan 23, 2022
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.

Refactor report building so that it can be used outside of expectations.
2 participants