Make stake token address as mandatory in certain use cases (#1914)

* Stake token address is mandatory if not run in test mode

* Mandatory stake token in rootchain fund command (in case those are minted)

* Add unit tests

command/rootchain/fund/params.go

@@ -2,9 +2,11 @@ package fund
 
 import (
 	"errors"
+	"fmt"
 	"math/big"
 
 	cmdhelper "github.com/0xPolygon/polygon-edge/command/helper"
+	rootHelper "github.com/0xPolygon/polygon-edge/command/rootchain/helper"
 	"github.com/0xPolygon/polygon-edge/types"
 )
 
@@ -56,5 +58,15 @@ func (fp *fundParams) validateFlags() error {
 		fp.amountValues[i] = amountValue
 	}
 
+	if fp.mintStakeToken {
+		if fp.stakeTokenAddr == "" {
+			return rootHelper.ErrMandatoryStakeToken
+		}
+
+		if err := types.IsValidAddress(fp.stakeTokenAddr); err != nil {
+			return fmt.Errorf("invalid stake token address is provided: %w", err)
+		}
+	}
+
 	return nil
 }

command/rootchain/fund/params_test.go

@@ -0,0 +1,105 @@
+package fund
+
+import (
+	"fmt"
+	"testing"
+
+	"github.com/stretchr/testify/require"
+
+	rootHelper "github.com/0xPolygon/polygon-edge/command/rootchain/helper"
+	"github.com/0xPolygon/polygon-edge/types"
+)
+
+func Test_validateFlags(t *testing.T) {
+	t.Parallel()
+
+	cases := []struct {
+		buildParamsFn func() *fundParams
+		err           string
+	}{
+		{
+			// no addresses provided
+			buildParamsFn: func() *fundParams {
+				return &fundParams{}
+			},
+			err: errNoAddressesProvided.Error(),
+		},
+		{
+			// inconsistent length (addresses vs amounts)
+			buildParamsFn: func() *fundParams {
+				return &fundParams{
+					addresses: []string{"0x1", "0x2"},
+					amounts:   []string{"10"},
+				}
+			},
+			err: errInconsistentLength.Error(),
+		},
+		{
+			// address contains invalid characters
+			buildParamsFn: func() *fundParams {
+				return &fundParams{
+					addresses: []string{"0x10", "0x20"},
+					amounts:   []string{"10", "20"},
+				}
+			},
+			err: "address \x10 has invalid length",
+		},
+		{
+			// stake token address omitted
+			buildParamsFn: func() *fundParams {
+				return &fundParams{
+					mintStakeToken: true,
+					addresses: []string{
+						types.StringToAddress("0x10").String(),
+						types.StringToAddress("0x20").String()},
+					amounts: []string{"10", "20"},
+				}
+			},
+			err: rootHelper.ErrMandatoryStakeToken.Error(),
+		},
+		{
+			// stake token address omitted
+			buildParamsFn: func() *fundParams {
+				return &fundParams{
+					mintStakeToken: true,
+					stakeTokenAddr: "0xA",
+					addresses: []string{
+						types.StringToAddress("0x10").String(),
+						types.StringToAddress("0x20").String()},
+					amounts: []string{"10", "20"},
+				}
+			},
+			err: "invalid stake token address is provided",
+		},
+		{
+			// valid scenario
+			buildParamsFn: func() *fundParams {
+				return &fundParams{
+					addresses: []string{
+						types.StringToAddress("0x10").String(),
+						types.StringToAddress("0x20").String()},
+					amounts: []string{"10", "20"},
+				}
+			},
+			err: "",
+		},
+	}
+
+	for i, c := range cases {
+		c := c
+		i := i
+
+		t.Run(fmt.Sprintf("case#%d", i+1), func(t *testing.T) {
+			t.Parallel()
+
+			fp := c.buildParamsFn()
+
+			err := fp.validateFlags()
+			if c.err != "" {
+				require.ErrorContains(t, err, c.err)
+			} else {
+				require.NoError(t, err)
+			}
+		})
+	}
+}

command/rootchain/helper/utils.go

@@ -40,9 +40,10 @@ const (
 )
 
 var (
-	ErrRootchainNotFound = errors.New("rootchain not found")
-	ErrRootchainPortBind = errors.New("port 8545 is not bind with localhost")
-	errTestModeSecrets   = errors.New("rootchain test mode does not imply specifying secrets parameters")
+	ErrRootchainNotFound   = errors.New("rootchain not found")
+	ErrRootchainPortBind   = errors.New("port 8545 is not bind with localhost")
+	ErrMandatoryStakeToken = errors.New("stake token address is mandatory")
+	errTestModeSecrets     = errors.New("rootchain test mode does not imply specifying secrets parameters")
 
 	rootchainAccountKey *wallet.Key
 )

