Clippy clean

derive/src/newtype.rs

@@ -76,6 +76,7 @@ pub fn generate(newtype_args: &args::NewType) -> GeneratorResult<TokenStream> {
             }
         }
 
+        #[allow(clippy::from_over_into)]
         impl #impl_generics ::std::convert::Into<#inner_ty> for #ident #ty_generics #where_clause {
             fn into(self) -> #inner_ty {
                 self.0

src/types/maybe_undefined.rs

@@ -408,37 +408,31 @@ mod tests {
 
     #[test]
     fn test_as_opt_ref() {
-        let mut value: MaybeUndefined<String>;
-        let mut r: Option<Option<&String>>;
-
-        value = MaybeUndefined::Undefined;
-        r = value.as_opt_ref();
+        let value = MaybeUndefined::<String>::Undefined;
+        let r = value.as_opt_ref();
         assert_eq!(r, None);
 
-        value = MaybeUndefined::Null;
-        r = value.as_opt_ref();
+        let value = MaybeUndefined::<String>::Null;
+        let r = value.as_opt_ref();
         assert_eq!(r, Some(None));
 
-        value = MaybeUndefined::Value("abc".to_string());
-        r = value.as_opt_ref();
+        let value = MaybeUndefined::<String>::Value("abc".to_string());
+        let r = value.as_opt_ref();
         assert_eq!(r, Some(Some(&"abc".to_string())));
     }
 
     #[test]
     fn test_as_opt_deref() {
-        let mut value: MaybeUndefined<String>;
-        let mut r: Option<Option<&str>>;
-
-        value = MaybeUndefined::Undefined;
-        r = value.as_opt_deref();
+        let value = MaybeUndefined::<String>::Undefined;
+        let r = value.as_opt_deref();
         assert_eq!(r, None);
 
-        value = MaybeUndefined::Null;
-        r = value.as_opt_deref();
+        let value = MaybeUndefined::<String>::Null;
+        let r = value.as_opt_deref();
         assert_eq!(r, Some(None));
 
-        value = MaybeUndefined::Value("abc".to_string());
-        r = value.as_opt_deref();
+        let value = MaybeUndefined::<String>::Value("abc".to_string());
+        let r = value.as_opt_deref();
         assert_eq!(r, Some(Some("abc")));
     }
 

tests/error_ext.rs

@@ -92,19 +92,17 @@ pub async fn test_failure() {
     #[Object]
     impl Query {
         async fn failure(&self) -> Result<i32> {
-            Err(Error::new_with_source(MyError::Error1).into())
+            Err(Error::new_with_source(MyError::Error1))
         }
 
         async fn failure2(&self) -> Result<i32> {
-            Err(Error::new_with_source(MyError::Error2))?;
-            Ok(1)
+            Err(Error::new_with_source(MyError::Error2))
         }
 
         async fn failure3(&self) -> Result<i32> {
             Err(Error::new_with_source(MyError::Error1)
                 .extend_with(|_, values| values.set("a", 1))
-                .extend_with(|_, values| values.set("b", 2)))?;
-            Ok(1)
+                .extend_with(|_, values| values.set("b", 2)))
         }
 
         async fn failure4(&self) -> Result<i32> {
@@ -180,7 +178,7 @@ pub async fn test_failure2() {
     #[Object]
     impl Query {
         async fn failure(&self) -> Result<i32, MyError> {
-            Err(MyError::Error1)?
+            Err(MyError::Error1)
         }
     }
 }

tests/generic_types.rs

@@ -318,11 +318,11 @@ pub async fn test_concrete_object() {
     #[Object]
     impl Query {
         async fn a(&self) -> GbObject<i32, i64> {
-            GbObject { 0: 10, 1: 20 }
+            GbObject(10, 20)
         }
 
         async fn b(&self) -> GbObject<f32, f64> {
-            GbObject { 0: 88.0, 1: 99.0 }
+            GbObject(88.0, 99.0)
         }
     }