Skip to content

Support multiple reporters#12

Merged
neocotic merged 9 commits intomasterfrom
reporters
Feb 9, 2017
Merged

Support multiple reporters#12
neocotic merged 9 commits intomasterfrom
reporters

Conversation

@neocotic
Copy link
Copy Markdown
Member

@neocotic neocotic commented Feb 9, 2017

In 0.1.0 the only thing that Verifier could do upon verification would be to assert that the verification passed and throw a VerifierException otherwise.

This PR changes this behaviour only slightly. By default, the above is still what happens, however, consumers of Verifier can now tap into the reporting process of the verification if needed and even prevent an error from being thrown if they so desired. The main intention here is to simply allow consumers to configure and customize Verifier as much as possible and this was a key step towards that. This would could allow consumers to even log verifications, if they wanted to, pass/fail.

Following on the back of #11, custom Reporters and even ReportExecutor are registered and loaded using Java's SPI.

@neocotic neocotic added this to the 0.2.0 milestone Feb 9, 2017
@neocotic neocotic self-assigned this Feb 9, 2017
@neocotic neocotic mentioned this pull request Feb 9, 2017
15 tasks
@neocotic neocotic merged commit 5cced60 into master Feb 9, 2017
@neocotic neocotic deleted the reporters branch February 9, 2017 16:47
@neocotic neocotic restored the reporters branch February 9, 2017 16:51
@neocotic
Copy link
Copy Markdown
Member Author

neocotic commented Feb 9, 2017

This PR was accidentally merged into master instead of develop. I have reverted this (#13) and merged the changes into develop (#14).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant