Skip to content

Commit 9998500

Browse files
committed
make assume inline, add additional tests
1 parent 8911199 commit 9998500

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

assertk/src/jvmMain/kotlin/assertk/assume.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,6 @@ package assertk
1010
* }
1111
* ```
1212
*/
13-
fun assume(f: () -> Unit) {
13+
inline fun assume(f: () -> Unit) {
1414
AssumptionFailure.run { f() }
1515
}

assertk/src/jvmTest/kotlin/test/assertk/assertions/AssumeTest.kt

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,12 @@
11
package test.assertk.assertions
22

33
import assertk.assertThat
4+
import assertk.assertions.isEqualTo
45
import assertk.assertions.isFalse
6+
import assertk.assertions.isTrue
57
import assertk.assume
68
import com.willowtreeapps.opentest4k.TestAbortedException
9+
import test.assertk.runTest
710
import kotlin.test.Test
811
import kotlin.test.assertEquals
912
import kotlin.test.assertFailsWith
@@ -25,4 +28,16 @@ class AssumeTest {
2528
// this test should be skipped instead of failing
2629
assume { assertThat(true).isFalse() }
2730
}
31+
32+
@Test
33+
fun assume_does_not_capture_unexpected_exceptions() {
34+
assertFailsWith<NullPointerException> {
35+
assume { throw NullPointerException() }
36+
}
37+
}
38+
39+
@Test
40+
fun assume_aborts_when_suspend() = runTest {
41+
assume { assertThat(suspend { true }()).isFalse() }
42+
}
2843
}

0 commit comments

Comments
 (0)