Conversation
yashnevatia
requested review from
agusaldasoro,
asoliman92,
chris-de-leon-cll,
makramkd,
matYang,
tt-cll and
winder
|
👋 yashnevatia, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
There was a problem hiding this comment.
Pull request overview
This PR re-enables the Test_CCIPMessaging_EVM2Solana integration test that was previously disabled in the CI workflow. The test was commented out with a note indicating it was broken as of 2026-02-20.
Changes:
- Uncomments the
Test_CCIPMessaging_EVM2Solanatest entry in the test matrix
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - name: "Messaging Test Test_CCIPMessaging_EVM2Solana" | ||
| file: ccip_messaging_test.go | ||
| run: "^Test_CCIPMessaging_EVM2Solana$" | ||
| timeout: 30m |
There was a problem hiding this comment.
This test is being re-enabled but the comment indicating it was broken is still present on line 65. If the test has been fixed and is ready to run, the comment about it being broken should be removed. If the test hasn't been fixed, it should remain commented out. Please either remove the "Tests broken as of 2026-02-20" comment if the test is now working, or keep the test commented out if it's still broken.
core ref: 17fd202a8b96ec8a362959a2b949c603a41c0bd5