gsmaverick / wren-test

A testing framework for Wren
MIT License
27 stars 2 forks source link

Use Fiber.abort instead of a failed Expectation for serious errors #3

Closed gsmaverick closed 9 years ago

gsmaverick commented 9 years ago

Right now if you try to use a Fiber matcher on a non-Fiber object we use a failed Expectation instance to signal this problem. It would probably be better to call Fiber.abort in this case so it can be more intelligently reported about.

gsmaverick commented 9 years ago

This also makes ensuring a certain value cleaner in Matchers and doesn't require complicated early returns.