more clippy to fix CI

mozilla · Nov 7, 2022 · 71a81aa · 71a81aa
1 parent c7c74e1
commit 71a81aa
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 12 deletions.
diff --git a/src/cobertura.rs b/src/cobertura.rs
@@ -704,7 +704,7 @@ mod tests {
     fn test_cobertura() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_cobertura.xml";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![(
             PathBuf::from("src/main.rs"),
@@ -738,7 +738,7 @@ mod tests {
     fn test_cobertura_double_lines() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_cobertura.xml";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![(
             PathBuf::from("src/main.rs"),
@@ -770,7 +770,7 @@ mod tests {
     fn test_cobertura_multiple_files() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_cobertura.xml";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![
             (

diff --git a/src/main.rs b/src/main.rs
@@ -118,7 +118,7 @@ struct Opt {
             - *ade* for the ActiveData-ETL specific format;\n\
             - *files* to only return a list of files.\n\
             - *markdown* for human easy read.\n\
-            - *cobertura* for output in coberura format.\n\
+            - *cobertura* for output in cobertura format.\n\
         ",
         value_name = "OUTPUT TYPE",
         default_value = "lcov",

diff --git a/src/output.rs b/src/output.rs
@@ -670,7 +670,7 @@ mod tests {
     fn test_lcov_brf_brh() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_lcov_brf_brh.info";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![(
             PathBuf::from("foo/bar/a.cpp"),
@@ -700,7 +700,7 @@ mod tests {
     fn test_lcov_demangle() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_lcov_demangle";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![(
             PathBuf::from("foo/bar/a.cpp"),
@@ -749,7 +749,7 @@ mod tests {
     fn test_covdir() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_covdir.json";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![
             (
@@ -793,7 +793,7 @@ mod tests {
         output_covdir(&results, Some(&file_path));
 
         let results: Value = serde_json::from_str(&read_file(&file_path)).unwrap();
-        let expected_path = PathBuf::from("./test/").join(&file_name);
+        let expected_path = PathBuf::from("./test/").join(file_name);
         let expected: Value = serde_json::from_str(&read_file(&expected_path)).unwrap();
 
         assert_eq!(results, expected);
@@ -803,7 +803,7 @@ mod tests {
     fn test_coveralls_service_job_id() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_coveralls_service_job_id.json";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![(
             PathBuf::from("foo/bar/a.cpp"),
@@ -842,7 +842,7 @@ mod tests {
     fn test_coveralls_token_field_is_absent_if_arg_is_none() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_coveralls_token.json";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![(
             PathBuf::from("foo/bar/a.cpp"),
@@ -881,7 +881,7 @@ mod tests {
     fn test_coveralls_service_fields_are_absent_if_args_are_none() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_coveralls_service_fields.json";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![(
             PathBuf::from("foo/bar/a.cpp"),
@@ -922,7 +922,7 @@ mod tests {
     fn test_markdown() {
         let tmp_dir = tempfile::tempdir().expect("Failed to create temporary directory");
         let file_name = "test_markdown";
-        let file_path = tmp_dir.path().join(&file_name);
+        let file_path = tmp_dir.path().join(file_name);
 
         let results = vec![
             (