test: fixing unit tests and clippy

szattila98 · Nov 2, 2023 · 5841f42 · 5841f42
1 parent 1d49911
commit 5841f42
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/server/core/src/logic/common.rs b/server/core/src/logic/common.rs
@@ -23,7 +23,7 @@ pub mod tests {
     use sea_orm::{DatabaseBackend, DeriveActiveModelBehavior, DeriveEntityModel, MockDatabase};
     use std::net::{IpAddr, Ipv4Addr, SocketAddr};
 
-    use crate::config::{AppConfig, MailConfig};
+    use crate::config::{AppConfig, MailConfig, MetricsConfig};
     use crate::logic::common::find_entity_by_id;
 
     pub const TEST_STR: &str = "test";
@@ -128,6 +128,9 @@ pub mod tests {
                 smtp_username: TEST_STR.to_string(),
                 smtp_password: TEST_STR.to_string(),
             },
+            metrics_config: MetricsConfig {
+                metrics_address: SocketAddr::new(IpAddr::V4(Ipv4Addr::new(127, 0, 0, 1)), 3001)
+            }
         };
     }
 

diff --git a/server/core/src/logic/expense_operations.rs b/server/core/src/logic/expense_operations.rs
@@ -48,7 +48,7 @@ pub async fn find_expenses_by_user_id(
     let recurrences = recurrences?;
 
     assert!(
-        vec![predefined_expenses.len(), grouped_transactions.len()]
+        [predefined_expenses.len(), grouped_transactions.len()]
             .iter()
             .all(|&x| x == expenses.len()),
         "the lengths of the fetched expense related lists should always be equal"
@@ -141,7 +141,7 @@ pub async fn find_predefined_expenses(
     let mut recurrences: VecDeque<recurrence::Model> = recurrences?.into_iter().flatten().collect();
 
     assert!(
-        vec![currencies.len(), recurrences.len()]
+        [currencies.len(), recurrences.len()]
             .iter()
             .all(|&x| x == predefined_expenses.len()),
         "the lengths of the fetched predefined expense related lists should be equal"