Applies TestRunner change to assert on EvaluationFail tests

partiql · Aug 8, 2024 · ff35761 · ff35761
1 parent 9a7bbc2
commit ff35761
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/test/partiql-tests-runner/src/test/kotlin/org/partiql/runner/test/TestRunner.kt b/test/partiql-tests-runner/src/test/kotlin/org/partiql/runner/test/TestRunner.kt
@@ -26,6 +26,8 @@ class TestRunner<T, V>(private val factory: TestExecutor.Factory<T, V>) {
         case.statements.forEach { run(it, case, executor) }
     }
 
+    private class TestRunnerException(message: String) : RuntimeException(message)
+
     private fun run(input: String, case: TestCase, executor: TestExecutor<T, V>) {
         val assertion = case.assertion
         try {
@@ -41,9 +43,11 @@ class TestRunner<T, V>(private val factory: TestExecutor.Factory<T, V>) {
                 }
                 is Assertion.EvaluationFailure -> {
                     val ion = executor.toIon(actual)
-                    error("Expected error to be thrown but none was thrown.\n${case.name}\nActual result: $ion")
+                    throw TestRunnerException("Expected error to be thrown but none was thrown.\n${case.name}\nActual result: $ion")
                 }
             }
+        } catch (e: TestRunnerException) {
+            throw TestRunnerException(e.message ?: "Expected error to be thrown but none was thrown.")
         } catch (e: Exception) {
             when (case.assertion) {
                 is Assertion.EvaluationSuccess -> throw IllegalStateException("Expected success but exception thrown.", e)