Should handle throwables as failed tests #20

src/main/kotlin/io/kotlintest/KTestJUnitRunner.kt

@@ -23,7 +23,7 @@ class KTestJUnitRunner(val testClass: Class<TestBase>) : Runner() {
         instance2.beforeEach()
         notifier.fireTestStarted(desc)
         testcase.test()
-      } catch(e: Exception) {
+      } catch(e: Throwable) {
         notifier.fireTestFailure(Failure(desc, e))
       } finally {
         notifier.fireTestFinished(desc)
@@ -42,7 +42,7 @@ class KTestJUnitRunner(val testClass: Class<TestBase>) : Runner() {
         notifier.fireTestStarted(desc)
         instance.beforeEach()
         testcase.test()
-      } catch(e: Exception) {
+      } catch(e: Throwable) {
         notifier.fireTestFailure(Failure(desc, e))
       } finally {
         notifier.fireTestFinished(desc)

src/test/kotlin/io/kotlintest/KotlinTestJunitRunnerTest.kt

@@ -0,0 +1,36 @@
+package io.kotlintest
+
+import io.kotlintest.specs.ShouldSpec
+import org.junit.runner.notification.Failure
+import org.junit.runner.notification.RunListener
+import org.junit.runner.notification.RunNotifier
+import java.util.concurrent.CountDownLatch
+import java.util.concurrent.TimeUnit
+
+class KotlinTestJunitRunnerTest : ShouldSpec() {
+  init {
+    should("handle throwables") {
+      System.setProperty("internal", "true")
+      val runner = KTestJUnitRunner(ThrowsThrowable::class.java as Class<TestBase>)
+      val n = RunNotifier()
+      val latch = CountDownLatch(1)
+      n.addListener(object : RunListener() {
+        override fun testFailure(failure: Failure?) {
+          latch.countDown()
+        }
+      })
+      runner.run(n)
+      latch.await(5, TimeUnit.SECONDS) shouldBe true
+      System.setProperty("internal", "false")
+    }
+  }
+}
+
+class ThrowsThrowable : ShouldSpec() {
+  init {
+    should("throw throwable") {
+      if (System.getProperty("internal") == "true")
+        throw Throwable("hello")
+    }
+  }
+}