feat: identity: register_user command

src/identity/application/services/register_user/command.rs

@@ -2,6 +2,7 @@
 //
 // SPDX-License-Identifier: AGPL-3.0-or-later
 
+use super::error::*;
 use derive_getters::Getters;
 use serde::{Deserialize, Serialize};
 
@@ -19,20 +20,20 @@ impl RegisterUserCommand {
         password: String,
         confirm_password: String,
         config: &argon2_creds::Config,
-    ) -> Self {
-        let username = config.username(&username).unwrap();
-        config.email(&email).unwrap();
+    ) -> RegisterUserCommandResult<Self> {
+        let username = config.username(&username)?;
+        config.email(&email)?;
 
         if password != confirm_password {
-            todo!("Passwords Don't match");
+            return Err(RegisterUserCommandError::PasswordsDontMatch);
         }
-        let hashed_password: String = config.password(&password).unwrap();
+        let hashed_password: String = config.password(&password)?;
 
-        Self {
+        Ok(Self {
             username,
             email,
             hashed_password,
-        }
+        })
     }
 }
 
@@ -49,8 +50,22 @@ mod tests {
             "asdfasdfasdfasdf".into(),
             "asdfasdfasdfasdf".into(),
             &config,
+        )
+        .unwrap();
+
+        assert_eq!(
+            RegisterUserCommand::new(
+                "realaravinth".into(),
+                "username".into(),
+                "password".into(),
+                "password".into(),
+                &config,
+            )
+            .err(),
+            Some(RegisterUserCommandError::BadEmail)
         );
-        let result = std::panic::catch_unwind(|| 
+
+        assert!(matches!(
             RegisterUserCommand::new(
                 "username".into(),
                 "username@example.com".into(),
@@ -58,10 +73,11 @@ mod tests {
                 "password".into(),
                 &config,
             )
-        ); // TODO: check for error, and not panic when err handling is implemented
-        assert!(result.is_err()); // Blacklist error
+            .err(),
+            Some(RegisterUserCommandError::BadUsername(_))
+        ));
 
-        let result = std::panic::catch_unwind(|| 
+        assert_eq!(
             RegisterUserCommand::new(
                 "realaravinth".into(),
                 "realaravinth@example.com".into(),
@@ -69,8 +85,8 @@ mod tests {
                 "mismatch_password".into(),
                 &config,
             )
-        ); // TODO: check for error, and not panic when err handling is implemented
-        assert!(result.is_err());
-        
+            .err(),
+            Some(RegisterUserCommandError::PasswordsDontMatch)
+        );
     }
 }

src/identity/application/services/register_user/error.rs

@@ -1,3 +1,31 @@
 // SPDX-FileCopyrightText: 2024 Aravinth Manivannan <realaravinth@batsense.net>
 //
 // SPDX-License-Identifier: AGPL-3.0-or-later
+
+use argon2_creds::CredsError;
+use derive_more::Display;
+use serde::{Deserialize, Serialize};
+
+pub type RegisterUserCommandResult<V> = Result<V, RegisterUserCommandError>;
+
+#[derive(Debug, Display, Clone, Serialize, Deserialize, PartialEq, Eq, PartialOrd, Ord)]
+pub enum RegisterUserCommandError {
+    BadUsername(String),
+    BadEmail,
+    BadPassowrd(String),
+    PasswordsDontMatch,
+}
+
+impl From<CredsError> for RegisterUserCommandError {
+    fn from(v: CredsError) -> Self {
+        match v {
+            CredsError::ProfainityError => Self::BadUsername(v.to_string()),
+            CredsError::UsernameCaseMappedError => Self::BadUsername(v.to_string()),
+            CredsError::BlacklistError => Self::BadUsername(v.to_string()),
+            CredsError::NotAnEmail => Self::BadEmail,
+            CredsError::PasswordTooShort => Self::BadPassowrd(v.to_string()),
+            CredsError::PasswordTooLong => Self::BadPassowrd(v.to_string()),
+            CredsError::Argon2Error(e) => Self::BadUsername(e.to_string()),
+        }
+    }
+}