feat(cli): Check the wallet is unlocked before prompting for name (#5395

)

* feat(cli): Check the wallet is unlocked before prompting for name

* feat(cli): Add this.exit(0)

* update message

* update tests

ironfish-cli/src/commands/wallet/create.ts

@@ -23,15 +23,14 @@ export class CreateCommand extends IronfishCommand {
 
   async start(): Promise<void> {
     const { args } = await this.parse(CreateCommand)
-    let name = args.name
+    const client = await this.connectRpc()
+    await checkWalletUnlocked(client)
 
+    let name = args.name
     if (!name) {
       name = await inputPrompt('Enter the name of the account', true)
     }
 
-    const client = await this.connectRpc()
-    await checkWalletUnlocked(client)
-
     this.log(`Creating account ${name}`)
     const result = await client.wallet.createAccount({ name })
 
@@ -44,5 +43,7 @@ export class CreateCommand extends IronfishCommand {
     } else {
       this.log(`Run "ironfish wallet:use ${name}" to set the account as default`)
     }
+
+    this.exit(0)
   }
 }

ironfish/src/rpc/routes/wallet/decrypt.test.ts

@@ -55,7 +55,7 @@ describe('Route wallet/encrypt', () => {
 
     await expect(
       routeTest.client.wallet.decrypt({ passphrase: invalidPassphrase }),
-    ).rejects.toThrow('Request failed (400) error: Failed to decrypt account')
+    ).rejects.toThrow('Request failed (400) error: Failed to decrypt wallet')
 
     status = await routeTest.client.wallet.getAccountsStatus()
     expect(status.content.encrypted).toBe(true)

ironfish/src/rpc/routes/wallet/unlock.test.ts

@@ -42,7 +42,7 @@ describe('Route wallet/unlock', () => {
 
     await expect(
       routeTest.client.wallet.unlock({ passphrase: invalidPassphrase }),
-    ).rejects.toThrow('Request failed (400) error: Failed to decrypt account')
+    ).rejects.toThrow('Request failed (400) error: Failed to decrypt wallet')
 
     status = await routeTest.client.wallet.getAccountsStatus()
     expect(status.content.encrypted).toBe(true)

ironfish/src/wallet/errors.ts

@@ -84,6 +84,6 @@ export class AccountDecryptionFailedError extends Error {
 
   constructor() {
     super()
-    this.message = 'Failed to decrypt account'
+    this.message = 'Failed to decrypt wallet'
   }
 }