better err checks in tests

oxidecomputer · Oct 16, 2024 · 556ccb7 · 556ccb7
1 parent 108ec11
commit 556ccb7
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/nexus/db-queries/src/db/datastore/clickhouse_policy.rs b/nexus/db-queries/src/db/datastore/clickhouse_policy.rs
@@ -206,14 +206,18 @@ mod tests {
         assert!(datastore
             .clickhouse_policy_insert_latest_version(&opctx, &policy)
             .await
-            .is_err());
+            .unwrap_err()
+            .to_string()
+            .contains("policy version must be greater than 0"));
 
         // Inserting version 2 before version 1 should not work
         policy.version = 2;
         assert!(datastore
             .clickhouse_policy_insert_latest_version(&opctx, &policy)
             .await
-            .is_err());
+            .unwrap_err()
+            .to_string()
+            .contains("policy version 2 is not the most recent"));
 
         // Inserting version 1 should work
         policy.version = 1;
@@ -234,7 +238,9 @@ mod tests {
         assert!(datastore
             .clickhouse_policy_insert_latest_version(&opctx, &policy)
             .await
-            .is_err());
+            .unwrap_err()
+            .to_string()
+            .contains("policy version 4 is not the most recent"));
 
         // Inserting version 3 should work
         policy.version = 3;