command/rootchain/supernet/stakemanager/params.go

@@ -5,7 +5,9 @@ import (
 	"os"
 
 	"github.com/0xPolygon/polygon-edge/command/helper"
+	rootHelper "github.com/0xPolygon/polygon-edge/command/rootchain/helper"
 	sidechainHelper "github.com/0xPolygon/polygon-edge/command/sidechain"
+	"github.com/0xPolygon/polygon-edge/types"
 )
 
 type stakeManagerDeployParams struct {
@@ -19,10 +21,23 @@ type stakeManagerDeployParams struct {
 }
 
 func (s *stakeManagerDeployParams) validateFlags() error {
-	if !s.isTestMode && s.privateKey == "" {
-		return sidechainHelper.ValidateSecretFlags(s.accountDir, s.accountConfig)
+	if !s.isTestMode {
+		// private key is mandatory
+		if s.privateKey == "" {
+			return sidechainHelper.ValidateSecretFlags(s.accountDir, s.accountConfig)
+		}
+
+		// stake token address is mandatory
+		if s.stakeTokenAddress == "" {
+			return rootHelper.ErrMandatoryStakeToken
+		}
+
+		if err := types.IsValidAddress(s.stakeTokenAddress); err != nil {
+			return fmt.Errorf("invalid stake token address is provided: %w", err)
+		}
 	}
 
+	// check if provided genesis path is valid
 	if _, err := os.Stat(s.genesisPath); err != nil {
 		return fmt.Errorf("provided genesis path '%s' is invalid. Error: %w ", s.genesisPath, err)
 	}

command/rootchain/supernet/stakemanager/params_test.go

@@ -0,0 +1,48 @@
+package stakemanager
+
+import (
+	"fmt"
+	"testing"
+
+	"github.com/stretchr/testify/require"
+
+	rootHelper "github.com/0xPolygon/polygon-edge/command/rootchain/helper"
+)
+
+func TestValidateFlags(t *testing.T) {
+	t.Parallel()
+
+	testCases := []struct {
+		params *stakeManagerDeployParams
+		errMsg string
+	}{
+		{
+			params: &stakeManagerDeployParams{
+				isTestMode:        false,
+				privateKey:        rootHelper.TestAccountPrivKey,
+				stakeTokenAddress: "",
+			},
+			errMsg: rootHelper.ErrMandatoryStakeToken.Error(),
+		},
+		{
+			params: &stakeManagerDeployParams{
+				isTestMode:        false,
+				privateKey:        rootHelper.TestAccountPrivKey,
+				stakeTokenAddress: "0x1B",
+			},
+			errMsg: "invalid stake token address is provided",
+		},
+	}
+
+	for i, tc := range testCases {
+		i := i
+		tc := tc
+
+		t.Run(fmt.Sprintf("case#%d", i+1), func(t *testing.T) {
+			t.Parallel()
+
+			err := tc.params.validateFlags()
+			require.ErrorContains(t, err, tc.errMsg)
+		})
+	}
+}

command/rootchain/supernet/stakemanager/stake_manager_deploy.go

@@ -24,7 +24,7 @@ func GetCommand() *cobra.Command {
 	stakeMgrDeployCmd := &cobra.Command{
 		Use:     "stake-manager-deploy",
 		Short:   "Command for deploying stake manager contract on rootchain",
-		PreRunE: runPreRun,
+		PreRunE: preRunCommand,
 		RunE:    runCommand,
 	}
 
@@ -33,7 +33,7 @@ func GetCommand() *cobra.Command {
 	return stakeMgrDeployCmd
 }
 
-func runPreRun(cmd *cobra.Command, _ []string) error {
+func preRunCommand(cmd *cobra.Command, _ []string) error {
 	params.jsonRPC = helper.GetJSONRPCAddress(cmd)
 
 	return params.validateFlags()

types/types.go

@@ -159,7 +159,7 @@ func IsValidAddress(address string) error {
 
 	// check if the address has the correct length
 	if len(decodedAddress) != AddressLength {
-		return fmt.Errorf("address %s has invalid length", address)
+		return fmt.Errorf("address %s has invalid length", string(decodedAddress))
 	}
 
 	return nil