Implement missing syncinfo

crates/core/app/tests/common/temp_storage_ext.rs

@@ -14,7 +14,6 @@ pub trait TempStorageExt: Sized {
 
 #[async_trait]
 impl TempStorageExt for TempStorage {
-    // TODO: move this to an Ext trait for Storage instead of TempStorage
     async fn new_with_penumbra_prefixes() -> anyhow::Result<TempStorage> {
         TempStorage::new_with_prefixes(SUBSTORE_PREFIXES.to_vec()).await
     }

crates/test/mock-client/src/lib.rs

@@ -42,16 +42,6 @@ impl MockClient {
         Ok(self)
     }
 
-    pub async fn with_sync_to_inner_storage(
-        mut self,
-        storage: cnidarium::Storage,
-    ) -> anyhow::Result<Self> {
-        let latest = storage.latest_snapshot();
-        self.sync_to_latest(latest).await?;
-
-        Ok(self)
-    }
-
     pub async fn sync_to_latest<R: StateRead>(&mut self, state: R) -> anyhow::Result<()> {
         let height = state.get_block_height().await?;
         self.sync_to(height, state).await?;

crates/test/mock-tendermint-proxy/src/proxy.rs

@@ -52,8 +52,8 @@ impl TestNodeProxy {
         Box::new(move |block| inner.on_block(block))
     }
 
-    /// Returns the last committed block height.
-    fn last_block_height(&self) -> tendermint::block::Height {
+    /// Returns the latest committed block height.
+    fn latest_block_height(&self) -> tendermint::block::Height {
         self.inner
             .blocks()
             .last_key_value()
@@ -164,12 +164,21 @@ impl TendermintProxyService for TestNodeProxy {
         req: tonic::Request<GetStatusRequest>,
     ) -> Result<tonic::Response<GetStatusResponse>, Status> {
         let GetStatusRequest { .. } = req.into_inner();
-        let latest_block_height = self.last_block_height().into();
+        let latest_block_height = self.latest_block_height().into();
         let block_ts: tendermint_proto::google::protobuf::Timestamp = self.timestamp().into();
         let sync_info = SyncInfo {
-            // TODO: these should get set
-            latest_block_hash: vec![],
-            latest_app_hash: vec![],
+            latest_block_hash: self
+                .inner
+                .blocks()
+                .last_key_value()
+                .map(|(_, b)| b.header.hash().into())
+                .unwrap_or_default(),
+            latest_app_hash: self
+                .inner
+                .blocks()
+                .last_key_value()
+                .map(|(_, b)| b.header.app_hash.clone().into())
+                .unwrap_or_default(),
             latest_block_height,
             latest_block_time: Some(pbjson_types::Timestamp {
                 seconds: block_ts.seconds,