[account] let LoadOrCreateAccount() return new account created flag

Makefile

@@ -109,7 +109,6 @@ build-readtip:
 .PHONY: fmt
 fmt:
 	$(GOCMD) fmt ./...
-	$(GOCMD) mod tidy
 
 .PHONY: lint
 lint:

action/protocol/account/transfer.go

@@ -34,7 +34,7 @@ func (p *Protocol) handleTransfer(ctx context.Context, tsf *action.Transfer, sm
 		accountCreationOpts = append(accountCreationOpts, state.LegacyNonceAccountTypeOption())
 	}
 	// check sender
-	sender, err := accountutil.LoadOrCreateAccount(sm, actionCtx.Caller, accountCreationOpts...)
+	sender, _, err := accountutil.LoadOrCreateAccount(sm, actionCtx.Caller, accountCreationOpts...)
 	if err != nil {
 		return nil, errors.Wrapf(err, "failed to load or create the account of sender %s", actionCtx.Caller.String())
 	}
@@ -119,7 +119,7 @@ func (p *Protocol) handleTransfer(ctx context.Context, tsf *action.Transfer, sm
 	}
 
 	// check recipient
-	recipient, err := accountutil.LoadOrCreateAccount(sm, recipientAddr, accountCreationOpts...)
+	recipient, _, err := accountutil.LoadOrCreateAccount(sm, recipientAddr, accountCreationOpts...)
 	if err != nil {
 		return nil, errors.Wrapf(err, "failed to load or create the account of recipient %s", tsf.Recipient())
 	}

action/protocol/account/util/util.go

@@ -18,26 +18,26 @@ import (
 )
 
 // LoadOrCreateAccount either loads an account state or creates an account state
-func LoadOrCreateAccount(sm protocol.StateManager, addr address.Address, opts ...state.AccountCreationOption) (*state.Account, error) {
+func LoadOrCreateAccount(sm protocol.StateManager, addr address.Address, opts ...state.AccountCreationOption) (*state.Account, bool, error) {
 	var (
 		account  = &state.Account{}
 		addrHash = hash.BytesToHash160(addr.Bytes())
 	)
 	_, err := sm.State(account, protocol.LegacyKeyOption(addrHash))
 	switch errors.Cause(err) {
 	case nil:
-		return account, nil
+		return account, false, nil
 	case state.ErrStateNotExist:
 		account, err := state.NewAccount(opts...)
 		if err != nil {
-			return nil, errors.Wrapf(err, "failed to create state account for %x", addrHash)
+			return nil, false, errors.Wrapf(err, "failed to create state account for %x", addrHash)
 		}
 		if _, err := sm.PutState(account, protocol.LegacyKeyOption(addrHash)); err != nil {
-			return nil, errors.Wrapf(err, "failed to put state for account %x", addrHash)
+			return nil, false, errors.Wrapf(err, "failed to put state for account %x", addrHash)
 		}
-		return account, nil
+		return account, true, nil
 	default:
-		return nil, err
+		return nil, false, err
 	}
 }
 

action/protocol/execution/evm/contract_test.go

@@ -61,8 +61,9 @@ func TestCreateContract(t *testing.T) {
 		}).AnyTimes()
 
 	addr := identityset.Address(28)
-	_, err = accountutil.LoadOrCreateAccount(sm, addr)
+	_, created, err := accountutil.LoadOrCreateAccount(sm, addr)
 	require.NoError(err)
+	require.True(created)
 	stateDB, err := NewStateDBAdapter(sm, 0, hash.ZeroHash256, NotFixTopicCopyBugOption())
 	require.NoError(err)
 
@@ -86,8 +87,9 @@ func TestCreateContract(t *testing.T) {
 	require.NoError(stateDB.CommitContracts())
 	stateDB.clear()
 	// reload same contract
-	contract1, err := accountutil.LoadOrCreateAccount(sm, addr)
+	contract1, created, err := accountutil.LoadOrCreateAccount(sm, addr)
 	require.NoError(err)
+	require.False(created)
 	require.Equal(codeHash[:], contract1.CodeHash)
 }
 

action/protocol/execution/evm/evm_test.go

