diff --git a/src/test/java/quality/SmokeMethodTestHarness.java b/src/test/java/quality/SmokeMethodTestHarness.java index 0b931ef..61d420d 100644 --- a/src/test/java/quality/SmokeMethodTestHarness.java +++ b/src/test/java/quality/SmokeMethodTestHarness.java @@ -58,7 +58,7 @@ public static void verify(Class clazz) { } String summary = String.format("%s attempted=%d failed=%d", clazz.getSimpleName(), attempted, failed); - Assertions.assertTrue(failed < attempted, () -> "Every executable smoke path failed: " + summary); + Assertions.assertEquals(0, failed, () -> "Executable smoke path failed: " + summary); } private static Object resolveTargetInstance(Method method, Class clazz) throws Exception { diff --git a/src/test/java/quality/SmokeMethodTestHarnessTest.java b/src/test/java/quality/SmokeMethodTestHarnessTest.java index 8fef2ae..8187185 100644 --- a/src/test/java/quality/SmokeMethodTestHarnessTest.java +++ b/src/test/java/quality/SmokeMethodTestHarnessTest.java @@ -1,6 +1,5 @@ package quality; -import static org.junit.jupiter.api.Assertions.assertDoesNotThrow; import static org.junit.jupiter.api.Assertions.assertThrows; import org.junit.jupiter.api.Test; @@ -13,8 +12,8 @@ void rejectsClassesWhoseEveryExecutablePathFails() { } @Test - void acceptsAClassWhenAtLeastOneExecutablePathSucceeds() { - assertDoesNotThrow(() -> SmokeMethodTestHarness.verify(PartiallyWorks.class)); + void rejectsAClassWhenAnyExecutablePathFails() { + assertThrows(AssertionError.class, () -> SmokeMethodTestHarness.verify(PartiallyWorks.class)); } private static final class AlwaysFails {