Implement the errors of account RPC methods

Author: joojis

Cargo.lock

@@ -37,6 +37,8 @@ pub enum Error {
     InvalidKeyFile(String),
     /// Account creation failed.
     CreationFailed,
+    /// Account already exists.
+    AlreadyExists,
     /// `ckeys` error
     CKey(CKeyError),
     /// `CCrypto` error
@@ -55,6 +57,7 @@ impl fmt::Display for Error {
             Error::InvalidMessage => "Invalid message".into(),
             Error::InvalidKeyFile(ref reason) => format!("Invalid key file: {}", reason),
             Error::CreationFailed => "Account creation failed".into(),
+            Error::AlreadyExists => "Account already exists".into(),
             Error::CKey(ref err) => err.to_string(),
             Error::CCrypto(ref err) => err.to_string(),
             Error::Custom(ref s) => s.clone(),

keystore/src/error.rs

@@ -64,7 +64,11 @@ impl KeyStore {
 
 impl SimpleSecretStore for KeyStore {
     fn insert_account(&self, secret: Secret, password: &Password) -> Result<Address, Error> {
-        self.store.insert_account(secret, password)
+        let keypair = KeyPair::from_private(secret.into()).map_err(|_| Error::CreationFailed)?;
+        match self.has_account(&keypair.address())? {
+            true => Err(Error::AlreadyExists),
+            false => self.store.insert_account(secret, password),
+        }
     }
 
     fn accounts(&self) -> Result<Vec<Address>, Error> {

keystore/src/keystore.rs

@@ -8,6 +8,7 @@ authors = ["CodeChain Team <[email protected]>"]
 [dependencies]
 codechain-core = { path = "../core" }
 codechain-key = { path = "../key" }
+codechain-keystore = { path = "../keystore" }
 codechain-logger = { path = "../util/logger" }
 codechain-network = { path = "../network" }
 codechain-state = { path = "../state" }

rpc/Cargo.toml

@@ -18,6 +18,7 @@ extern crate codechain_core as ccore;
 #[macro_use]
 extern crate codechain_logger as clogger;
 extern crate codechain_key as ckey;
+extern crate codechain_keystore as ckeystore;
 extern crate codechain_network as cnetwork;
 extern crate codechain_state as cstate;
 extern crate codechain_types as ctypes;

rpc/src/lib.rs

@@ -19,6 +19,7 @@ use std::fmt;
 use ccore::AccountProviderError;
 use ccore::Error as CoreError;
 use ckey::Error as KeyError;
+use ckeystore::Error as KeystoreError;
 use cnetwork::control::Error as NetworkControlError;
 use cstate::StateError;
 use ctypes::parcel::Error as ParcelError;
@@ -43,6 +44,11 @@ mod codes {
     pub const TOO_LOW_FEE: i64 = -32033;
     pub const TOO_CHEAP_TO_REPLACE: i64 = -32034;
     pub const INVALID_NONCE: i64 = -32035;
+    pub const KEYSTORE_ERROR: i64 = -32040;
+    pub const KEY_ERROR: i64 = -32041;
+    pub const ALREADY_EXISTS: i64 = -32042;
+    pub const WRONG_PASSWORD: i64 = -32043;
+    pub const NO_SUCH_ACCOUNT: i64 = -32044;
     pub const UNKNOWN_ERROR: i64 = -32099;
 }
 
@@ -148,10 +154,39 @@ pub fn rlp(error: DecoderError) -> Error {
 }
 
 pub fn account_provider(error: AccountProviderError) -> Error {
-    Error {
-        code: ErrorCode::ServerError(codes::ACCOUNT_PROVIDER_ERROR),
-        message: "AccountProvider error".into(),
-        data: Some(Value::String(format!("{:?}", error))),
+    match error {
+        AccountProviderError::KeystoreError(error) => match error {
+            KeystoreError::InvalidAccount => Error {
+                code: ErrorCode::ServerError(codes::NO_SUCH_ACCOUNT),
+                message: "No Such Account".into(),
+                data: Some(Value::String(format!("{:?}", error))),
+            },
+            KeystoreError::InvalidPassword => Error {
+                code: ErrorCode::ServerError(codes::WRONG_PASSWORD),
+                message: "Wrong Password".into(),
+                data: Some(Value::String(format!("{:?}", error))),
+            },
+            KeystoreError::AlreadyExists => Error {
+                code: ErrorCode::ServerError(codes::ALREADY_EXISTS),
+                message: "Already Exists".into(),
+                data: Some(Value::String(format!("{:?}", error))),
+            },
+            _ => Error {
+                code: ErrorCode::ServerError(codes::KEYSTORE_ERROR),
+                message: "Keystore Error".into(),
+                data: Some(Value::String(format!("{:?}", error))),
+            },
+        },
+        AccountProviderError::KeyError(_) => Error {
+            code: ErrorCode::ServerError(codes::KEY_ERROR),
+            message: "Key Error".into(),
+            data: Some(Value::String(format!("{:?}", error))),
+        },
+        _ => Error {
+            code: ErrorCode::ServerError(codes::ACCOUNT_PROVIDER_ERROR),
+            message: "AccountProvider Error".into(),
+            data: Some(Value::String(format!("{:?}", error))),
+        },
     }
 }