@@ -7,6 +7,7 @@ package evm
 
 import (
 	"context"
+	"encoding/base64"
 	"errors"
 	"math/big"
 	"testing"
@@ -16,6 +17,7 @@ import (
 	"github.com/golang/mock/gomock"
 	"github.com/stretchr/testify/require"
 
+	"github.com/iotexproject/go-pkgs/crypto"
 	"github.com/iotexproject/go-pkgs/hash"
 	"github.com/iotexproject/iotex-address/address"
 	"github.com/iotexproject/iotex-proto/golang/iotextypes"
@@ -467,3 +469,14 @@ func gasExecuteInEVM(gas, consume, refund, size uint64) (uint64, uint64, error)
 	remainingGas -= consume
 	return remainingGas, remainingGas + refund, nil
 }
+
+func TestFk(t *testing.T) {
+	r := require.New(t)
+	sk, err := base64.StdEncoding.DecodeString("MzUyMDAyNGNjNDcyZjUxNGYwOTM2N2FhNTAzMjBiMjRmZmRmODI2MmM2YjYxYTNkZDZiMTQ3OGJmNDE2MWRjYQ==")
+	r.NoError(err)
+	r.Equal(64, len(sk))
+	println("sk =", string(sk))
+	key, err := crypto.HexStringToPrivateKey(string(sk))
+	r.NoError(err)
+	println("addr =", key.PublicKey().Address().String())
+}

action/protocol/execution/evm/evmstatedbadapter.go

@@ -225,7 +225,7 @@ func (stateDB *StateDBAdapter) CreateAccount(evmAddr common.Address) {
 		log.L().Error("Failed to convert evm address.", zap.Error(err))
 		return
 	}
-	_, err = accountutil.LoadOrCreateAccount(stateDB.sm, addr, stateDB.accountCreationOpts()...)
+	_, _, err = accountutil.LoadOrCreateAccount(stateDB.sm, addr, stateDB.accountCreationOpts()...)
 	if err != nil {
 		log.L().Error("Failed to create account.", zap.Error(err))
 		stateDB.logError(err)
@@ -286,7 +286,7 @@ func (stateDB *StateDBAdapter) AddBalance(evmAddr common.Address, a256 *uint256.
 	if contract, ok := stateDB.cachedContract[addrHash]; ok {
 		state = contract.SelfState()
 	} else {
-		state, err = accountutil.LoadOrCreateAccount(stateDB.sm, addr, stateDB.accountCreationOpts()...)
+		state, _, err = accountutil.LoadOrCreateAccount(stateDB.sm, addr, stateDB.accountCreationOpts()...)
 		if err != nil {
 			log.L().Error("Failed to add balance.", log.Hex("addrHash", evmAddr[:]))
 			stateDB.logError(err)

action/protocol/poll/util.go

@@ -178,7 +178,7 @@ func setCandidates(
 		if err != nil {
 			return errors.Wrapf(err, "failed to decode delegate address %s", candidate.Address)
 		}
-		delegate, err := accountutil.LoadOrCreateAccount(sm, addr, accountCreationOpts...)
+		delegate, _, err := accountutil.LoadOrCreateAccount(sm, addr, accountCreationOpts...)
 		if err != nil {
 			return errors.Wrapf(err, "failed to load or create the account for delegate %s", candidate.Address)
 		}

action/protocol/rewarding/protocol.go

@@ -444,7 +444,7 @@ func (p *Protocol) increaseNonce(ctx context.Context, sm protocol.StateManager,
 	if protocol.MustGetFeatureCtx(ctx).CreateLegacyNonceAccount {
 		accountCreationOpts = append(accountCreationOpts, state.LegacyNonceAccountTypeOption())
 	}
-	acc, err := accountutil.LoadOrCreateAccount(sm, addr, accountCreationOpts...)
+	acc, _, err := accountutil.LoadOrCreateAccount(sm, addr, accountCreationOpts...)
 	if err != nil {
 		return err
 	}

action/protocol/rewarding/reward.go

@@ -392,7 +392,7 @@ func (p *Protocol) claimFromAccount(ctx context.Context, sm protocol.StateManage
 		accountCreationOpts = append(accountCreationOpts, state.LegacyNonceAccountTypeOption())
 	}
 	// Update primary account
-	primAcc, err := accountutil.LoadOrCreateAccount(sm, addr, accountCreationOpts...)
+	primAcc, _, err := accountutil.LoadOrCreateAccount(sm, addr, accountCreationOpts...)
 	if err != nil {
 		return err
 	}

action/protocol/staking/handlers_test.go

@@ -3239,7 +3239,7 @@ func setupAccount(sm protocol.StateManager, addr address.Address, balance int64)
 	if balance < 0 {
 		return errors.New("balance cannot be negative")
 	}
-	account, err := accountutil.LoadOrCreateAccount(sm, addr, state.LegacyNonceAccountTypeOption())
+	account, _, err := accountutil.LoadOrCreateAccount(sm, addr, state.LegacyNonceAccountTypeOption())
 	if err != nil {
 		return err
 	}