tidiness

Author: hawkw

dev-tools/omdb/src/bin/omdb/db/sitrep.rs

@@ -238,7 +238,7 @@ async fn cmd_db_sitrep_show(
         }
     };
 
-    let fm::Sitrep { metadata, cases, alerts_requested } = sitrep;
+    let fm::Sitrep { metadata, cases } = sitrep;
     let fm::SitrepMetadata {
         id,
         creator_id,

nexus/db-queries/src/db/datastore/fm.rs

@@ -145,7 +145,6 @@ impl DataStore {
         Ok(Sitrep {
             metadata,
             // TODO(eliza) read these
-            alerts_requested: Default::default(),
             cases: Default::default(),
         })
     }
@@ -760,6 +759,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: None,
             },
+            cases: Default::default(),
         };
 
         datastore.fm_sitrep_insert(&opctx, &sitrep).await.unwrap();
@@ -806,6 +806,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: None,
             },
+            cases: Default::default(),
         };
         datastore.fm_sitrep_insert(&opctx, &sitrep1).await.unwrap();
 
@@ -819,6 +820,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: Some(sitrep1.id()),
             },
+            cases: Default::default(),
         };
         datastore.fm_sitrep_insert(&opctx, &sitrep2).await.expect(
             "inserting a sitrep whose parent is current should succeed",
@@ -859,6 +861,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: None,
             },
+            cases: Default::default(),
         };
         datastore.fm_sitrep_insert(&opctx, &sitrep1).await.unwrap();
 
@@ -873,6 +876,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: Some(nonexistent_id),
             },
+            cases: Default::default(),
         };
 
         let result = datastore.fm_sitrep_insert(&opctx, &sitrep2).await;
@@ -907,6 +911,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: None,
             },
+            cases: Default::default(),
         };
         datastore.fm_sitrep_insert(&opctx, &sitrep1).await.unwrap();
 
@@ -920,6 +925,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: Some(sitrep1.id()),
             },
+            cases: Default::default(),
         };
         datastore.fm_sitrep_insert(&opctx, &sitrep2).await.unwrap();
 
@@ -934,6 +940,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: Some(sitrep1.id()),
             },
+            cases: Default::default(),
         };
         let result = datastore.fm_sitrep_insert(&opctx, &sitrep3).await;
 
@@ -974,6 +981,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: None,
             },
+            cases: Default::default(),
         };
         datastore
             .fm_sitrep_insert(&opctx, &sitrep1)
@@ -1014,6 +1022,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: Some(sitrep1.metadata.id),
             },
+            cases: Default::default(),
         };
         datastore
             .fm_sitrep_insert(&opctx, &sitrep2)
@@ -1077,6 +1086,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id,
             },
+            cases: Default::default(),
         };
         match datastore.fm_sitrep_insert(&opctx, &sitrep).await {
             Ok(_) => {

nexus/src/app/background/tasks/fm_sitrep_gc.rs

@@ -152,6 +152,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: None,
             },
+            cases: Default::default(),
         };
         datastore
             .fm_sitrep_insert(&opctx, &sitrep1)
@@ -174,6 +175,7 @@ mod tests {
                 time_created: Utc::now(),
                 parent_sitrep_id: Some(sitrep1.metadata.id),
             },
+            cases: Default::default(),
         };
         datastore
             .fm_sitrep_insert(&opctx, &sitrep2)
@@ -264,7 +266,10 @@ mod tests {
                 comment: format!("test sitrep v{i}; orphan {i}"),
                 time_created: Utc::now(),
                 parent_sitrep_id,
+                // TODO(eliza): we should populate cases and assert they get
+                // cleaned up...
             },
+            cases: Default::default(),
         };
         match datastore.fm_sitrep_insert(&opctx, &sitrep).await {
             Ok(_) => {

nexus/src/app/background/tasks/fm_sitrep_load.rs

@@ -224,6 +224,7 @@ mod test {
                 comment: "test sitrep 1".to_string(),
                 time_created: Utc::now(),
             },
+            cases: Default::default(),
         };
         datastore
             .fm_sitrep_insert(&opctx, &sitrep1)
@@ -288,6 +289,7 @@ mod test {
                 comment: "test sitrep 2".to_string(),
                 time_created: Utc::now(),
             },
+            cases: Default::default(),
         };
         datastore
             .fm_sitrep_insert(&opctx, &sitrep2)