Skip to content

Commit 722e93b

Browse files
committed
feat: add v2.0.0 upgrade handler and update e2e tests
- Add app/upgrades/v2_0_0/ with constants (UpgradeName, StoreUpgrades) and upgrade handler (noop with RunMigrations) - Register v2.0.0 handler in app.go setupUpgradeHandlers() - Wire store loader for v2.0.0 store key additions/deletions - Update e2e upgrade tests: base version v1.1.2 → v1.2.0, upgrade target → v2.0.0
1 parent d6ddb64 commit 722e93b

6 files changed

Lines changed: 46 additions & 14 deletions

File tree

app/app.go

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,7 @@ import (
150150

151151
ethante "github.com/evmos/evmos/v12/app/ante/evm"
152152
"github.com/evmos/evmos/v12/app/upgrades"
153+
v2_0_0 "github.com/evmos/evmos/v12/app/upgrades/v2_0_0"
153154
"github.com/evmos/evmos/v12/encoding"
154155
servercfg "github.com/evmos/evmos/v12/server/config"
155156
srvflags "github.com/evmos/evmos/v12/server/flags"
@@ -1559,16 +1560,18 @@ func (app *Evmos) setupUpgradeHandlers() {
15591560
return app.mm.RunMigrations(ctx, app.configurator, fromVM)
15601561
})
15611562

1563+
app.UpgradeKeeper.SetUpgradeHandler(
1564+
v2_0_0.UpgradeName,
1565+
v2_0_0.CreateUpgradeHandler(app.mm, app.configurator),
1566+
)
1567+
15621568
// testnet only upgrade Handlers
15631569
app.UpgradeKeeper.SetUpgradeHandler(
15641570
"testnet-gov-param-fix",
15651571
upgrades.TestnetGovParamFix(&app.GovKeeper, app.EvmKeeper, app.mm, app.configurator),
15661572
)
15671573

1568-
var storeUpgrades *storetypes.StoreUpgrades
1569-
1570-
if storeUpgrades != nil {
1571-
// configure store loader that checks if version == upgradeHeight and applies store upgrades
1572-
app.SetStoreLoader(upgradetypes.UpgradeStoreLoader(upgradeInfo.Height, storeUpgrades))
1574+
if upgradeInfo.Name == v2_0_0.UpgradeName && !app.UpgradeKeeper.IsSkipHeight(upgradeInfo.Height) {
1575+
app.SetStoreLoader(upgradetypes.UpgradeStoreLoader(upgradeInfo.Height, v2_0_0.StoreUpgrades))
15731576
}
15741577
}

app/upgrades/v2_0_0/constants.go

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
package v2_0_0
2+
3+
import (
4+
storetypes "cosmossdk.io/store/types"
5+
)
6+
7+
const UpgradeName = "v2.0.0"
8+
9+
var StoreUpgrades = &storetypes.StoreUpgrades{
10+
Added: []string{},
11+
Deleted: []string{},
12+
}

app/upgrades/v2_0_0/upgrades.go

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
package v2_0_0
2+
3+
import (
4+
"context"
5+
6+
upgradetypes "cosmossdk.io/x/upgrade/types"
7+
"github.com/cosmos/cosmos-sdk/types/module"
8+
)
9+
10+
func CreateUpgradeHandler(
11+
mm *module.Manager,
12+
configurator module.Configurator,
13+
) upgradetypes.UpgradeHandler {
14+
return func(ctx context.Context, _ upgradetypes.Plan, fromVM module.VersionMap) (module.VersionMap, error) {
15+
return mm.RunMigrations(ctx, configurator, fromVM)
16+
}
17+
}

e2e/kind/tests/test_upgrade_comprehensive.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
# then runs all verify functions.
1010
#
1111
# Usage:
12-
# OLD_VERSION=v1.1.2 bash tests/test_upgrade_comprehensive.sh
12+
# OLD_VERSION=v1.2.0 bash tests/test_upgrade_comprehensive.sh
1313
# UPGRADE_MODE=hardfork OLD_VERSION=main bash tests/test_upgrade_comprehensive.sh
1414
# TX_ROUNDS=5 bash tests/test_upgrade_comprehensive.sh # 5 rounds of all txs
1515
#
@@ -20,8 +20,8 @@
2020
source "$(dirname "$0")/../framework/framework.sh"
2121
fw_init
2222

23-
OLD_VERSION="${OLD_VERSION:-v1.1.2}"
24-
UPGRADE_NAME="${UPGRADE_NAME:-v1.2.0}"
23+
OLD_VERSION="${OLD_VERSION:-v1.2.0}"
24+
UPGRADE_NAME="${UPGRADE_NAME:-v2.0.0}"
2525
UPGRADE_MODE="${UPGRADE_MODE:-governance}"
2626
TX_ROUNDS="${TX_ROUNDS:-3}"
2727
RELEASE_TAG="${RELEASE_TAG:-}"

e2e/kind/tests/test_upgrade_governance.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,13 @@
77
# release tag (e.g., v1.1.2) — not main or the current version.
88
#
99
# Usage:
10-
# OLD_VERSION=v1.1.2 bash tests/test_upgrade_governance.sh
10+
# OLD_VERSION=v1.2.0 bash tests/test_upgrade_governance.sh
1111

1212
source "$(dirname "$0")/../framework/framework.sh"
1313
fw_init
1414

15-
OLD_VERSION="${OLD_VERSION:-v1.1.2}"
16-
UPGRADE_NAME="${UPGRADE_NAME:-v1.2.0}"
15+
OLD_VERSION="${OLD_VERSION:-v1.2.0}"
16+
UPGRADE_NAME="${UPGRADE_NAME:-v2.0.0}"
1717
FEES="200000000000000amoca"
1818

1919
# ── Setup: deploy old version ────────────────────────────────────────────────

e2e/kind/tests/test_upgrade_hardfork.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,14 @@
33
# and validates that state is preserved and the chain continues.
44
#
55
# Usage:
6-
# OLD_VERSION=v1.1.2 bash tests/test_upgrade_hardfork.sh
6+
# OLD_VERSION=v1.2.0 bash tests/test_upgrade_hardfork.sh
77
# OLD_VERSION=main bash tests/test_upgrade_hardfork.sh
88

99
source "$(dirname "$0")/../framework/framework.sh"
1010
fw_init
1111

12-
OLD_VERSION="${OLD_VERSION:-v1.1.2}"
13-
UPGRADE_NAME="${UPGRADE_NAME:-v1.2.0}"
12+
OLD_VERSION="${OLD_VERSION:-v1.2.0}"
13+
UPGRADE_NAME="${UPGRADE_NAME:-v2.0.0}"
1414
FEES="200000000000000amoca"
1515

1616
# ── Setup: deploy old version ────────────────────────────────────────────────

0 commit comments

Comments
 (0)