Tests: migrate remaining YAML integration tests to Go runner (PLT-738)#3612
Tests: migrate remaining YAML integration tests to Go runner (PLT-738)#3612amir-deris wants to merge 1 commit into
Conversation
PR SummaryLow Risk Overview The integration-test workflow replaces every
Reviewed by Cursor Bugbot for commit 415dfb7. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3612 +/- ##
==========================================
- Coverage 59.02% 58.14% -0.89%
==========================================
Files 2215 2141 -74
Lines 182521 173974 -8547
==========================================
- Hits 107734 101157 -6577
+ Misses 65091 63821 -1270
+ Partials 9696 8996 -700
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 415dfb7. Configure here.
| "python3 integration_test/scripts/runner.py integration_test/staking_module/staking_test.yaml", | ||
| "python3 integration_test/scripts/runner.py integration_test/bank_module/send_funds_test.yaml", | ||
| "python3 integration_test/scripts/runner.py integration_test/mint_module/mint_test.yaml" | ||
| "go test -tags yaml_integration -v -timeout $JOB_TIMEOUT ./integration_test/runner/... -run \"TestMintModule|TestStakingModule|TestBankModule\"" |
There was a problem hiding this comment.
Autobahn bank runs block-mode txs
High Severity
The Autobahn Mint & Staking & Bank matrix job now runs the full TestBankModule, which includes multi_sig_send_test.yaml and simulation_tx.yaml. Those YAML cases still broadcast with -b block, which Autobahn does not support and can hang CI. The previous Python runner only executed send_funds_test.yaml, which uses sync helpers.
Triggered by learned rule: test: use -b sync + side-effect polling, not -b block, in seid tx helpers
Reviewed by Cursor Bugbot for commit 415dfb7. Configure here.


Summary
major_upgrade_test.yaml,minor_upgrade_test.yaml) asTestUpgradeMajor/TestUpgradeMinorinrunner_test.goTestWasmModule→TestWasmModuleCore+TestWasmModuleEmergencyWithdraw(CI re-deploys the timelocked token contract between them; the withdraw test exhausts state the emergency-withdraw test depends on)TestSeiDB→TestSeiDBFlatKV+TestSeiDBStateStore(they run in separate CI matrix jobs)python3 integration_test/scripts/runner.pycall inintegration-test.ymlwith the equivalentgo test -tags yaml_integrationcommand, including the shared startup verify stepactions/setup-pythonandpip3 install pyyamlfrom the CI job stepsupgrade_module/scripts/launch_cluster.sh(local cluster helper) to use the Go runnerintegration_test/scripts/runner.pyTest plan
Upgrade Module (Major/Minor)and their Autobahn variants) exercise the newTestUpgradeMajor/TestUpgradeMinorfunctionsTestWasmModuleCoreandTestWasmModuleEmergencyWithdrawpython3orpyyamlreferences remain in CI workflow🤖 Generated with Claude Code