Skip to content

Commit 8b075bd

Browse files
Joseph Coopergrodin
authored andcommitted
Rename Throwable extractor functions
1 parent d529997 commit 8b075bd

File tree

4 files changed

+16
-18
lines changed

4 files changed

+16
-18
lines changed

assertk/src/commonMain/kotlin/assertk/assertions/throwable.kt

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,38 +6,38 @@ import assertk.all
66
/**
77
* Returns an assert on the throwable's message.
88
*/
9-
fun Assert<Throwable>.message() = having("message", Throwable::message)
9+
fun Assert<Throwable>.havingMessage() = having("message", Throwable::message)
1010

1111
/**
1212
* Returns an assert on the throwable's cause.
1313
*/
14-
fun Assert<Throwable>.cause() = having("cause", Throwable::cause)
14+
fun Assert<Throwable>.havingCause() = having("cause", Throwable::cause)
1515

1616
/**
1717
* Returns an assert on the throwable's root cause.
1818
*/
19-
fun Assert<Throwable>.rootCause() = having("rootCause", Throwable::rootCause)
19+
fun Assert<Throwable>.havingRootCause() = having("rootCause", Throwable::rootCause)
2020

2121
/**
2222
* Asserts the throwable has the expected message.
2323
*/
2424
fun Assert<Throwable>.hasMessage(message: String?) {
25-
message().isEqualTo(message)
25+
havingMessage().isEqualTo(message)
2626
}
2727

2828
/**
2929
* Asserts the throwable contains the expected text
3030
*/
3131
fun Assert<Throwable>.messageContains(text: String) {
32-
message().isNotNull().contains(text)
32+
havingMessage().isNotNull().contains(text)
3333
}
3434

3535
/**
3636
* Asserts the throwable is similar to the expected cause, checking the type and message.
3737
* @see [hasNoCause]
3838
*/
3939
fun Assert<Throwable>.hasCause(cause: Throwable) {
40-
cause().isNotNull().all {
40+
havingCause().isNotNull().all {
4141
kClass().isEqualTo(cause::class)
4242
hasMessage(cause.message)
4343
}
@@ -48,14 +48,14 @@ fun Assert<Throwable>.hasCause(cause: Throwable) {
4848
* @see [hasCause]
4949
*/
5050
fun Assert<Throwable>.hasNoCause() {
51-
cause().isNull()
51+
havingCause().isNull()
5252
}
5353

5454
/**
5555
* Asserts the throwable is similar to the expected root cause, checking the type and message.
5656
*/
5757
fun Assert<Throwable>.hasRootCause(cause: Throwable) {
58-
rootCause().all {
58+
havingRootCause().all {
5959
kClass().isEqualTo(cause::class)
6060
hasMessage(cause.message)
6161
}

assertk/src/commonTest/kotlin/test/assertk/AssertFailureTest.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ package test.assertk
33
import assertk.assertFailure
44
import assertk.assertions.isEqualTo
55
import assertk.assertions.isInstanceOf
6-
import assertk.assertions.message
6+
import assertk.assertions.havingMessage
77
import com.willowtreeapps.opentest4k.AssertionFailedError
88
import test.assertk.assertions.valueOrFail
99
import kotlin.coroutines.resume
@@ -28,7 +28,7 @@ class AssertFailureTest {
2828
fun failure_originating_subject_not_wrapped_in_result() {
2929
val t = assertFailsWith<AssertionFailedError> {
3030
assertFailure { throw RuntimeException("foo") }
31-
.message()
31+
.havingMessage()
3232
.isEqualTo("bar")
3333
}
3434
assertTrue("RuntimeException" in t.message!!)

assertk/src/commonTest/kotlin/test/assertk/assertions/ThrowableTest.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,17 +14,17 @@ class ThrowableTest {
1414

1515
@Test
1616
fun extracts_message() {
17-
assertEquals(subject.message, assertThat(subject).message().valueOrFail)
17+
assertEquals(subject.message, assertThat(subject).havingMessage().valueOrFail)
1818
}
1919

2020
@Test
2121
fun extracts_cause() {
22-
assertEquals(cause, assertThat(subject).cause().valueOrFail)
22+
assertEquals(cause, assertThat(subject).havingCause().valueOrFail)
2323
}
2424

2525
@Test
2626
fun extracts_root_cause() {
27-
assertEquals(rootCause, assertThat(subject).rootCause().valueOrFail)
27+
assertEquals(rootCause, assertThat(subject).havingRootCause().valueOrFail)
2828
}
2929

3030
//region hasMessage

assertk/src/commonTest/kotlin/test/assertk/assertions/support/SupportTest.kt

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

33
import assertk.assertThat
4-
import assertk.assertions.hasMessage
5-
import assertk.assertions.isEqualTo
64
import assertk.assertions.isFailure
75
import assertk.assertions.isSuccess
8-
import assertk.assertions.message
6+
import assertk.assertions.havingMessage
97
import assertk.assertions.support.expected
108
import assertk.assertions.support.fail
119
import assertk.assertions.support.show
@@ -236,7 +234,7 @@ class SupportTest {
236234
@Test
237235
fun expected_originating_throwable_included_as_cause() {
238236
val subject = RuntimeException()
239-
val assert = assertThat(subject).message()
237+
val assert = assertThat(subject).havingMessage()
240238
val error = assertFailsWith<AssertionFailedError> {
241239
assert.expected("message")
242240
}
@@ -246,7 +244,7 @@ class SupportTest {
246244
@Test
247245
fun expected_originating_failure_result_included_as_cause() {
248246
val subject = RuntimeException()
249-
val assert = assertThat(Result.failure<String>(subject)).isFailure().message()
247+
val assert = assertThat(Result.failure<String>(subject)).isFailure().havingMessage()
250248
val error = assertFailsWith<AssertionFailedError> {
251249
assert.expected("message")
252250
}

0 commit comments

Comments
 (0)