mirror of
https://github.com/nspcc-dev/neo-go.git
synced 2024-11-22 19:29:39 +00:00
685d095fa4
`wallet1_solo.json` can't be used in this test because it contains committee member that receives reward for accepted blocks. The race in this test happens due to the fact that it uses running blockchain and checks expected GAS balance via call to blockchain's GetUtilityTokenBalance API. This call is racy with the CLI command executed prior to the call which causes the following test failure: ``` 2023-04-06T08:44:08.6038406Z === NAME TestNEP17Balance/all_accounts 2023-04-06T08:44:08.6038738Z executor.go:240: 2023-04-06T08:44:08.6039454Z Error Trace: /opt/github-runner/_work/neo-go/neo-go/cli/nep_test/executor.go:240 2023-04-06T08:44:08.6040628Z /opt/github-runner/_work/neo-go/neo-go/cli/nep_test/executor.go:236 2023-04-06T08:44:08.6041787Z /opt/github-runner/_work/neo-go/neo-go/cli/nep_test/nep17_test.go:73 2023-04-06T08:44:08.6042389Z Error: Expect " Amount : 1" to match "^\s*Amount\s*:\s*1.5$" 2023-04-06T08:44:08.6042865Z Test: TestNEP17Balance/all_accounts ``` The fix is do not use committee accounts for balance checks. Let's use side accounts and transfer funds to some of them before the test start. Close #2960. Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>
398 lines
14 KiB
Go
398 lines
14 KiB
Go
package nep_test
|
|
|
|
import (
|
|
"io"
|
|
"math/big"
|
|
"path/filepath"
|
|
"strconv"
|
|
"strings"
|
|
"testing"
|
|
|
|
"github.com/nspcc-dev/neo-go/internal/testcli"
|
|
"github.com/nspcc-dev/neo-go/pkg/core/native/nativenames"
|
|
"github.com/nspcc-dev/neo-go/pkg/encoding/address"
|
|
"github.com/nspcc-dev/neo-go/pkg/encoding/fixedn"
|
|
"github.com/nspcc-dev/neo-go/pkg/smartcontract/manifest"
|
|
"github.com/nspcc-dev/neo-go/pkg/wallet"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestNEP17Balance(t *testing.T) {
|
|
e := testcli.NewExecutor(t, true)
|
|
|
|
args := []string{
|
|
"neo-go", "wallet", "nep17", "multitransfer",
|
|
"--rpc-endpoint", "http://" + e.RPC.Addresses()[0],
|
|
"--wallet", testcli.ValidatorWallet,
|
|
"--from", testcli.ValidatorAddr,
|
|
"GAS:" + testcli.TestWalletMultiAccount1 + ":1",
|
|
"NEO:" + testcli.TestWalletMultiAccount1 + ":10",
|
|
"GAS:" + testcli.TestWalletMultiAccount3 + ":3",
|
|
"--force",
|
|
}
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, args...)
|
|
e.CheckTxPersisted(t)
|
|
|
|
cmdbalance := []string{"neo-go", "wallet", "nep17", "balance"}
|
|
cmdbase := append(cmdbalance,
|
|
"--rpc-endpoint", "http://"+e.RPC.Addresses()[0],
|
|
"--wallet", testcli.TestWalletMultiPath,
|
|
)
|
|
cmd := append(cmdbase, "--address", testcli.TestWalletMultiAccount1)
|
|
t.Run("excessive parameters", func(t *testing.T) {
|
|
e.RunWithError(t, append(cmd, "--token", "NEO", "gas")...)
|
|
})
|
|
t.Run("NEO", func(t *testing.T) {
|
|
b, index := e.Chain.GetGoverningTokenBalance(testcli.TestWalletMultiAccount1Hash)
|
|
checkResult := func(t *testing.T) {
|
|
e.CheckNextLine(t, "^\\s*Account\\s+"+testcli.TestWalletMultiAccount1)
|
|
e.CheckNextLine(t, "^\\s*NEO:\\s+NeoToken \\("+e.Chain.GoverningTokenHash().StringLE()+"\\)")
|
|
e.CheckNextLine(t, "^\\s*Amount\\s*:\\s*"+b.String()+"$")
|
|
e.CheckNextLine(t, "^\\s*Updated\\s*:\\s*"+strconv.FormatUint(uint64(index), 10))
|
|
e.CheckEOF(t)
|
|
}
|
|
t.Run("Alias", func(t *testing.T) {
|
|
e.Run(t, append(cmd, "--token", "NEO")...)
|
|
checkResult(t)
|
|
})
|
|
t.Run("Hash", func(t *testing.T) {
|
|
e.Run(t, append(cmd, "--token", e.Chain.GoverningTokenHash().StringLE())...)
|
|
checkResult(t)
|
|
})
|
|
})
|
|
t.Run("GAS", func(t *testing.T) {
|
|
e.Run(t, append(cmd, "--token", "GAS")...)
|
|
e.CheckNextLine(t, "^\\s*Account\\s+"+testcli.TestWalletMultiAccount1)
|
|
e.CheckNextLine(t, "^\\s*GAS:\\s+GasToken \\("+e.Chain.UtilityTokenHash().StringLE()+"\\)")
|
|
b := e.Chain.GetUtilityTokenBalance(testcli.TestWalletMultiAccount1Hash)
|
|
e.CheckNextLine(t, "^\\s*Amount\\s*:\\s*"+fixedn.Fixed8(b.Int64()).String()+"$")
|
|
})
|
|
t.Run("zero balance of known token", func(t *testing.T) {
|
|
e.Run(t, append(cmdbase, []string{"--token", "NEO", "--address", testcli.TestWalletMultiAccount2}...)...)
|
|
e.CheckNextLine(t, "^Account "+testcli.TestWalletMultiAccount2)
|
|
e.CheckNextLine(t, "^\\s*NEO:\\s+NeoToken \\("+e.Chain.GoverningTokenHash().StringLE()+"\\)")
|
|
e.CheckNextLine(t, "^\\s*Amount\\s*:\\s*"+fixedn.Fixed8(0).String()+"$")
|
|
e.CheckNextLine(t, "^\\s*Updated:")
|
|
e.CheckEOF(t)
|
|
})
|
|
t.Run("all accounts", func(t *testing.T) {
|
|
e.Run(t, cmdbase...)
|
|
|
|
e.CheckNextLine(t, "^Account "+testcli.TestWalletMultiAccount1)
|
|
// The order of assets is undefined.
|
|
for i := 0; i < 2; i++ {
|
|
line := e.GetNextLine(t)
|
|
if strings.Contains(line, "GAS") {
|
|
e.CheckLine(t, line, "^\\s*GAS:\\s+GasToken \\("+e.Chain.UtilityTokenHash().StringLE()+"\\)")
|
|
balance := e.Chain.GetUtilityTokenBalance(testcli.TestWalletMultiAccount1Hash)
|
|
e.CheckNextLine(t, "^\\s*Amount\\s*:\\s*"+fixedn.Fixed8(balance.Int64()).String()+"$")
|
|
e.CheckNextLine(t, "^\\s*Updated:")
|
|
} else {
|
|
balance, index := e.Chain.GetGoverningTokenBalance(testcli.TestWalletMultiAccount1Hash)
|
|
e.CheckLine(t, line, "^\\s*NEO:\\s+NeoToken \\("+e.Chain.GoverningTokenHash().StringLE()+"\\)")
|
|
e.CheckNextLine(t, "^\\s*Amount\\s*:\\s*"+balance.String()+"$")
|
|
e.CheckNextLine(t, "^\\s*Updated\\s*:\\s*"+strconv.FormatUint(uint64(index), 10))
|
|
}
|
|
}
|
|
e.CheckNextLine(t, "^\\s*$")
|
|
|
|
e.CheckNextLine(t, "^Account "+testcli.TestWalletMultiAccount2)
|
|
e.CheckNextLine(t, "^\\s*$")
|
|
|
|
e.CheckNextLine(t, "^Account "+testcli.TestWalletMultiAccount3)
|
|
e.CheckNextLine(t, "^\\s*GAS:\\s+GasToken \\("+e.Chain.UtilityTokenHash().StringLE()+"\\)")
|
|
balance := e.Chain.GetUtilityTokenBalance(testcli.TestWalletMultiAccount3Hash)
|
|
e.CheckNextLine(t, "^\\s*Amount\\s*:\\s*"+fixedn.Fixed8(balance.Int64()).String()+"$")
|
|
e.CheckNextLine(t, "^\\s*Updated:")
|
|
e.CheckEOF(t)
|
|
})
|
|
t.Run("Bad token", func(t *testing.T) {
|
|
e.Run(t, append(cmd, "--token", "kek")...)
|
|
e.CheckNextLine(t, "^\\s*Account\\s+"+testcli.TestWalletMultiAccount1)
|
|
e.CheckNextLine(t, `^\s*Can't find data for "kek" token\s*`)
|
|
e.CheckEOF(t)
|
|
})
|
|
t.Run("Bad wallet", func(t *testing.T) {
|
|
e.RunWithError(t, append(cmdbalance, "--wallet", "/dev/null")...)
|
|
})
|
|
}
|
|
|
|
func TestNEP17Transfer(t *testing.T) {
|
|
w, err := wallet.NewWalletFromFile("../testdata/testwallet.json")
|
|
require.NoError(t, err)
|
|
|
|
e := testcli.NewExecutor(t, true)
|
|
args := []string{
|
|
"neo-go", "wallet", "nep17", "transfer",
|
|
"--rpc-endpoint", "http://" + e.RPC.Addresses()[0],
|
|
"--wallet", testcli.ValidatorWallet,
|
|
"--to", w.Accounts[0].Address,
|
|
"--token", "NEO",
|
|
"--amount", "1",
|
|
"--from", testcli.ValidatorAddr,
|
|
}
|
|
|
|
t.Run("missing receiver", func(t *testing.T) {
|
|
as := append([]string{}, args[:8]...)
|
|
as = append(as, args[10:]...)
|
|
e.In.WriteString("one\r")
|
|
e.RunWithError(t, as...)
|
|
e.In.Reset()
|
|
})
|
|
|
|
t.Run("InvalidPassword", func(t *testing.T) {
|
|
e.In.WriteString("onetwothree\r")
|
|
e.RunWithError(t, args...)
|
|
e.In.Reset()
|
|
})
|
|
|
|
t.Run("no confirmation", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.RunWithError(t, args...)
|
|
e.In.Reset()
|
|
})
|
|
t.Run("cancel after prompt", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.RunWithError(t, args...)
|
|
e.In.Reset()
|
|
})
|
|
|
|
e.In.WriteString("one\r")
|
|
e.In.WriteString("Y\r")
|
|
e.Run(t, args...)
|
|
e.CheckNextLine(t, `^Network fee:\s*(\d|\.)+`)
|
|
e.CheckNextLine(t, `^System fee:\s*(\d|\.)+`)
|
|
e.CheckNextLine(t, `^Total fee:\s*(\d|\.)+`)
|
|
e.CheckTxPersisted(t)
|
|
|
|
sh := w.Accounts[0].ScriptHash()
|
|
b, _ := e.Chain.GetGoverningTokenBalance(sh)
|
|
require.Equal(t, big.NewInt(1), b)
|
|
|
|
t.Run("with force", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, append(args, "--force")...)
|
|
e.CheckTxPersisted(t)
|
|
|
|
b, _ := e.Chain.GetGoverningTokenBalance(sh)
|
|
require.Equal(t, big.NewInt(2), b)
|
|
})
|
|
|
|
hVerify := deployVerifyContract(t, e)
|
|
const validatorDefault = "Nhfg3TbpwogLvDGVvAvqyThbsHgoSUKwtn"
|
|
|
|
t.Run("default address", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, "neo-go", "wallet", "nep17", "multitransfer",
|
|
"--rpc-endpoint", "http://"+e.RPC.Addresses()[0],
|
|
"--wallet", testcli.ValidatorWallet,
|
|
"--from", testcli.ValidatorAddr,
|
|
"--force",
|
|
"NEO:"+validatorDefault+":42",
|
|
"GAS:"+validatorDefault+":7")
|
|
e.CheckTxPersisted(t)
|
|
|
|
args := args[:len(args)-2] // cut '--from' argument
|
|
args = append(args, "--force")
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, args...)
|
|
e.CheckTxPersisted(t)
|
|
|
|
b, _ := e.Chain.GetGoverningTokenBalance(sh)
|
|
require.Equal(t, big.NewInt(3), b)
|
|
|
|
sh, err = address.StringToUint160(validatorDefault)
|
|
require.NoError(t, err)
|
|
b, _ = e.Chain.GetGoverningTokenBalance(sh)
|
|
require.Equal(t, big.NewInt(41), b)
|
|
})
|
|
|
|
t.Run("with signers", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, "neo-go", "wallet", "nep17", "multitransfer",
|
|
"--rpc-endpoint", "http://"+e.RPC.Addresses()[0],
|
|
"--wallet", testcli.ValidatorWallet,
|
|
"--from", testcli.ValidatorAddr,
|
|
"--force",
|
|
"NEO:"+validatorDefault+":42",
|
|
"GAS:"+validatorDefault+":7",
|
|
"--", testcli.ValidatorAddr+":Global")
|
|
e.CheckTxPersisted(t)
|
|
})
|
|
|
|
validTil := e.Chain.BlockHeight() + 100
|
|
cmd := []string{
|
|
"neo-go", "wallet", "nep17", "transfer",
|
|
"--rpc-endpoint", "http://" + e.RPC.Addresses()[0],
|
|
"--wallet", testcli.ValidatorWallet,
|
|
"--token", "GAS",
|
|
"--amount", "1",
|
|
"--force",
|
|
"--from", testcli.ValidatorAddr}
|
|
|
|
t.Run("with await", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, append(cmd, "--to", nftOwnerAddr, "--await")...)
|
|
e.CheckAwaitableTxPersisted(t)
|
|
})
|
|
|
|
cmd = append(cmd, "--to", address.Uint160ToString(e.Chain.GetNotaryContractScriptHash()),
|
|
"[", testcli.ValidatorAddr, strconv.Itoa(int(validTil)), "]")
|
|
|
|
t.Run("with data", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, cmd...)
|
|
e.CheckTxPersisted(t)
|
|
})
|
|
|
|
t.Run("with data and signers", func(t *testing.T) {
|
|
t.Run("invalid sender's scope", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.RunWithError(t, append(cmd, "--", testcli.ValidatorAddr+":None")...)
|
|
})
|
|
t.Run("good", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, append(cmd, "--", testcli.ValidatorAddr+":Global")...) // CalledByEntry is enough, but it's the default value, so check something else
|
|
e.CheckTxPersisted(t)
|
|
})
|
|
t.Run("several signers", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, append(cmd, "--", testcli.ValidatorAddr, hVerify.StringLE())...)
|
|
e.CheckTxPersisted(t)
|
|
})
|
|
})
|
|
}
|
|
|
|
func TestNEP17MultiTransfer(t *testing.T) {
|
|
privs, _ := testcli.GenerateKeys(t, 3)
|
|
|
|
e := testcli.NewExecutor(t, true)
|
|
neoContractHash, err := e.Chain.GetNativeContractScriptHash(nativenames.Neo)
|
|
require.NoError(t, err)
|
|
args := []string{
|
|
"neo-go", "wallet", "nep17", "multitransfer",
|
|
"--rpc-endpoint", "http://" + e.RPC.Addresses()[0],
|
|
"--wallet", testcli.ValidatorWallet,
|
|
"--from", testcli.ValidatorAddr,
|
|
"--force",
|
|
"NEO:" + privs[0].Address() + ":42",
|
|
"GAS:" + privs[1].Address() + ":7",
|
|
neoContractHash.StringLE() + ":" + privs[2].Address() + ":13",
|
|
}
|
|
hVerify := deployVerifyContract(t, e)
|
|
|
|
t.Run("no cosigners", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, args...)
|
|
e.CheckTxPersisted(t)
|
|
|
|
b, _ := e.Chain.GetGoverningTokenBalance(privs[0].GetScriptHash())
|
|
require.Equal(t, big.NewInt(42), b)
|
|
b = e.Chain.GetUtilityTokenBalance(privs[1].GetScriptHash())
|
|
require.Equal(t, big.NewInt(int64(fixedn.Fixed8FromInt64(7))), b)
|
|
b, _ = e.Chain.GetGoverningTokenBalance(privs[2].GetScriptHash())
|
|
require.Equal(t, big.NewInt(13), b)
|
|
})
|
|
|
|
t.Run("invalid sender scope", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.RunWithError(t, append(args,
|
|
"--", testcli.ValidatorAddr+":None")...) // invalid sender scope
|
|
})
|
|
t.Run("Global sender scope", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, append(args,
|
|
"--", testcli.ValidatorAddr+":Global")...)
|
|
e.CheckTxPersisted(t)
|
|
})
|
|
t.Run("Several cosigners", func(t *testing.T) {
|
|
e.In.WriteString("one\r")
|
|
e.Run(t, append(args,
|
|
"--", testcli.ValidatorAddr, hVerify.StringLE())...)
|
|
e.CheckTxPersisted(t)
|
|
})
|
|
}
|
|
|
|
func TestNEP17ImportToken(t *testing.T) {
|
|
e := testcli.NewExecutor(t, true)
|
|
tmpDir := t.TempDir()
|
|
walletPath := filepath.Join(tmpDir, "walletForImport.json")
|
|
|
|
neoContractHash, err := e.Chain.GetNativeContractScriptHash(nativenames.Neo)
|
|
require.NoError(t, err)
|
|
gasContractHash, err := e.Chain.GetNativeContractScriptHash(nativenames.Gas)
|
|
require.NoError(t, err)
|
|
nnsContractHash := deployNNSContract(t, e)
|
|
e.Run(t, "neo-go", "wallet", "init", "--wallet", walletPath)
|
|
|
|
// missing token hash
|
|
e.RunWithError(t, "neo-go", "wallet", "nep17", "import",
|
|
"--rpc-endpoint", "http://"+e.RPC.Addresses()[0],
|
|
"--wallet", walletPath)
|
|
|
|
// additional parameter
|
|
e.RunWithError(t, "neo-go", "wallet", "nep17", "import",
|
|
"--rpc-endpoint", "http://"+e.RPC.Addresses()[0],
|
|
"--wallet", walletPath,
|
|
"--token", gasContractHash.StringLE(), "useless")
|
|
e.Run(t, "neo-go", "wallet", "nep17", "import",
|
|
"--rpc-endpoint", "http://"+e.RPC.Addresses()[0],
|
|
"--wallet", walletPath,
|
|
"--token", gasContractHash.StringLE())
|
|
e.Run(t, "neo-go", "wallet", "nep17", "import",
|
|
"--rpc-endpoint", "http://"+e.RPC.Addresses()[0],
|
|
"--wallet", walletPath,
|
|
"--token", address.Uint160ToString(neoContractHash)) // try address instead of sh
|
|
|
|
// not a NEP-17 token
|
|
e.RunWithError(t, "neo-go", "wallet", "nep17", "import",
|
|
"--rpc-endpoint", "http://"+e.RPC.Addresses()[0],
|
|
"--wallet", walletPath,
|
|
"--token", nnsContractHash.StringLE())
|
|
|
|
t.Run("Info", func(t *testing.T) {
|
|
checkGASInfo := func(t *testing.T) {
|
|
e.CheckNextLine(t, "^Name:\\s*GasToken")
|
|
e.CheckNextLine(t, "^Symbol:\\s*GAS")
|
|
e.CheckNextLine(t, "^Hash:\\s*"+gasContractHash.StringLE())
|
|
e.CheckNextLine(t, "^Decimals:\\s*8")
|
|
e.CheckNextLine(t, "^Address:\\s*"+address.Uint160ToString(gasContractHash))
|
|
e.CheckNextLine(t, "^Standard:\\s*"+string(manifest.NEP17StandardName))
|
|
}
|
|
t.Run("excessive parameters", func(t *testing.T) {
|
|
e.RunWithError(t, "neo-go", "wallet", "nep17", "info",
|
|
"--wallet", walletPath, "--token", gasContractHash.StringLE(), "parameter")
|
|
})
|
|
t.Run("WithToken", func(t *testing.T) {
|
|
e.Run(t, "neo-go", "wallet", "nep17", "info",
|
|
"--wallet", walletPath, "--token", gasContractHash.StringLE())
|
|
checkGASInfo(t)
|
|
})
|
|
t.Run("NoToken", func(t *testing.T) {
|
|
e.Run(t, "neo-go", "wallet", "nep17", "info",
|
|
"--wallet", walletPath)
|
|
checkGASInfo(t)
|
|
_, err := e.Out.ReadString('\n')
|
|
require.NoError(t, err)
|
|
e.CheckNextLine(t, "^Name:\\s*NeoToken")
|
|
e.CheckNextLine(t, "^Symbol:\\s*NEO")
|
|
e.CheckNextLine(t, "^Hash:\\s*"+neoContractHash.StringLE())
|
|
e.CheckNextLine(t, "^Decimals:\\s*0")
|
|
e.CheckNextLine(t, "^Address:\\s*"+address.Uint160ToString(neoContractHash))
|
|
e.CheckNextLine(t, "^Standard:\\s*"+string(manifest.NEP17StandardName))
|
|
})
|
|
t.Run("Remove", func(t *testing.T) {
|
|
e.RunWithError(t, "neo-go", "wallet", "nep17", "remove",
|
|
"--wallet", walletPath, "--token", neoContractHash.StringLE(), "add")
|
|
e.In.WriteString("y\r")
|
|
e.Run(t, "neo-go", "wallet", "nep17", "remove",
|
|
"--wallet", walletPath, "--token", neoContractHash.StringLE())
|
|
e.Run(t, "neo-go", "wallet", "nep17", "info",
|
|
"--wallet", walletPath)
|
|
checkGASInfo(t)
|
|
_, err := e.Out.ReadString('\n')
|
|
require.Equal(t, err, io.EOF)
|
|
})
|
|
})
|
|
}
|