Fix clippy lints in tests

.github/workflows/test.yml

@@ -60,4 +60,4 @@ jobs:
       - uses: actions-rs/cargo@v1
         with:
           command: clippy
-          args: -- -D warnings
+          args: --all-targets -- -D warnings

src/main.rs

@@ -441,7 +441,7 @@ mod tests {
 
         let cleaned_projects = cleanup(&mocked_deps, &mut registry).unwrap();
         assert_eq!(cleaned_projects.len(), 1);
-        assert_eq!(cleaned_projects.get(0).unwrap().0, String::from("app3"));
+        assert_eq!(cleaned_projects.first().unwrap().0, String::from("app3"));
     }
 
     #[test]

src/mocks.rs

@@ -12,11 +12,7 @@ use unimock::{matching, Clause, MockFn, Unimock};
 pub fn args_mock(args: &str) -> impl Clause {
     ArgsMock
         .each_call(matching!())
-        .returns(
-            args.split(' ')
-                .map(|arg| String::from(arg))
-                .collect::<Vec<_>>(),
-        )
+        .returns(args.split(' ').map(String::from).collect::<Vec<_>>())
         .n_times(1)
 }
 
@@ -28,17 +24,17 @@ pub fn choose_port_mock() -> impl Clause {
 }
 
 pub fn cwd_mock(project: &str) -> impl Clause {
-    let path = PathBuf::from(format!("/projects/{}", project));
+    let path = PathBuf::from(format!("/projects/{project}"));
     WorkingDirectoryMock
         .each_call(matching!())
-        .answers(move |_| Ok(path.clone()))
+        .answers(move |()| Ok(path.clone()))
         .at_least_times(1)
 }
 
 pub fn data_dir_mock() -> impl Clause {
     DataDirMock
         .each_call(matching!())
-        .answers(|_| Ok(std::path::PathBuf::from("/data")))
+        .answers(|()| Ok(std::path::PathBuf::from("/data")))
         .at_least_times(1)
 }
 

src/registry.rs

@@ -241,7 +241,7 @@ impl Registry {
             bail!("Project {project_name} does not exist");
         }
 
-        for (name, project) in &mut self.projects.iter_mut() {
+        for (name, project) in &mut self.projects {
             if project.port == linked_port {
                 // Take the port from the project so that it can be used by the linked port
                 project.port = self.allocator.allocate(deps, None)?;
@@ -598,7 +598,7 @@ linked_port = 3000",
         assert!(registry
             .create(
                 &mocked_deps,
-                &"app4",
+                "app4",
                 Some(PathBuf::from("/projects/app3")),
                 None,
             )