orSkip and throwing exceptions

32 views
Skip to first unread message

Robin Green

unread,
Sep 29, 2012, 7:30:35 AM9/29/12
to specs2...@googlegroups.com
I had a strange issue last week in which it seemed orSkip was being
ignored, no matter where I put it. I think this was because I was
using a mutable specification, in which, by default, match failures
throw exceptions. It would be nice if this mistake (trying to use
orSkip in conjunction with matchers that throw exceptions) could be
made more obvious somehow - and in any case, I think it should be
mentioned when "orSkip" is talked about in the documentation.

etorreborre

unread,
Sep 29, 2012, 9:14:18 AM9/29/12
to specs2...@googlegroups.com
A precise example would be interesting.

I just had a look at the "orSkip" code and it looks like failure are intercepted ok but not errors. Is that errors you're having which should be skipped?

Thanks,

Eric.

Robin Green

unread,
Sep 29, 2012, 10:57:30 AM9/29/12
to specs2...@googlegroups.com
I don't have the code in front of me right now because I'm not at work
- I can check it next week - but I think it was because the failing
matcher threw an exception, so .orSkip was never reached at all, so it
could not possibly have any effect.

etorreborre

unread,
Oct 2, 2012, 6:47:33 PM10/2/12
to specs2...@googlegroups.com
Hi Robin, 

You can try the latest 1.12.2-SNAPSHOT where the matchers are more robusts when using "orSkip" or "orPending".

Thanks,

Eric.

Daniel Sobral

unread,
Oct 2, 2012, 8:15:57 PM10/2/12
to specs2...@googlegroups.com
On Tue, Oct 2, 2012 at 7:47 PM, etorreborre <etorr...@gmail.com> wrote:
> Hi Robin,
>
> You can try the latest 1.12.2-SNAPSHOT where the matchers are more robusts
> when using "orSkip" or "orPending".

Where are orSkip and orPending in the documentation? I don't recall
ever seeing them.
--
Daniel C. Sobral

I travel to the future all the time.

etorreborre

unread,
Oct 2, 2012, 8:48:22 PM10/2/12
to specs2...@googlegroups.com
They are buried into this big paragraph showing all the ways to create custom matchers from existing ones:


E.
Reply all
Reply to author
Forward
0 new messages