BytesPs is Copy, no need to .clone()

helium · Jun 11, 2024 · 419b273 · 419b273
1 parent cfc884a
commit 419b273
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/coverage_point_calculator/tests/coverage_point_calculator.rs b/coverage_point_calculator/tests/coverage_point_calculator.rs
@@ -404,6 +404,8 @@ impl BoostedHexMap for NoBoostedHexes {
 }
 
 fn speedtests(tier: SpeedtestTier) -> Vec<Speedtest> {
+    // SpeedtestTier is determined solely by upload_speed.
+    // Other values are far surpassing ::Good.
     let upload_speed = BytesPs::mbps(match tier {
         SpeedtestTier::Good => 10,
         SpeedtestTier::Acceptable => 8,
@@ -414,13 +416,13 @@ fn speedtests(tier: SpeedtestTier) -> Vec<Speedtest> {
 
     vec![
         Speedtest {
-            upload_speed: upload_speed.clone(),
+            upload_speed,
             download_speed: BytesPs::mbps(150),
             latency_millis: 0,
             timestamp: Utc::now(),
         },
         Speedtest {
-            upload_speed: upload_speed.clone(),
+            upload_speed,
             download_speed: BytesPs::mbps(150),
             latency_millis: 0,
             timestamp: Utc::now(),