graph: Make attempts to create invalid BlockTime easier to diagnose

lutter · lutter · commit 7747c6375458 · 2024-02-13T10:52:51.000-08:00
We can't really pass a Result back since this is called in a lot of places
where Results aren't accepted. We make at least sure that the panic from
trying to create an invalid BlockTime is easier to diagnose from logs.
diff --git a/graph/src/blockchain/types.rs b/graph/src/blockchain/types.rs
@@ -341,7 +341,11 @@ impl BlockTime {
     /// Construct a block time that is the given number of seconds and
     /// nanoseconds after the Unix epoch
     pub fn since_epoch(secs: i64, nanos: u32) -> Self {
-        Self(DateTime::from_timestamp(secs, nanos).unwrap())
+        Self(
+            DateTime::from_timestamp(secs, nanos)
+                .ok_or_else(|| anyhow!("invalid block time: {}s {}ns", secs, nanos))
+                .unwrap(),
+        )
     }
 
     /// Construct a block time for tests where blocks are exactly 10s apart
