diff --git a/mitra-cli/src/cli.rs b/mitra-cli/src/cli.rs
index 7c6c50f..46ff564 100644
--- a/mitra-cli/src/cli.rs
+++ b/mitra-cli/src/cli.rs
@@ -196,7 +196,12 @@ impl SetRole {
         &self,
         db_client: &impl DatabaseClient,
     ) -> Result<(), Error> {
-        let role = Role::from_name(&self.role)?;
+        let role = match self.role.as_str() {
+            "user" => Role::NormalUser,
+            "admin" => Role::Admin,
+            "read_only_user" => Role::ReadOnlyUser,
+            _ => return Err(anyhow!("unknown role")),
+        };
         set_user_role(db_client, &self.id, role).await?;
         println!("role changed");
         Ok(())
diff --git a/src/models/users/types.rs b/src/models/users/types.rs
index 9e26e12..8379450 100644
--- a/src/models/users/types.rs
+++ b/src/models/users/types.rs
@@ -47,16 +47,6 @@ impl Default for Role {
 }
 
 impl Role {
-    pub fn from_name(name: &str) -> Result<Self, ValidationError> {
-        let role = match name {
-            "user" => Self::NormalUser,
-            "admin" => Self::Admin,
-            "read_only_user" => Self::ReadOnlyUser,
-            _ => return Err(ValidationError("unknown role")),
-        };
-        Ok(role)
-    }
-
     pub fn get_permissions(&self) -> Vec<Permission> {
         match self {
             Self::Guest => vec![],