Fix clippy errors in tests

.github/workflows/ci.yml

@@ -29,7 +29,7 @@ jobs:
           toolchain: stable
           components: clippy
 
-      - run: cargo clippy --workspace --all-features -- -D warnings
+      - run: cargo clippy --workspace --all-features --all-targets -- -D warnings
 
   rustfmt:
     runs-on: ubuntu-latest

Cargo.toml

@@ -45,7 +45,7 @@ features = ["full"]
 rustdoc-args = ["--cfg", "docsrs"]
 
 [lints.rust]
-unexpected_cfgs = { level = "warn", check-cfg = ['cfg(ci)'] }
+unexpected_cfgs = { level = "warn", check-cfg = ["cfg(ci)", "cfg(msrv)", "cfg(nightly)"] }
 
 [features]
 default = ["std"]

examples/deny_missing_docs.rs

@@ -1,6 +1,7 @@
 //! Some docs
 
 #![deny(missing_docs)]
+#![allow(dead_code)] // for illustration purposes
 
 use derive_more::{
     Add, AddAssign, Constructor, Deref, DerefMut, Display, From, FromStr, Index,
@@ -33,7 +34,6 @@ pub struct MyBoxedInt(Box<i32>);
 pub struct MyVec(Vec<i32>);
 
 /// Some docs
-#[allow(dead_code)]
 #[derive(Clone, Copy, TryInto)]
 #[derive(IsVariant)]
 enum MixedInts {

impl/Cargo.toml

@@ -44,7 +44,7 @@ features = ["full"]
 rustdoc-args = ["--cfg", "docsrs"]
 
 [lints.rust]
-unexpected_cfgs = { level = "warn", check-cfg = ['cfg(ci)'] }
+unexpected_cfgs = { level = "warn", check-cfg = ["cfg(ci)", "cfg(nighthly)"] }
 
 [features]
 default = []

impl/src/fmt/mod.rs

@@ -495,7 +495,6 @@ where
 mod fmt_attribute_spec {
     use itertools::Itertools as _;
     use quote::ToTokens;
-    use syn;
 
     use super::FmtAttribute;
 
@@ -574,7 +573,7 @@ mod placeholder_parse_fmt_string_spec {
     fn indicates_position_and_trait_name_for_each_fmt_placeholder() {
         let fmt_string = "{},{:?},{{}},{{{1:0$}}}-{2:.1$x}{par:#?}{:width$}";
         assert_eq!(
-            Placeholder::parse_fmt_string(&fmt_string),
+            Placeholder::parse_fmt_string(fmt_string),
             vec![
                 Placeholder {
                     arg: Parameter::Positional(0),

impl/src/into.rs

@@ -178,6 +178,7 @@ impl<'a> Expansion<'a> {
                 let tys: Vec<_> = fields_tys.validate_type(out_ty)?.collect();
 
                 Ok(quote! {
+                    #[allow(clippy::unused_unit)]
                     #[automatically_derived]
                     impl #impl_gens derive_more::core::convert::From<#r #lf #m #input_ident #ty_gens>
                      for ( #( #r #lf #m #tys ),* ) #where_clause

impl/src/utils.rs

@@ -1,7 +1,6 @@
 #![cfg_attr(
     not(all(feature = "add", feature = "mul")),
-    allow(dead_code),
-    allow(unused_mut)
+    allow(dead_code, unused_mut)
 )]
 
 use proc_macro2::TokenStream;

tests/add.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 use derive_more::Add;
 

tests/add_assign.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 use derive_more::AddAssign;
 

tests/as_mut.rs

@@ -1,5 +1,6 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code, clippy::unnecessary_mut_passed)]
+#![allow(clippy::unnecessary_mut_passed)] // testing correct signatures rather than actual code
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

tests/as_ref.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

tests/boats_display_derive.rs

@@ -35,6 +35,7 @@ fn tuple_struct() {
     assert_eq!(s, "Error code: 2");
 }
 
+#[allow(clippy::enum_variant_names)] // because of the original source
 #[derive(Display)]
 enum EnumError {
     #[display("Error code: {}", code)]

tests/constructor.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 use derive_more::Constructor;
 

tests/debug.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;
@@ -53,8 +53,8 @@ mod structs {
             use derive_more::Debug;
 
             const I32: i32 = 11;
-            const F64: f64 = 3.14;
-            const POINTER: &f64 = &3.14;
+            const F64: f64 = 3.15;
+            const POINTER: &f64 = &3.15;
 
             #[derive(Debug)]
             #[debug("{I32}")]
@@ -100,8 +100,8 @@ mod structs {
                 assert_eq!(format!("{:07?}", Octal), "0000013");
                 assert_eq!(format!("{:03?}", LowerHex), "00b");
                 assert_eq!(format!("{:03?}", UpperHex), "00B");
-                assert_eq!(format!("{:07?}", LowerExp), "03.14e0");
-                assert_eq!(format!("{:07?}", UpperExp), "03.14E0");
+                assert_eq!(format!("{:07?}", LowerExp), "03.15e0");
+                assert_eq!(format!("{:07?}", UpperExp), "03.15E0");
                 assert_eq!(format!("{:018?}", Pointer).len(), 18);
             }
 
@@ -113,7 +113,7 @@ mod structs {
                     use derive_more::Debug;
 
                     const I32: i32 = 11;
-                    const F64: f64 = 3.14;
+                    const F64: f64 = 3.15;
 
                     #[derive(Debug)]
                     #[debug("{I32:x?}")]
@@ -593,7 +593,7 @@ mod structs {
             fn assert() {
                 assert_eq!(format!("{:03?}", TupleOctal(9, 4)), "011");
                 assert_eq!(
-                    format!("{:.1?}", StructLowerExp { a: 7, b: 3.14 }),
+                    format!("{:.1?}", StructLowerExp { a: 7, b: 3.15 }),
                     "3.1e0",
                 );
             }
@@ -642,8 +642,8 @@ mod enums {
             use derive_more::Debug;
 
             const I32: i32 = 11;
-            const F64: f64 = 3.14;
-            const POINTER: &f64 = &3.14;
+            const F64: f64 = 3.15;
+            const POINTER: &f64 = &3.15;
 
             #[derive(Debug)]
             enum Unit {
@@ -675,8 +675,8 @@ mod enums {
                 assert_eq!(format!("{:07?}", Unit::Octal), "0000013");
                 assert_eq!(format!("{:03?}", Unit::LowerHex), "00b");
                 assert_eq!(format!("{:03?}", Unit::UpperHex), "00B");
-                assert_eq!(format!("{:07?}", Unit::LowerExp), "03.14e0");
-                assert_eq!(format!("{:07?}", Unit::UpperExp), "03.14E0");
+                assert_eq!(format!("{:07?}", Unit::LowerExp), "03.15e0");
+                assert_eq!(format!("{:07?}", Unit::UpperExp), "03.15E0");
                 assert_eq!(format!("{:018?}", Unit::Pointer).len(), 18);
             }
 
@@ -688,7 +688,7 @@ mod enums {
                     use derive_more::Debug;
 
                     const I32: i32 = 11;
-                    const F64: f64 = 3.14;
+                    const F64: f64 = 3.15;
 
                     #[derive(Debug)]
                     enum Unit {
@@ -1069,7 +1069,7 @@ mod enums {
             fn assert() {
                 assert_eq!(format!("{:03?}", Enum::TupleOctal(9, 4)), "011");
                 assert_eq!(
-                    format!("{:.1?}", Enum::StructLowerExp { a: 7, b: 3.14 }),
+                    format!("{:.1?}", Enum::StructLowerExp { a: 7, b: 3.15 }),
                     "3.1e0",
                 );
             }
@@ -1840,7 +1840,7 @@ mod generic {
         #[test]
         fn assert() {
             assert_eq!(format!("{:03?}", Tuple(9)), "011");
-            assert_eq!(format!("{:.1?}", Struct { a: 9, b: 3.14 }), "3.1e0");
+            assert_eq!(format!("{:.1?}", Struct { a: 9, b: 3.15 }), "3.1e0");
             assert_eq!(format!("{:03?}", Enum::<_, u8, u8>::Debug(7)), "007");
             assert_eq!(
                 format!("{:03?}", Enum::<u8, _, _>::Display { b: 7, c: 8 }),

tests/deref.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code, unused_imports)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

tests/deref_mut.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code, unused_imports)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

tests/display.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code, unused_imports)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;
@@ -91,8 +91,8 @@ mod structs {
                 use super::*;
 
                 const I32: i32 = 11;
-                const F64: f64 = 3.14;
-                const POINTER: &f64 = &3.14;
+                const F64: f64 = 3.15;
+                const POINTER: &f64 = &3.15;
 
                 #[derive(Display)]
                 #[display("{I32}")]
@@ -138,8 +138,8 @@ mod structs {
                     assert_eq!(format!("{:07}", Octal), "0000013");
                     assert_eq!(format!("{:03}", LowerHex), "00b");
                     assert_eq!(format!("{:03}", UpperHex), "00B");
-                    assert_eq!(format!("{:07}", LowerExp), "03.14e0");
-                    assert_eq!(format!("{:07}", UpperExp), "03.14E0");
+                    assert_eq!(format!("{:07}", LowerExp), "03.15e0");
+                    assert_eq!(format!("{:07}", UpperExp), "03.15E0");
                     assert_eq!(format!("{:018}", Pointer).len(), 18);
                 }
             }
@@ -151,7 +151,7 @@ mod structs {
                     use super::*;
 
                     const I32: i32 = 11;
-                    const F64: f64 = 3.14;
+                    const F64: f64 = 3.15;
 
                     #[derive(Display)]
                     #[display("{I32:x?}")]
@@ -728,8 +728,8 @@ mod enums {
                 use super::*;
 
                 const I32: i32 = 11;
-                const F64: f64 = 3.14;
-                const POINTER: &f64 = &3.14;
+                const F64: f64 = 3.15;
+                const POINTER: &f64 = &3.15;
 
                 #[derive(Display)]
                 enum Unit {
@@ -761,8 +761,8 @@ mod enums {
                     assert_eq!(format!("{:07}", Unit::Octal), "0000013");
                     assert_eq!(format!("{:03}", Unit::LowerHex), "00b");
                     assert_eq!(format!("{:03}", Unit::UpperHex), "00B");
-                    assert_eq!(format!("{:07}", Unit::LowerExp), "03.14e0");
-                    assert_eq!(format!("{:07}", Unit::UpperExp), "03.14E0");
+                    assert_eq!(format!("{:07}", Unit::LowerExp), "03.15e0");
+                    assert_eq!(format!("{:07}", Unit::UpperExp), "03.15E0");
                     assert_eq!(format!("{:018}", Unit::Pointer).len(), 18);
                 }
             }
@@ -774,7 +774,7 @@ mod enums {
                     use super::*;
 
                     const I32: i32 = 11;
-                    const F64: f64 = 3.14;
+                    const F64: f64 = 3.15;
 
                     #[derive(Display)]
                     enum Unit {

tests/error/derives_for_enums_with_source.rs

@@ -1,4 +1,5 @@
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
+
 use super::*;
 
 derive_display!(TestErr);

tests/error/derives_for_generic_enums_with_source.rs

@@ -1,4 +1,5 @@
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
+
 use super::*;
 
 derive_display!(TestErr, T, E);

tests/error/derives_for_generic_structs_with_source.rs

@@ -1,4 +1,5 @@
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
+
 use super::*;
 
 #[test]

tests/error/derives_for_structs_with_source.rs

@@ -1,4 +1,5 @@
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
+
 use super::*;
 
 #[test]

tests/error/nightly/derives_for_enums_with_backtrace.rs

@@ -1,4 +1,6 @@
-#![allow(dead_code)]
+// For creating backtraces with different addresses in tests.
+#![allow(clippy::redundant_closure, clippy::redundant_closure_call)] // for testing
+#![allow(dead_code)] // some code is tested for type checking only
 
 use core::error::{request_ref, request_value};
 

tests/error/nightly/derives_for_generic_enums_with_backtrace.rs

@@ -1,4 +1,6 @@
-#![allow(dead_code)]
+// For creating backtraces with different addresses in tests.
+#![allow(clippy::redundant_closure, clippy::redundant_closure_call)] // for testing
+#![allow(dead_code)] // some code is tested for type checking only
 
 use core::error::{request_ref, request_value};
 

tests/error/nightly/derives_for_generic_structs_with_backtrace.rs

@@ -1,4 +1,6 @@
-#![allow(dead_code)]
+// For creating backtraces with different addresses in tests.
+#![allow(clippy::redundant_closure, clippy::redundant_closure_call)] // for testing
+#![allow(dead_code)] // some code is tested for type checking only
 
 use core::error::{request_ref, request_value};
 

tests/error/nightly/derives_for_structs_with_backtrace.rs

@@ -1,4 +1,6 @@
-#![allow(dead_code)]
+// For creating backtraces with different addresses in tests.
+#![allow(clippy::redundant_closure, clippy::redundant_closure_call)] // for testing
+#![allow(dead_code)] // some code is tested for type checking only
 
 use core::error::{request_ref, request_value};
 

tests/error_tests.rs

@@ -1,4 +1,4 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![cfg_attr(nightly, feature(error_generic_member_access, error_in_core))]
+#![cfg_attr(nightly, feature(error_generic_member_access))]
 
 mod error;

tests/from.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

tests/from_str.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

tests/generics.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code, non_camel_case_types)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 use derive_more::{
     Add, AddAssign, Constructor, Deref, DerefMut, Display, Error, From, FromStr, Index,

tests/index.rs

@@ -1,5 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![allow(dead_code, unused_imports)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

tests/index_mut.rs

@@ -1,4 +1,4 @@
-#![allow(dead_code, unused_imports)]
+#![allow(dead_code)] // some code is tested for type checking only
 
 use derive_more::IndexMut;