From 9486518ac3dca481c041d8125f6d24f754cd47ad Mon Sep 17 00:00:00 2001 From: marcello33 Date: Thu, 15 Feb 2024 15:29:31 +0100 Subject: [PATCH] chg: increase test balances to pay for the DefaultMaxFee in heimdall / re-enable TestAccountRetriever test --- testutil/network/network.go | 17 +++++++++-------- types/staking.go | 2 +- x/auth/types/account_retriever_test.go | 1 - 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/testutil/network/network.go b/testutil/network/network.go index bb016a0a2541..35437ffabe39 100644 --- a/testutil/network/network.go +++ b/testutil/network/network.go @@ -150,14 +150,15 @@ func DefaultConfig(factory TestFixtureFactory) Config { NumValidators: 4, BondDenom: sdk.DefaultBondDenom, MinGasPrices: fmt.Sprintf("0.000006%s", sdk.DefaultBondDenom), - AccountTokens: sdk.TokensFromConsensusPower(1000, sdk.DefaultPowerReduction), - StakingTokens: sdk.TokensFromConsensusPower(500, sdk.DefaultPowerReduction), - BondedTokens: sdk.TokensFromConsensusPower(100, sdk.DefaultPowerReduction), - PruningStrategy: pruningtypes.PruningOptionNothing, - CleanupDir: true, - SigningAlgo: string(hd.Secp256k1Type), - KeyringOptions: []keyring.Option{}, - PrintMnemonic: false, + // HV2: increasing tokens for tests to be able to pay the `DefaultTxFees` defined in heimdall + AccountTokens: sdk.TokensFromConsensusPower(1000000000000000000, sdk.DefaultPowerReduction), + StakingTokens: sdk.TokensFromConsensusPower(500000000000000000, sdk.DefaultPowerReduction), + BondedTokens: sdk.TokensFromConsensusPower(100000000, sdk.DefaultPowerReduction), + PruningStrategy: pruningtypes.PruningOptionNothing, + CleanupDir: true, + SigningAlgo: string(hd.Secp256k1Type), + KeyringOptions: []keyring.Option{}, + PrintMnemonic: false, } } diff --git a/types/staking.go b/types/staking.go index 38019d46a422..2978f3f7fc8f 100644 --- a/types/staking.go +++ b/types/staking.go @@ -18,7 +18,7 @@ const ValidatorUpdateDelay int64 = 1 var ( // DefaultBondDenom is the default bondable coin denomination (defaults to stake) // Overwriting this value has the side effect of changing the default denomination in genesis - DefaultBondDenom = "stake" + DefaultBondDenom = "matic" // DefaultPowerReduction is the default amount of staking tokens required for 1 unit of consensus-engine power DefaultPowerReduction = sdkmath.NewIntFromUint64(1000000) diff --git a/x/auth/types/account_retriever_test.go b/x/auth/types/account_retriever_test.go index 576407118462..c3b47c62cf90 100644 --- a/x/auth/types/account_retriever_test.go +++ b/x/auth/types/account_retriever_test.go @@ -11,7 +11,6 @@ import ( ) func TestAccountRetriever(t *testing.T) { - t.Skip("skipping test as not relevant to Heimdall (no depinject)") // TODO HV2: fix this test cfg, err := network.DefaultConfigWithAppConfig(testutil.AppConfig) require.NoError(t, err) cfg.NumValidators = 1