Fix lint

Author: owjs3901

crates/vespera_macro/src/args.rs

@@ -180,11 +180,8 @@ mod tests {
                         if let syn::Expr::Lit(syn::ExprLit {
                             lit: syn::Lit::Int(lit_int),
                             ..
-                        }) = elem
-                        {
-                            if let Ok(code) = lit_int.base10_parse::<u16>() {
+                        }) = elem &&  let Ok(code) = lit_int.base10_parse::<u16>() {
                                 status_codes.push(code);
-                            }
                         }
                     }
                     assert_eq!(

crates/vespera_macro/src/collector.rs

@@ -118,8 +118,6 @@ pub fn get_users() -> String {
 }
 "#,
         )],
-        1, // expected routes
-        0, // expected structs
         "get",
         "/users",
         "get_users",
@@ -136,8 +134,6 @@ pub fn create_user() -> String {
 }
 "#,
         )],
-        1,
-        0,
         "post",
         "/create-user",
         "create_user",
@@ -154,8 +150,6 @@ pub fn get_users() -> String {
 }
 "#,
         )],
-        1,
-        0,
         "get",
         "/users/api/users",
         "get_users",
@@ -172,8 +166,6 @@ pub fn get_users() -> String {
 }
 "#,
         )],
-        1,
-        0,
         "get",
         "/users",
         "get_users",
@@ -190,8 +182,6 @@ pub fn get_users() -> String {
 }
 "#,
         )],
-        1,
-        0,
         "get",
         "/api/users",
         "get_users",
@@ -208,8 +198,6 @@ pub fn get_users() -> String {
 }
 "#,
         )],
-        1,
-        0,
         "get",
         "/api/v1/users",
         "get_users",
@@ -218,8 +206,6 @@ pub fn get_users() -> String {
     fn test_collect_metadata_routes(
         #[case] folder_name: &str,
         #[case] files: Vec<(&str, &str)>,
-        #[case] expected_routes: usize,
-        #[case] expected_structs: usize,
         #[case] expected_method: &str,
         #[case] expected_path: &str,
         #[case] expected_function_name: &str,
@@ -233,22 +219,17 @@ pub fn get_users() -> String {
 
         let metadata = collect_metadata(temp_dir.path(), folder_name).unwrap();
 
-        assert_eq!(metadata.routes.len(), expected_routes);
-        assert_eq!(metadata.structs.len(), expected_structs);
-
-        if expected_routes > 0 {
-            let route = &metadata.routes[0];
-            assert_eq!(route.method, expected_method);
-            assert_eq!(route.path, expected_path);
-            assert_eq!(route.function_name, expected_function_name);
-            assert_eq!(route.module_path, expected_module_path);
-            if let Some((first_filename, _)) = files.first() {
-                assert!(
-                    route
-                        .file_path
-                        .contains(first_filename.split('/').next().unwrap())
-                );
-            }
+        let route = &metadata.routes[0];
+        assert_eq!(route.method, expected_method);
+        assert_eq!(route.path, expected_path);
+        assert_eq!(route.function_name, expected_function_name);
+        assert_eq!(route.module_path, expected_module_path);
+        if let Some((first_filename, _)) = files.first() {
+            assert!(
+                route
+                    .file_path
+                    .contains(first_filename.split('/').next().unwrap())
+            );
         }
 
         drop(temp_dir);

crates/vespera_macro/src/lib.rs

@@ -435,7 +435,7 @@ mod tests {
         let folder_name = "routes";
 
         let result = generate_router_code(
-            &collect_metadata(&temp_dir.path(), folder_name).unwrap(),
+            &collect_metadata(temp_dir.path(), folder_name).unwrap(),
             None,
             None,
         );
@@ -592,7 +592,7 @@ pub fn get_users() -> String {
         }
 
         let result = generate_router_code(
-            &collect_metadata(&temp_dir.path(), folder_name).unwrap(),
+            &collect_metadata(temp_dir.path(), folder_name).unwrap(),
             None,
             None,
         );
@@ -677,7 +677,7 @@ pub fn update_user() -> String {
         );
 
         let result = generate_router_code(
-            &collect_metadata(&temp_dir.path(), folder_name).unwrap(),
+            &collect_metadata(temp_dir.path(), folder_name).unwrap(),
             None,
             None,
         );
@@ -731,7 +731,7 @@ pub fn create_users() -> String {
         );
 
         let result = generate_router_code(
-            &collect_metadata(&temp_dir.path(), folder_name).unwrap(),
+            &collect_metadata(temp_dir.path(), folder_name).unwrap(),
             None,
             None,
         );
@@ -777,7 +777,7 @@ pub fn index() -> String {
         );
 
         let result = generate_router_code(
-            &collect_metadata(&temp_dir.path(), folder_name).unwrap(),
+            &collect_metadata(temp_dir.path(), folder_name).unwrap(),
             None,
             None,
         );
@@ -813,7 +813,7 @@ pub fn get_users() -> String {
         );
 
         let result = generate_router_code(
-            &collect_metadata(&temp_dir.path(), folder_name).unwrap(),
+            &collect_metadata(temp_dir.path(), folder_name).unwrap(),
             None,
             None,
         );

crates/vespera_macro/src/openapi_generator.rs

@@ -494,10 +494,8 @@ pub fn create_user() -> String {
             assert!(doc.components.as_ref().unwrap().schemas.is_some());
             let schemas = doc.components.as_ref().unwrap().schemas.as_ref().unwrap();
             assert!(schemas.contains_key("User"));
-        } else {
-            if let Some(schemas) = doc.components.as_ref().unwrap().schemas.as_ref() {
-                assert!(!schemas.contains_key("User"));
-            }
+        } else if let Some(schemas) = doc.components.as_ref().unwrap().schemas.as_ref() {
+            assert!(!schemas.contains_key("User"));
         }
 
         // Check route

crates/vespera_macro/src/parser/response.rs

@@ -377,10 +377,10 @@ mod tests {
                     assert_eq!(schema.schema_type, Some(SchemaType::Array));
                     assert!(schema.items.is_some());
                     // Check that items is String
-                    if let Some(items) = &schema.items {
-                        if let SchemaRef::Inline(items_schema) = items.as_ref() {
-                            assert_eq!(items_schema.schema_type, Some(SchemaType::String));
-                        }
+                    if let Some(items) = &schema.items
+                        && let SchemaRef::Inline(items_schema) = items.as_ref()
+                    {
+                        assert_eq!(items_schema.schema_type, Some(SchemaType::String));
                     }
                 } else {
                     panic!("Expected inline Array schema");

crates/vespera_macro/src/route/utils.rs

@@ -122,10 +122,10 @@ mod tests {
         let file: syn::File = syn::parse_str(&full_code).expect("Failed to parse with attribute");
 
         // Extract the first attribute from the function
-        if let Some(syn::Item::Fn(fn_item)) = file.items.first() {
-            if let Some(attr) = fn_item.attrs.first() {
-                return attr.meta.clone();
-            }
+        if let Some(syn::Item::Fn(fn_item)) = file.items.first()
+            && let Some(attr) = fn_item.attrs.first()
+        {
+            return attr.meta.clone();
         }
 
         panic!("Failed to extract meta from attribute: {}", attr_str);