Skip to content

feat: add v2.0.0 upgrade handler#142

Merged
puneet2019 merged 2 commits into
mainfrom
feat/v2.0.0-upgrade-handler
Mar 31, 2026
Merged

feat: add v2.0.0 upgrade handler#142
puneet2019 merged 2 commits into
mainfrom
feat/v2.0.0-upgrade-handler

Conversation

@puneet2019

Copy link
Copy Markdown
Contributor

Summary

  • Add app/upgrades/v2_0_0/ with UpgradeName, StoreUpgrades (added/deleted store keys), and CreateUpgradeHandler
  • Register v2.0.0 handler in setupUpgradeHandlers() with store loader wiring
  • Update all e2e upgrade tests: base version v1.2.0, upgrade target v2.0.0

Files changed

  • app/upgrades/v2_0_0/constants.go — upgrade name + store key changes
  • app/upgrades/v2_0_0/upgrades.go — upgrade handler (noop + RunMigrations)
  • app/app.go — register handler + store loader
  • e2e/kind/tests/test_upgrade_*.sh — update base/target versions

Test plan

  • go build ./... passes
  • E2E governance upgrade test: OLD_VERSION=v1.2.0 bash tests/test_upgrade_governance.sh
  • E2E hardfork upgrade test: OLD_VERSION=v1.2.0 bash tests/test_upgrade_hardfork.sh
  • E2E comprehensive upgrade test: OLD_VERSION=v1.2.0 bash tests/test_upgrade_comprehensive.sh

🤖 Generated with Claude Code

- 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
@puneet2019 puneet2019 requested a review from a team March 31, 2026 02:57
Remove separate v2_0_0 package — keep everything inline in
setupUpgradeHandlers() matching existing v1.1.0/v1.2.0 pattern.
@puneet2019 puneet2019 merged commit 6b39b21 into main Mar 31, 2026
20 of 26 checks passed
@puneet2019 puneet2019 deleted the feat/v2.0.0-upgrade-handler branch March 31, 2026 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants