Added new type of test case: shouldThrowWithMessage

[skinny85]

Hey Roberto,

firstly, thanks a ton for the contribution!

In general, it looks good. One idea that I had that I wanted to run past you is changing the API slightly to allow more assertions on the exception from shouldThrow than the message.

My idea was to shouldThrow return a mutable object (let's call it ExceptionAssertion). It would have a bunch of methods that all return this and each add another condition that the thrown Exception has to satisfy. In code:

it.shouldThrow(NumberFormatException.class, "when trying to parse a long from the string 'long'", () -> {
    new Long("long");
}).withMessage("For input string: \"long\"");

This allows you to easily add other Exception assertions to the ExceptionAssertion class without adding more methods to the main SpecBuilder interface. Some possible examples, inspired by AssertJ's Exception assertions:

• withMessageContaining("\"long\"")
• withMessageStartingWith("For input string: ")
• withNoCause()
• withCause().withMessage("sth")

etc. etc.

Thoughts on this idea?

[skinny85]

Hey Roberto,

I've submitted a new PR with the changes we talked about above (unfortunately, I can't update your PR). Can we move the discussion there? #6

Thanks, Adam

[skinny85]

I'm closing this one in favor of #6 which was merged. Thanks again!