Skip to content

Tests: migrate remaining YAML integration tests to Go runner (PLT-738)#3612

Open
amir-deris wants to merge 3 commits into
mainfrom
amir/plt-738-migrate-rest-yaml-tests-to-go-runner
Open

Tests: migrate remaining YAML integration tests to Go runner (PLT-738)#3612
amir-deris wants to merge 3 commits into
mainfrom
amir/plt-738-migrate-rest-yaml-tests-to-go-runner

Conversation

@amir-deris

@amir-deris amir-deris commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Wires up the two remaining YAML test files (major_upgrade_test.yaml, minor_upgrade_test.yaml) as TestUpgradeMajor / TestUpgradeMinor in runner_test.go
  • Splits TestWasmModuleTestWasmModuleCore + TestWasmModuleEmergencyWithdraw (CI re-deploys the timelocked token contract between them; the withdraw test exhausts state the emergency-withdraw test depends on)
  • Splits TestSeiDBTestSeiDBFlatKV + TestSeiDBStateStore (they run in separate CI matrix jobs)
  • Replaces every python3 integration_test/scripts/runner.py call in integration-test.yml with the equivalent go test -tags yaml_integration command, including the shared startup verify step
  • Removes actions/setup-python and pip3 install pyyaml from the CI job steps
  • Updates upgrade_module/scripts/launch_cluster.sh (local cluster helper) to use the Go runner
  • Deletes integration_test/scripts/runner.py

Test plan

  • CI matrix passes end-to-end (all matrix jobs use the Go runner now)
  • Upgrade module jobs (Upgrade Module (Major/Minor) and their Autobahn variants) exercise the new TestUpgradeMajor / TestUpgradeMinor functions
  • Wasm module job redeploys contract between TestWasmModuleCore and TestWasmModuleEmergencyWithdraw
  • No python3 or pyyaml references remain in CI workflow

🤖 Generated with Claude Code

@amir-deris amir-deris self-assigned this Jun 18, 2026
@amir-deris amir-deris changed the title Migrated remaining yaml tests to go runner Tests: migrate remaining YAML integration tests to Go runner (PLT-738) Jun 18, 2026
@cursor

cursor Bot commented Jun 18, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes how all Docker integration matrix jobs execute YAML suites; any behavioral gap between the deleted Python runner and the Go driver would surface as CI failures rather than affecting production code.

Overview
Completes the YAML integration test migration by removing the Python integration_test/scripts/runner.py driver and running every former runner.py invocation through go test -tags yaml_integration on ./integration_test/runner/....

CI (integration-test.yml) matrix scripts and the shared startup check now call named Go tests (e.g. TestWasmModuleCore, TestUpgradeMajor, TestSeiDBFlatKV) instead of per-file Python commands. Python setup (actions/setup-python, pip3 install pyyaml) is dropped from the job.

runner_test.go adds or reshapes entrypoints so matrix jobs match old script boundaries: Wasm split into core vs emergency withdraw (with redeploy between them in CI), SeiDB split into FlatKV vs state store, new TestUpgradeMajor / TestUpgradeMinor, and TestAutobahnBankModule (only send_funds_test.yaml because Autobahn hangs on -b block cases).

launch_cluster.sh uses TestStartup via Go test instead of Python for local upgrade cluster bring-up.

Reviewed by Cursor Bugbot for commit cee9714. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedJun 22, 2026, 7:58 PM

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.12%. Comparing base (daba2e7) to head (cee9714).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3612      +/-   ##
==========================================
- Coverage   58.99%   58.12%   -0.88%     
==========================================
  Files        2224     2150      -74     
  Lines      182708   174161    -8547     
==========================================
- Hits       107782   101223    -6559     
+ Misses      65235    63947    -1288     
+ Partials     9691     8991     -700     
Flag Coverage Δ
sei-chain-pr 0.00% <ø> (?)
sei-db 70.41% <ø> (ø)
sei-db-state-db ?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 74 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ 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.

Comment thread .github/workflows/integration-test.yml Outdated
@amir-deris amir-deris requested review from bdchatham and masih June 22, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants