Make clippy happy

Author: la10736

rstest/src/timeout.rs

@@ -23,7 +23,7 @@ pub fn execute_with_timeout_sync<T: 'static + Send, F: FnOnce() -> T + Send + 's
             handle.join().unwrap().unwrap();
             result
         }
-        Err(mpsc::RecvTimeoutError::Timeout) => panic!("Timeout {:?} expired", timeout),
+        Err(mpsc::RecvTimeoutError::Timeout) => panic!("Timeout {timeout:?} expired"),
         Err(mpsc::RecvTimeoutError::Disconnected) => match handle.join() {
             Err(any) => std::panic::resume_unwind(any),
             Ok(_) => unreachable!(),

rstest_macros/src/parse/arguments.rs

@@ -95,12 +95,12 @@ impl Args {
 #[derive(Clone, PartialEq, Debug)]
 pub enum TestAttr {
     InAttrs,
-    Explicit(syn::Attribute),
+    Explicit(Box<syn::Attribute>),
 }
 
 impl From<syn::Attribute> for TestAttr {
     fn from(attr: syn::Attribute) -> Self {
-        TestAttr::Explicit(attr)
+        TestAttr::Explicit(attr.into())
     }
 }
 

rstest_macros/src/parse/fixture.rs

@@ -173,7 +173,7 @@ pub(crate) struct FixtureData {
 impl FixtureData {
     pub(crate) fn fixtures(&self) -> impl Iterator<Item = &Fixture> {
         self.items.iter().filter_map(|f| match f {
-            FixtureItem::Fixture(ref fixture) => Some(fixture),
+            FixtureItem::Fixture(ref fixture) => Some(fixture.as_ref()),
             _ => None,
         })
     }
@@ -212,13 +212,13 @@ impl ArgumentValue {
 
 #[derive(PartialEq, Debug)]
 pub(crate) enum FixtureItem {
-    Fixture(Fixture),
+    Fixture(Box<Fixture>),
     ArgumentValue(Box<ArgumentValue>),
 }
 
 impl From<Fixture> for FixtureItem {
     fn from(f: Fixture) -> Self {
-        FixtureItem::Fixture(f)
+        FixtureItem::Fixture(f.into())
     }
 }
 
@@ -235,7 +235,7 @@ impl Parse for FixtureItem {
 impl RefPat for FixtureItem {
     fn pat(&self) -> &Pat {
         match self {
-            FixtureItem::Fixture(Fixture { ref arg, .. }) => arg,
+            FixtureItem::Fixture(ref fix) => &fix.arg,
             FixtureItem::ArgumentValue(ref av) => &av.arg,
         }
     }

rstest_macros/src/parse/mod.rs

@@ -334,7 +334,7 @@ impl VisitMut for PartialsTypeFunctionExtractor {
                 Err(e) => errors.push(e),
             }
         }
-        self.0 = if errors.len() > 0 {
+        self.0 = if !errors.is_empty() {
             Err(errors)
         } else {
             Ok(data)

rstest_macros/src/parse/rstest/files.rs

@@ -85,8 +85,7 @@ impl FilesGlobReferences {
                 (None, _) if ignore_missing => String::new(),
                 (None, _) => {
                     return Err(attr.error(&format!(
-                        "Could not find the environment variable {:?}",
-                        var_name
+                        "Could not find the environment variable {var_name:?}"
                     )))
                 }
             };

rstest_macros/src/parse/rstest/test_attr.rs

@@ -70,7 +70,7 @@ mod should {
     use crate::test::{attr, ToAst};
 
     fn explicit(a: &str) -> TestAttr {
-        TestAttr::Explicit(attr(format!("#[{a}]")))
+        TestAttr::Explicit(attr(format!("#[{a}]")).into())
     }
 
     fn in_attrs() -> TestAttr {

rstest_macros/src/render/test.rs

@@ -1978,7 +1978,7 @@ mod test_attribute_should {
         let mut out = String::from("#[rstest]\n");
         match attr_style {
             TestAttrStyle::Explicit => {
-                info.arguments.set_test_attr(Some(TestAttr::Explicit(attr("#[my_explicit_test_attr]"))));
+                info.arguments.set_test_attr(Some(TestAttr::Explicit(attr("#[my_explicit_test_attr]").into())));
             }
             TestAttrStyle::Implicit => {
                 info.arguments.set_test_attr(Some(TestAttr::InAttrs));

rstest_test/src/prj.rs

@@ -150,7 +150,7 @@ impl Project {
                 self
             }
 
-            code => panic!("cargo init return an error code: {}", code),
+            code => panic!("cargo init return an error code: {code}"),
         }
     }