Clippy fixes

catanatron_rust/src/game.rs

@@ -91,7 +91,7 @@ mod tests {
 
         assert_eq!(state.generate_playable_actions().len(), 54);
         play_tick(&players, &mut state);
-        assert_eq!(state.is_initial_build_phase(), true);
+        assert!(state.is_initial_build_phase());
         assert_eq!(state.generate_playable_actions().len(), 3);
     }
 }

catanatron_rust/src/state.rs

@@ -195,8 +195,8 @@ mod tests {
     fn test_initial_build_phase() {
         let state = setup_state();
 
-        assert_eq!(state.is_initial_build_phase(), true);
-        assert_eq!(state.is_moving_robber(), false);
-        assert_eq!(state.is_discarding(), false);
+        assert!(state.is_initial_build_phase());
+        assert!(!state.is_moving_robber());
+        assert!(!state.is_discarding());
     }
 }

catanatron_rust/src/state/mutations.rs

@@ -20,7 +20,7 @@ impl State {
             let component = HashSet::from([node_id]);
             self.connected_components
                 .entry(color)
-                .or_insert(Vec::new())
+                .or_default()
                 .push(component);
         } else {
             todo!();
@@ -36,7 +36,7 @@ impl State {
 
         let is_initial_build_phase = self.is_initial_build_phase();
         if !is_initial_build_phase {
-            freqdeck_sub(&mut self.get_mut_player_hand(color), SETTLEMENT_COST);
+            freqdeck_sub(self.get_mut_player_hand(color), SETTLEMENT_COST);
             freqdeck_add(&mut self.vector[BANK_RESOURCE_SLICE], SETTLEMENT_COST);
         }
     }

catanatron_rust/src/state_vector.rs

@@ -1,4 +1,4 @@
-use crate::{decks::starting_dev_listdeck, map_instance::MapInstance};
+use crate::decks::starting_dev_listdeck;
 use rand::seq::SliceRandom;
 
 use crate::enums::COLORS;