fix: update username validation to allow maximum 255 characters

crates/backend/src/controller/user.rs

@@ -17,11 +17,10 @@ pub struct CreateUser {
     /// Username (minimum 4 characters, maximum 255 characters)
     /// TODO: Don't allow spaces, only alphanumeric characters and underscores
     username: String,
-    #[validate(length(min = 3))]
+    /// Full name of the user
-    /// Full name of the user (minimum 3 characters)
     name: String,
-    #[validate(length(min = 8, max = 255))]
+    #[validate(length(min = 8))]
-    /// Password (minimum 8 characters, maximum 255 characters)
+    /// Password (minimum 8 characters)
     password: String,
 }
 
@@ -136,74 +135,3 @@ async fn delete_user(
     db.delete_user(id).await?;
     Ok(web::Json(format!("User {} deleted", id)))
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[actix_web::test]
-    async fn test_validation_create_user_struct_valid() {
-        let user = CreateUser {
-            username: "testuser".to_string(),
-            name: "Test User".to_string(),
-            password: "password123".to_string(),
-        };
-        let validation_result = user.validate();
-        assert!(validation_result.is_ok());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_user_struct_username_invalid() {
-        let user = CreateUser {
-            username: "usr".to_string(), // too short
-            name: "Test User".to_string(),
-            password: "password".to_string(),
-        };
-        let validation_result = user.validate();
-        assert!(validation_result.is_err());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_user_struct_username_too_long() {
-        let user = CreateUser {
-            username: "a".repeat(256), // too long
-            name: "Test User".to_string(),
-            password: "password123".to_string(),
-        };
-        let validation_result = user.validate();
-        assert!(validation_result.is_err());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_user_struct_name_invalid() {
-        let user = CreateUser {
-            username: "testuser".to_string(),
-            name: "".to_string(), // empty name
-            password: "password123".to_string(),
-        };
-        let validation_result = user.validate();
-        assert!(validation_result.is_err());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_user_struct_password_invalid() {
-        let user = CreateUser {
-            username: "testuser".to_string(),
-            name: "Test User".to_string(),
-            password: "pass".to_string(), // too short
-        };
-        let validation_result = user.validate();
-        assert!(validation_result.is_err());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_user_struct_password_too_long() {
-        let user = CreateUser {
-            username: "testuser".to_string(),
-            name: "Test User".to_string(),
-            password: "a".repeat(256), // too long
-        };
-        let validation_result = user.validate();
-        assert!(validation_result.is_err());
-    }
-}

crates/backend/src/db/project.rs

@@ -12,8 +12,8 @@ use validator::Validate;
 
 #[derive(Deserialize, Validate, ToSchema)]
 pub struct CreateProject {
-    #[validate(length(min = 3, max = 255))]
+    #[validate(length(min = 3))]
-    /// Project name (minimum 3 characters and maximum 255 characters)
+    /// Project name (minimum 3 characters)
     pub name: String,
 }
 
@@ -84,53 +84,3 @@ impl Database {
         Ok(project)
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[actix_web::test]
-    async fn test_validation_create_project_struct_valid() {
-        let project = CreateProject {
-            name: "Test Project".to_string(),
-        };
-        let validation_result = project.validate();
-        assert!(validation_result.is_ok());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_project_struct_invalid_too_short() {
-        let project = CreateProject {
-            name: "TP".to_string(), // too short
-        };
-        let validation_result = project.validate();
-        assert!(validation_result.is_err());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_project_struct_empty() {
-        let project = CreateProject {
-            name: "".to_string(), // empty string
-        };
-        let validation_result = project.validate();
-        assert!(validation_result.is_err());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_project_struct_min_length() {
-        let project = CreateProject {
-            name: "abc".to_string(), // exactly at min length
-        };
-        let validation_result = project.validate();
-        assert!(validation_result.is_ok());
-    }
-
-    #[actix_web::test]
-    async fn test_validation_create_project_struct_long_name() {
-        // 256 characters long should be invalid because of max length
-        let long_name = "a".repeat(256);
-        let project = CreateProject { name: long_name };
-        let validation_result = project.validate();
-        assert!(validation_result.is_err());
-    }
-}