Clarify account creation error messages in CLI (bp #7719) (#7745)

automerge

cli/src/nonce.rs

@@ -363,12 +363,18 @@ pub fn process_create_nonce_account(
         (&nonce_account_pubkey, "nonce_account_pubkey".to_string()),
     )?;
 
-    if rpc_client.get_account(&nonce_account_pubkey).is_ok() {
+    if let Ok(nonce_account) = rpc_client.get_account(&nonce_account_pubkey) {
-        return Err(CliError::BadParameter(format!(
+        let err_msg = if nonce_account.owner == system_program::id()
-            "Unable to create nonce account. Nonce account already exists: {}",
+            && State::<NonceState>::state(&nonce_account).is_ok()
-            nonce_account_pubkey,
+        {
-        ))
+            format!("Nonce account {} already exists", nonce_account_pubkey)
-        .into());
+        } else {
+            format!(
+                "Account {} already exists and is not a nonce account",
+                nonce_account_pubkey
+            )
+        };
+        return Err(CliError::BadParameter(err_msg).into());
     }
 
     let minimum_balance = rpc_client.get_minimum_balance_for_rent_exemption(NonceState::size())?;

cli/src/stake.rs

@@ -527,12 +527,16 @@ pub fn process_create_stake_account(
         (&stake_account_pubkey, "stake_account_pubkey".to_string()),
     )?;
 
-    if rpc_client.get_account(&stake_account_pubkey).is_ok() {
+    if let Ok(stake_account) = rpc_client.get_account(&stake_account_pubkey) {
-        return Err(CliError::BadParameter(format!(
+        let err_msg = if stake_account.owner == solana_stake_program::id() {
-            "Unable to create stake account. Stake account already exists: {}",
+            format!("Stake account {} already exists", stake_account_pubkey)
+        } else {
+            format!(
+                "Account {} already exists and is not a stake account",
                 stake_account_pubkey
-        ))
+            )
-        .into());
+        };
+        return Err(CliError::BadParameter(err_msg).into());
     }
 
     let minimum_balance =

cli/src/storage.rs

@@ -163,6 +163,19 @@ pub fn process_create_storage_account(
             "storage_account_pubkey".to_string(),
         ),
     )?;
+
+    if let Ok(storage_account) = rpc_client.get_account(&storage_account_pubkey) {
+        let err_msg = if storage_account.owner == solana_storage_program::id() {
+            format!("Storage account {} already exists", storage_account_pubkey)
+        } else {
+            format!(
+                "Account {} already exists and is not a storage account",
+                storage_account_pubkey
+            )
+        };
+        return Err(CliError::BadParameter(err_msg).into());
+    }
+
     use solana_storage_program::storage_contract::STORAGE_ACCOUNT_SPACE;
     let required_balance = rpc_client
         .get_minimum_balance_for_rent_exemption(STORAGE_ACCOUNT_SPACE as usize)?

cli/src/vote.rs

@@ -9,10 +9,9 @@ use crate::{
 use clap::{value_t_or_exit, App, Arg, ArgMatches, SubCommand};
 use solana_clap_utils::{input_parsers::*, input_validators::*};
 use solana_client::rpc_client::RpcClient;
-use solana_sdk::signature::Keypair;
 use solana_sdk::{
-    account::Account, pubkey::Pubkey, signature::KeypairUtil, system_instruction::SystemError,
+    account::Account, pubkey::Pubkey, signature::Keypair, signature::KeypairUtil,
-    transaction::Transaction,
+    system_instruction::SystemError, transaction::Transaction,
 };
 use solana_vote_program::{
     vote_instruction::{self, VoteError},
@@ -295,6 +294,18 @@ pub fn process_create_vote_account(
         (&vote_account_pubkey, "vote_account_pubkey".to_string()),
     )?;
 
+    if let Ok(vote_account) = rpc_client.get_account(&vote_account_pubkey) {
+        let err_msg = if vote_account.owner == solana_vote_program::id() {
+            format!("Vote account {} already exists", vote_account_pubkey)
+        } else {
+            format!(
+                "Account {} already exists and is not a vote account",
+                vote_account_pubkey
+            )
+        };
+        return Err(CliError::BadParameter(err_msg).into());
+    }
+
     let required_balance = rpc_client
         .get_minimum_balance_for_rent_exemption(VoteState::size_of())?
         .max(1);