eclipse-vertx / vertx-junit5

Testing Vert.x applications with JUnit 5
Apache License 2.0
44 stars 32 forks source link

Add failNow overload with string argument #88

Closed ndr-brt closed 4 years ago

ndr-brt commented 4 years ago

As motivated in #87

jponge commented 4 years ago

How about using VertxException rather than NoStackTraceThrowable? I think it's useful to have the stack trace anyway.

ndr-brt commented 4 years ago

@jponge yes, but not in this case I think. The failNow is used at the end of the test, the test fails but not because an exception, so the stacktrace won't give useful informations.

jponge commented 4 years ago

Right