Fix exception thrown for null start time (#777)

* Fix exception thrown for null start time * Update release notes
Flank · May 13, 2020 · f146c48 · f146c48
1 parent feafecc
commit f146c48
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/release_notes.md b/release_notes.md
@@ -1,4 +1,5 @@
 ## next (unreleased)
+- [#775](https://github.com/Flank/flank/issues/775) Fix exception thrown for null start time. ([pawelpasterz](https://github.com/pawelpasterz))
 
 ## v20.05.0
 

diff --git a/test_runner/src/main/kotlin/ftl/reports/api/CreateTestExecutionData.kt b/test_runner/src/main/kotlin/ftl/reports/api/CreateTestExecutionData.kt
@@ -48,6 +48,7 @@ private suspend fun getAsync(toolResultsStep: ToolResultsStep) = coroutineScope
 
 // Unfortunately is not possible to obtain from api exact the same timestamp as is in autogenerated test_result_1.xml from google cloud.
 // This one is a little bit lower but close as possible. The difference is around ~3 seconds.
-private fun List<TestCase>.getStartTimestamp(): Timestamp = minBy { testCase ->
-    testCase.startTime.asUnixTimestamp()
-}?.startTime ?: Timestamp()
+private fun List<TestCase>.getStartTimestamp(): Timestamp = this
+    .mapNotNull { it.startTime }
+    .minBy { it.asUnixTimestamp() }
+    ?: Timestamp()