fix(host_metrics): use assert!() insead of assert_eq!()

Signed-off-by: Sanskar Jaiswal <jaiswalsanskar078@gmail.com>
vectordotdev · Jan 3, 2025 · 9faccc7 · 9faccc7
1 parent b11bf61
commit 9faccc7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/src/sources/host_metrics/netlink_tcp.rs b/src/sources/host_metrics/netlink_tcp.rs
@@ -272,7 +272,7 @@ mod tests {
                 dst = true;
             }
         }
-        assert_eq!(source, true);
-        assert_eq!(dst, true);
+        assert!(source);
+        assert!(dst);
     }
 }
diff --git a/src/sources/host_metrics/network.rs b/src/sources/host_metrics/network.rs
@@ -196,13 +196,11 @@ mod tests {
         // and the network_tcp_connections_total has the "state" tag.
         #[cfg(target_os = "linux")]
         {
-            let mut n_tcp_conns_total_metric = 0;
             let mut n_tx_queued_bytes_metric = 0;
             let mut n_rx_queued_bytes_metric = 0;
 
             metrics.iter().for_each(|metric| {
                 if metric.name() == NETWORK_TCP_CONNS_TOTAL {
-                    n_tcp_conns_total_metric += 1;
                     let tags = metric.tags().unwrap();
                     assert!(
                         tags.contains_key(TCP_CONN_STATE),
@@ -216,7 +214,6 @@ mod tests {
                     return;
                 }
             });
-            assert_eq!(n_tcp_conns_total_metric, 1);
             assert_eq!(n_tx_queued_bytes_metric, 1);
             assert_eq!(n_rx_queued_bytes_metric, 1);
         }