From f83395e89761e6371150a06fbedc1e23c6884e36 Mon Sep 17 00:00:00 2001
From: Evgeniy Stratonikov <evgeniy@nspcc.ru>
Date: Mon, 2 Aug 2021 11:19:30 +0300
Subject: [PATCH] cli/test: move test wallet path to constant

Signed-off-by: Evgeniy Stratonikov <evgeniy@nspcc.ru>
---
 cli/executor_test.go |  3 +++
 cli/wallet_test.go   | 21 ++++++++-------------
 2 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/cli/executor_test.go b/cli/executor_test.go
index 835ac77dd..d1e4b1402 100644
--- a/cli/executor_test.go
+++ b/cli/executor_test.go
@@ -34,6 +34,9 @@ const (
 	validatorAddr = "NfgHwwTi3wHAS8aFAN243C5vGbkYDpqLHP"
 	multisigAddr  = "NVTiAjNgagDkTr5HTzDmQP9kPwPHN5BgVq"
 
+	testWalletPath    = "testdata/testwallet.json"
+	testWalletAccount = "Nfyz4KcsgYepRJw1W5C2uKCi6QWKf7v6gG"
+
 	validatorWallet = "testdata/wallet1_solo.json"
 )
 
diff --git a/cli/wallet_test.go b/cli/wallet_test.go
index 098660d42..7c6d80791 100644
--- a/cli/wallet_test.go
+++ b/cli/wallet_test.go
@@ -228,24 +228,19 @@ func TestWalletExport(t *testing.T) {
 func TestClaimGas(t *testing.T) {
 	e := newExecutor(t, true)
 
-	const walletPath = "testdata/testwallet.json"
-	w, err := wallet.NewWalletFromFile(walletPath)
-	require.NoError(t, err)
-	t.Cleanup(w.Close)
-
 	args := []string{
 		"neo-go", "wallet", "nep17", "multitransfer",
 		"--rpc-endpoint", "http://" + e.RPC.Addr,
 		"--wallet", validatorWallet,
 		"--from", validatorAddr,
-		"NEO:" + w.Accounts[0].Address + ":1000",
-		"GAS:" + w.Accounts[0].Address + ":1000", // for tx send
+		"NEO:" + testWalletAccount + ":1000",
+		"GAS:" + testWalletAccount + ":1000", // for tx send
 	}
 	e.In.WriteString("one\r")
 	e.Run(t, args...)
 	e.checkTxPersisted(t)
 
-	h, err := address.StringToUint160(w.Accounts[0].Address)
+	h, err := address.StringToUint160(testWalletAccount)
 	require.NoError(t, err)
 
 	balanceBefore := e.Chain.GetUtilityTokenBalance(h)
@@ -257,8 +252,8 @@ func TestClaimGas(t *testing.T) {
 	e.In.WriteString("testpass\r")
 	e.Run(t, "neo-go", "wallet", "claim",
 		"--rpc-endpoint", "http://"+e.RPC.Addr,
-		"--wallet", walletPath,
-		"--address", w.Accounts[0].Address)
+		"--wallet", testWalletPath,
+		"--address", testWalletAccount)
 	tx, height := e.checkTxPersisted(t)
 	balanceBefore.Sub(balanceBefore, big.NewInt(tx.NetworkFee+tx.SystemFee))
 	balanceBefore.Add(balanceBefore, cl)
@@ -337,13 +332,13 @@ func TestImportDeployed(t *testing.T) {
 func TestWalletDump(t *testing.T) {
 	e := newExecutor(t, false)
 
-	cmd := []string{"neo-go", "wallet", "dump", "--wallet", "testdata/testwallet.json"}
+	cmd := []string{"neo-go", "wallet", "dump", "--wallet", testWalletPath}
 	e.Run(t, cmd...)
 	rawStr := strings.TrimSpace(e.Out.String())
 	w := new(wallet.Wallet)
 	require.NoError(t, json.Unmarshal([]byte(rawStr), w))
 	require.Equal(t, 1, len(w.Accounts))
-	require.Equal(t, "Nfyz4KcsgYepRJw1W5C2uKCi6QWKf7v6gG", w.Accounts[0].Address)
+	require.Equal(t, testWalletAccount, w.Accounts[0].Address)
 
 	t.Run("with decrypt", func(t *testing.T) {
 		cmd = append(cmd, "--decrypt")
@@ -358,7 +353,7 @@ func TestWalletDump(t *testing.T) {
 		w := new(wallet.Wallet)
 		require.NoError(t, json.Unmarshal([]byte(rawStr), w))
 		require.Equal(t, 1, len(w.Accounts))
-		require.Equal(t, "Nfyz4KcsgYepRJw1W5C2uKCi6QWKf7v6gG", w.Accounts[0].Address)
+		require.Equal(t, testWalletAccount, w.Accounts[0].Address)
 	})
 }