modify test

Author: Jamesbarford

database/src/pool.rs

@@ -980,9 +980,28 @@ mod tests {
 
             db.mark_benchmark_request_as_completed(tag).await.unwrap();
 
-            let completed = db.load_benchmark_request_index().await.unwrap();
+            /* From the status page view we can see that the duration has been
+             * updated. Albeit that it will be a very short duration. */
+            let status_page_view = db.get_status_page_data().await.unwrap();
+            let req = &status_page_view
+                .completed_requests
+                .iter()
+                .find(|it| it.0.tag() == Some(tag))
+                .unwrap()
+                .0;
+
+            assert!(matches!(
+                req.status(),
+                BenchmarkRequestStatus::Completed { .. }
+            ));
+            let BenchmarkRequestStatus::Completed { duration_ms, .. } = req.status() else {
+                unreachable!();
+            };
+            assert!(duration_ms >= 1);
+
+            let completed_index = db.load_benchmark_request_index().await.unwrap();
+            assert!(completed_index.contains_tag("sha-1"));
 
-            assert!(completed.contains_tag("sha-1"));
             Ok(ctx)
         })
         .await;

database/src/pool/postgres.rs

@@ -2082,7 +2082,8 @@ where
                 (
                     row_to_benchmark_request(it),
                     // The errors, if there are none this will be an empty vector
-                    it.get::<_, Vec<String>>(11),
+                    it.get::<_, Option<Vec<String>>>(11)
+                        .unwrap_or_else(|| vec![]),
                 )
             })
             .collect();