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.

Sign up for GitHub

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

Jump to bottom

Fix type signature of ExpectedException#except() #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix type signature of ExpectedException#except() #13

wants to merge 2 commits into from

Commits on Sep 10, 2015

  1. Fix type signature of ExpectedException#expect() 

    Following the PECS guideline expect() should take Matcher<? super
Throwable>.

One test had to be adjusted so that the code still compiles:
any(Expection.class) was replaced by instanceOf(Expection.class).
Judging from the JavaDoc of both matchers the latter is anyway the
better fitting one.
    @UrsMetz
    UrsMetz committed Sep 10, 2015
    Configuration menu
    Copy the full SHA
    e66f6f7 View commit details
    Browse the repository at this point in the history

  2. Simplify ExpectedExceptionMatcherBuilder 

    * use Java 7's diamond operator
* harmonize generics
* remove unused "generics handling" code
    @UrsMetz
    UrsMetz committed Sep 10, 2015
    Configuration menu
    Copy the full SHA
    f79ebe2 View commit details
    Browse the repository at this point in the history