Skip to content

Support multiple reporters (Take 2)#14

Merged
neocotic merged 2 commits intodevelopfrom
reporters
Feb 9, 2017
Merged

Support multiple reporters (Take 2)#14
neocotic merged 2 commits intodevelopfrom
reporters

Conversation

@neocotic
Copy link
Copy Markdown
Member

@neocotic neocotic commented Feb 9, 2017

This is a second attempt PR as the first (#12) was accidentally merged into master and reverted (#13), so let's try again, but this time merging into develop.


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
This was referenced Feb 9, 2017
@neocotic neocotic merged commit 623c460 into develop Feb 9, 2017
@neocotic neocotic deleted the reporters branch February 9, 2017 17:04
neocotic added a commit that referenced this pull request Feb 11, 2017
* java8 is now the minimum supported version since java7 is post-LTS

* bump to 0.2.0-SNAPSHOT

* Merge branch 'develop' into java8

* improved handling of multi-dimensional arrays and circular references in DefaultArrayFormatter

* Support different locales (#11)

* added initial support for multiple locales

* attempting to fix broken build

* added initial support for multiple verification reporters (#14)

* added assignableFromAll and assignableFromAny methods to ClassVerifier (#15)

* Rename ClassVerifier#interfacing to ClassVerifier#interfaced (#16)

* renamed interfacing method in ClassVerifier to interfaced

* corrected error message for interfaced method on ClassVerifier

* renamed defaulting method in LocaleVerifier to defaulted (#17)

* added matchAll and matchAny methods to StringVerifier (#18)

* added causedByAll and causedByAny methods to ThrowableVerifier (#19)

* updated copyright year throughout

* Update Crowdin configuration file

* added more Formatters and simplified Verification API (#21)

* added and & andComparable methods to CustomVerifier to compliment verify & verifyComparable methods on Verifier while using a partial copy of the current Verification (#22)

* added crowdin badge to README

* added JavaDoc badge to README

* configured Codecov and added badge to README

* added changelog

* roll 0.2.0
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