Fix: Addressed CI errors

Author: tnsc

src/graphql/email.rs

@@ -911,7 +911,8 @@ mod tests {
     #[tokio::test]
     async fn test_email_validation_caching() {
         // Create a test Redis client with a short TTL
-        let email_query = EmailQuery::new("redis://127.0.0.1:6379", 5).unwrap_or_else(|_| EmailQuery::default());
+        let email_query =
+            EmailQuery::new("redis://127.0.0.1:6379", 5).unwrap_or_else(|_| EmailQuery::default());
 
         let test_email = "test@example.com";
 
@@ -994,7 +995,10 @@ mod tests {
         assert_eq!(original.is_valid, converted.is_valid);
         assert_eq!(original.status, converted.status);
         assert!(original.error.is_some() && converted.error.is_some());
-        assert_eq!(original.error.as_ref().unwrap().code, converted.error.as_ref().unwrap().code);
+        assert_eq!(
+            original.error.as_ref().unwrap().code,
+            converted.error.as_ref().unwrap().code
+        );
     }
 
     #[tokio::test]

src/graphql/schema.rs

@@ -130,7 +130,7 @@ mod tests {
         // Just ensure we can create a default instance
         // This tests the Default trait implementation
         let schema = Schema::build(root_query, EmptyMutation, EmptySubscription).finish();
-        
+
         let query = "{ health { status } }";
         let result = tokio_test::block_on(schema.execute(query));
         assert!(result.errors.is_empty());
@@ -161,7 +161,11 @@ mod tests {
         "#;
 
         let result = tokio_test::block_on(schema.execute(query));
-        assert!(result.errors.is_empty(), "GraphQL query has errors: {:?}", result.errors);
+        assert!(
+            result.errors.is_empty(),
+            "GraphQL query has errors: {:?}",
+            result.errors
+        );
 
         let data = result.data.into_json().unwrap();
         assert!(data["validateEmailsBulk"]["results"].is_array());

src/routes/email.rs

@@ -302,7 +302,7 @@ mod tests {
     async fn test_role_based_email_detection_when_enabled() {
         // Load environment variables from .env file for test
         dotenv::dotenv().ok();
-        
+
         let app = create_test_app().await;
         let req = test::TestRequest::post()
             .uri("/validate-email?check_role_based=true")
@@ -387,11 +387,11 @@ mod tests {
     #[actix_web::test]
     async fn test_redis_cache_methods() {
         let redis_cache = RedisCache::test_dummy();
-        
+
         // Test get_dns_validation with cache miss
         let result = redis_cache.get_dns_validation("example.com").await;
         assert!(result.is_ok());
-        
+
         // Test set_dns_validation
         let result = redis_cache.set_dns_validation("example.com", true).await;
         assert!(result.is_ok());
@@ -402,7 +402,7 @@ mod tests {
         // Test with valid Redis URL
         let result = RedisCache::new("redis://127.0.0.1:6379", 3600);
         assert!(result.is_ok() || result.is_err()); // Either works or fails gracefully
-        
+
         // Test with invalid Redis URL
         let result = RedisCache::new("invalid://url", 3600);
         assert!(result.is_err());
@@ -416,15 +416,16 @@ mod tests {
         let app = test::init_service(
             App::new()
                 .app_data(web::Data::new(RedisCache::test_dummy()))
-                .configure(configure_routes)
-        ).await;
-        
+                .configure(configure_routes),
+        )
+        .await;
+
         // Test that the routes are configured by making a request
         let req = test::TestRequest::post()
             .uri("/validate-email")
             .set_json(json!({ "email": "test@example.com" }))
             .to_request();
-        
+
         let resp = test::call_service(&app, req).await;
         // Should not be 404 (not found), meaning route is configured
         assert_ne!(resp.status().as_u16(), 404);

src/routes/graphql.rs

@@ -13,32 +13,31 @@ pub fn configure_routes(cfg: &mut web::ServiceConfig) {
 mod tests {
     use super::*;
     use crate::graphql::schema::create_schema;
-    use actix_web::{test, App, web};
+    use actix_web::{App, test, web};
 
     #[actix_web::test]
     async fn test_configure_routes() {
         let schema = create_schema();
-        
+
         let app = test::init_service(
             App::new()
                 .app_data(web::Data::new(schema))
-                .configure(configure_routes)
-        ).await;
+                .configure(configure_routes),
+        )
+        .await;
 
         // Test GraphQL endpoint
         let req = test::TestRequest::post()
             .uri("/graphql")
             .set_json(serde_json::json!({"query": "{ __typename }"}))
             .to_request();
-        
+
         let resp = test::call_service(&app, req).await;
         assert_eq!(resp.status(), 200);
 
         // Test playground endpoint
-        let req = test::TestRequest::get()
-            .uri("/playground")
-            .to_request();
-        
+        let req = test::TestRequest::get().uri("/playground").to_request();
+
         let resp = test::call_service(&app, req).await;
         assert_eq!(resp.status(), 200);
     }

src/routes/health.rs

@@ -104,15 +104,11 @@ mod tests {
     async fn test_configure_routes_function() {
         // Test that configure_routes function exists and can be called
         // We test through the app initialization since ServiceConfig::new is private
-        let app = test::init_service(
-            App::new().configure(configure_routes)
-        ).await;
-        
+        let app = test::init_service(App::new().configure(configure_routes)).await;
+
         // Test that the health route is configured by making a request
-        let req = test::TestRequest::get()
-            .uri("/health")
-            .to_request();
-        
+        let req = test::TestRequest::get().uri("/health").to_request();
+
         let resp = test::call_service(&app, req).await;
         // Should be 200 OK, meaning route is configured correctly
         assert_eq!(resp.status().as_u16(), 200);