Fix the assertion description for isAfter

robfletcher · Mar 23, 2022 · 9395b82 · 9395b82
1 parent 29df71b
commit 9395b82
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/strikt-jvm/src/main/kotlin/strikt/java/time/TemporalAccessor.kt b/strikt-jvm/src/main/kotlin/strikt/java/time/TemporalAccessor.kt
@@ -43,7 +43,7 @@ infix fun <T : TemporalAccessor> Assertion.Builder<T>.isBefore(expected: Tempora
  * temporal type.
  */
 infix fun <T : TemporalAccessor> Assertion.Builder<T>.isAfter(expected: TemporalAccessor): Assertion.Builder<T> =
-  assertThat("is before %s", expected) {
+  assertThat("is after %s", expected) {
     when (it) {
       is Instant -> it.isAfter(Instant.from(expected))
       is ChronoLocalDate -> it.isAfter(LocalDate.from(expected))

diff --git a/strikt-jvm/src/test/kotlin/strikt/java/time/TemporalAssertions.kt b/strikt-jvm/src/test/kotlin/strikt/java/time/TemporalAssertions.kt
@@ -181,7 +181,7 @@ internal class TemporalAssertions : JUnit5Minutests {
         local to local.minusDays(1),
         Pair(localOffset, localOffset.minusNanos(1))
       ).forEach { (subject, expected) ->
-        test("passes asserting $subject (${subject.javaClass.simpleName}) is before $expected (${expected.javaClass.simpleName})") {
+        test("passes asserting $subject (${subject.javaClass.simpleName}) is after $expected (${expected.javaClass.simpleName})") {
           expectThat(subject).isAfter(expected)
         }
       }
@@ -215,7 +215,7 @@ internal class TemporalAssertions : JUnit5Minutests {
         local to local.plusDays(1),
         Pair(localOffset, localOffset.plusNanos(1))
       ).forEach { (subject, expected) ->
-        test("fails asserting $subject (${subject.javaClass.simpleName}) is before $expected (${expected.javaClass.simpleName})") {
+        test("fails asserting $subject (${subject.javaClass.simpleName}) is after $expected (${expected.javaClass.simpleName})") {
           assertThrows<AssertionFailedError> {
             expectThat(subject).isAfter(expected)
           }