make assume inline, add additional tests

willowtreeapps · Dec 5, 2023 · 270449d · 270449d
1 parent e73f725
commit 270449d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/assertk/src/jvmMain/kotlin/assertk/assume.kt b/assertk/src/jvmMain/kotlin/assertk/assume.kt
@@ -10,6 +10,6 @@ package assertk
  * }
  * ```
  */
-fun assume(f: () -> Unit) {
+inline fun assume(f: () -> Unit) {
     AssumptionFailure.run { f() }
 }
diff --git a/assertk/src/jvmTest/kotlin/test/assertk/assertions/AssumeTest.kt b/assertk/src/jvmTest/kotlin/test/assertk/assertions/AssumeTest.kt
@@ -1,9 +1,12 @@
 package test.assertk.assertions
 
 import assertk.assertThat
+import assertk.assertions.isEqualTo
 import assertk.assertions.isFalse
+import assertk.assertions.isTrue
 import assertk.assume
 import com.willowtreeapps.opentest4k.TestAbortedException
+import test.assertk.runTest
 import kotlin.test.Test
 import kotlin.test.assertEquals
 import kotlin.test.assertFailsWith
@@ -25,4 +28,16 @@ class AssumeTest {
         // this test should be skipped instead of failing
         assume { assertThat(true).isFalse() }
     }
+
+    @Test
+    fun assume_does_not_capture_unexpected_exceptions() {
+        assertFailsWith<NullPointerException> {
+            assume { throw NullPointerException() }
+        }
+    }
+
+    @Test
+    fun assume_aborts_when_suspend() = runTest {
+        assume { assertThat(suspend { true }()).isFalse()  }
+    }
 }