Skip to content

Commit

Permalink
Fix to use ISO-8601 instead of epoch millis
Browse files Browse the repository at this point in the history
  • Loading branch information
icepeppermint committed Jan 9, 2023
1 parent 6e2e8e4 commit e8cf8ac
Showing 1 changed file with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ package com.linecorp.conditional.kotlin

import com.linecorp.conditional.kotlin.CoroutineConditionMatchState.*
import kotlinx.coroutines.CoroutineName
import java.time.Instant
import java.time.ZoneId
import kotlin.coroutines.CoroutineContext

@UnstableApi
Expand Down Expand Up @@ -53,12 +55,14 @@ class CoroutineConditionMatchResult internal constructor(
.append(", coroutine=${coroutineContext[CoroutineName]}")
.append(", delay=${millisAsString(condition.delayMillis)}")
.append(", timeout=${millisAsString(condition.timeoutMillis)}")
.append(", startTime=${millisAsString(startTimeMillis)}")
.append(", endTime=${millisAsString(endTimeMillis)}")
.append(", startTime=${millisAsISO8601String(startTimeMillis)}")
.append(", endTime=${millisAsISO8601String(endTimeMillis)}")
.append(", duration=${millisAsString(durationMillis)}")
.append('}')
toString()
}

private fun millisAsString(millis: Long) = if (millis == Long.MAX_VALUE) "INF" else "${millis}ms"
private fun millisAsISO8601String(millis: Long): String =
with(Instant.ofEpochMilli(millis)) { atZone(ZoneId.systemDefault()).toOffsetDateTime() }.toString()
}

0 comments on commit e8cf8ac

Please sign in to comment.