style: simplify string formatting for readability

Author: hamirmahal

integration-tests/tests/integration/fixtures.rs

@@ -285,8 +285,7 @@ pub fn check_list_binaries_output(stdout: &[u8]) {
 
     assert_eq!(
         expected_binary_ids, actual_binary_ids,
-        "expected binaries:\n{:?}\nactual binaries\n{:?}",
-        expected_binary_ids, actual_binary_ids
+        "expected binaries:\n{expected_binary_ids:?}\nactual binaries\n{actual_binary_ids:?}"
     );
 }
 

integration-tests/tests/integration/main.rs

@@ -441,7 +441,7 @@ fn test_run_from_archive_with_no_includes() {
         _ = extracted_target
             .join(path)
             .symlink_metadata()
-            .map(|_| panic!("file {} must not be included in the archive", path));
+            .map(|_| panic!("file {path} must not be included in the archive"));
     }
 }
 
@@ -478,7 +478,7 @@ archive.include = [
         _ = extracted_target
             .join(path)
             .symlink_metadata()
-            .map(|_| panic!("file {} must not be included in the archive", path));
+            .map(|_| panic!("file {path} must not be included in the archive"));
     }
 }
 

integration-tests/tests/integration/temp_project.rs

@@ -143,7 +143,7 @@ mod unix {
         // This creates a Unix domain socket by binding it to a path.
         use std::os::unix::net::UnixListener;
 
-        UnixListener::bind(path).wrap_err_with(|| format!("failed to bind UDS at {}", path))?;
+        UnixListener::bind(path).wrap_err_with(|| format!("failed to bind UDS at {path}"))?;
         Ok(UdsStatus::Created)
     }
 }

nextest-filtering/tests/match.rs

@@ -196,7 +196,7 @@ fn test_expr_package_regex() {
 fn test_expr_binary_id_glob() {
     let graph = load_graph();
     let expr = parse("binary_id(crate_[ab])", &graph);
-    println!("{:?}", expr);
+    println!("{expr:?}");
 
     let pid_a = mk_pid('a');
     let pid_b = mk_pid('b');

nextest-runner/src/cargo_config/custom_platform.rs

@@ -34,7 +34,7 @@ impl ExtractedCustomPlatform {
                 error,
             })?;
 
-        let path = temp_dir.path().join(format!("{}.json", triple_str));
+        let path = temp_dir.path().join(format!("{triple_str}.json"));
 
         std::fs::write(&path, json).map_err(|error| {
             TargetTripleError::CustomPlatformWriteError {

nextest-runner/src/config/archive.rs

@@ -161,7 +161,7 @@ impl RecursionDepth {
 impl fmt::Display for RecursionDepth {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         match self {
-            Self::Finite(n) => write!(f, "{}", n),
+            Self::Finite(n) => write!(f, "{n}"),
             Self::Infinite => write!(f, "infinite"),
         }
     }

nextest-runner/src/config/helpers.rs

@@ -54,8 +54,8 @@ mod tests {
 
         for &input in valid {
             let path = de_relative_path(input.into_deserializer())
-                .wrap_err_with(|| format!("error deserializing valid path {:?}: error", input))?;
-            assert_eq!(path, Utf8PathBuf::from(input), "path matches: {:?}", path);
+                .wrap_err_with(|| format!("error deserializing valid path {input:?}: error"))?;
+            assert_eq!(path, Utf8PathBuf::from(input), "path matches: {path:?}");
         }
 
         for &input in invalid {
@@ -66,8 +66,7 @@ mod tests {
             assert_eq!(
                 error.to_string(),
                 format!(
-                    "invalid value: string {:?}, expected a relative path with no parent components",
-                    input
+                    "invalid value: string {input:?}, expected a relative path with no parent components"
                 )
             );
         }

nextest-runner/src/config/nextest_version.rs

@@ -425,8 +425,7 @@ impl<'de> Deserialize<'de> for NextestVersionDeserialize {
                 if let (Some(required), Some(recommended)) = (&required, &recommended) {
                     if required > recommended {
                         return Err(serde::de::Error::custom(format!(
-                            "required version ({}) must not be greater than recommended version ({})",
-                            required, recommended
+                            "required version ({required}) must not be greater than recommended version ({recommended})"
                         )));
                     }
                 }
@@ -547,9 +546,7 @@ mod tests {
         let err = toml::from_str::<VersionOnlyDeserialize>(input).unwrap_err();
         assert!(
             err.to_string().contains(error_message),
-            "error `{}` contains `{}`",
-            err,
-            error_message
+            "error `{err}` contains `{error_message}`"
         );
     }
 

nextest-runner/src/config/scripts.rs

@@ -700,7 +700,7 @@ mod tests {
             ConfigParseErrorKind::ExperimentalFeatureNotEnabled { feature } => {
                 assert_eq!(*feature, ConfigExperimental::SetupScripts);
             }
-            other => panic!("unexpected error kind: {:?}", other),
+            other => panic!("unexpected error kind: {other:?}"),
         }
 
         // Now, check with the experimental feature enabled.

nextest-runner/src/helpers.rs

@@ -217,7 +217,7 @@ pub(crate) fn convert_rel_path_to_main_sep(rel_path: &Utf8Path) -> Utf8PathBuf {
 pub(crate) fn rel_path_join(rel_path: &Utf8Path, path: &Utf8Path) -> Utf8PathBuf {
     assert!(rel_path.is_relative(), "rel_path {rel_path} is relative");
     assert!(path.is_relative(), "path {path} is relative",);
-    format!("{}/{}", rel_path, path).into()
+    format!("{rel_path}/{path}").into()
 }
 
 #[derive(Debug)]