Merge pull request #1505 from cashapp/jwilson.0117.nicer_error

Friendlier error when freshAtEpochMs is null
cashapp · Jan 17, 2025 · 5e737ce · 5e737ce
2 parents 136188d + a75494c
commit 5e737ce
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/...e-loader/src/commonMain/kotlin/app/cash/zipline/loader/internal/fetcher/LoadedManifest.kt b/...e-loader/src/commonMain/kotlin/app/cash/zipline/loader/internal/fetcher/LoadedManifest.kt
@@ -45,5 +45,7 @@ internal fun LoadedManifest(manifestBytes: ByteString, freshAtEpochMs: Long): Lo
 
 internal fun LoadedManifest(manifestBytes: ByteString): LoadedManifest {
   val manifest = ZiplineManifest.decodeJson(manifestBytes.utf8())
-  return LoadedManifest(manifestBytes, manifest, manifest.freshAtEpochMs!!)
+  val freshAtEpochMs = manifest.freshAtEpochMs
+    ?: error("freshAtEpochMs is required for loaded manifests, but was null")
+  return LoadedManifest(manifestBytes, manifest, freshAtEpochMs)
 }