Repository navigation
disable flaky tests temporarily & update version - #20531
Conversation
There was a problem hiding this comment.
Pull request overview
This PR temporarily disables two flaky CCIP upgrade tests related to EVM to Sui integration by commenting them out in the integration test configuration file.
Key changes:
- Commented out
Test_CCIP_Upgrade_EVM2Suitest configuration - Commented out
Test_CCIP_Upgrade_CommonPkg_EVM2Suitest configuration
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # - id: smoke/ccip/ccip_sui_upgrade_test.go:Test_CCIP_Upgrade_EVM2Sui | ||
| # path: integration-tests/smoke/ccip/ccip_sui_upgrade_test.go | ||
| # test_env_type: in-memory | ||
| # runs_on_self_hosted: runs-on/cpu=16/ram=64/family=m6i+m5.*/spot=false/image=ubuntu24-full-x64/extras=s3-cache+tmpfs | ||
| # triggers: | ||
| # - PR Integration CCIP Tests | ||
| # - Nightly Integration CCIP Tests | ||
| # test_cmd: | | ||
| # go test ./smoke/ccip -run "Test_CCIP_Upgrade_EVM2Sui" -timeout 25m -test.parallel=1 -count=1 -json | ||
| # test_go_project_path: integration-tests | ||
| # sui_cli_version: mainnet-1.60.1 | ||
| # install_plugins_public: true | ||
| # free_disk_space: true |
There was a problem hiding this comment.
Add a comment explaining why these tests are disabled and reference a tracking issue (e.g., 'Temporarily disabled due to flakiness - see LINK-777'). This helps maintainers understand when and why these tests should be re-enabled.
| # - id: smoke/ccip/ccip_sui_upgrade_test.go:Test_CCIP_Upgrade_CommonPkg_EVM2Sui | ||
| # path: integration-tests/smoke/ccip/ccip_sui_upgrade_test.go | ||
| # test_env_type: in-memory | ||
| # runs_on: ubuntu-latest | ||
| # triggers: | ||
| # - PR Integration CCIP Tests | ||
| # - Nightly Integration CCIP Tests | ||
| # test_cmd: | | ||
| # go test ./smoke/ccip -run "Test_CCIP_Upgrade_CommonPkg_EVM2Sui" -timeout 25m -test.parallel=1 -count=1 -json | ||
| # test_go_project_path: integration-tests | ||
| # sui_cli_version: mainnet-1.60.1 | ||
| # install_plugins_public: true | ||
| # free_disk_space: true |
There was a problem hiding this comment.
Add a comment explaining why these tests are disabled and reference a tracking issue (e.g., 'Temporarily disabled due to flakiness - see LINK-777'). This helps maintainers understand when and why these tests should be re-enabled.
|
added a ticket to track fixing these: https://smartcontract-it.atlassian.net/browse/NONEVM-2558 |
6eeed22
FelixFan1992
changed the title
|
Requires
Supports