Skip to content

[PoC] integration-tests: add new module #969

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

makramkd
Copy link
Collaborator

@makramkd makramkd commented Jun 3, 2025

integration-tests is a new go module that will contain all of the CCIP E2E tests. This PR just moves over ccip_messaging_test.go from core but in principle all other tests are movable.

One thing to keep in mind is that the interface implementations in chainlink are still a risk when changes occur, we need to come back here to bump the chainlink/v2 and chainlink/deployment versions in order to test with the new implementations.

"github.com/smartcontractkit/chainlink/deployment/ccip/manualexechelpers"
"github.com/smartcontractkit/chainlink/deployment/ccip/shared/stateview"
"github.com/smartcontractkit/chainlink/deployment/environment/memory"
testsetups "github.com/smartcontractkit/chainlink/integration-tests/testsetups/ccip"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Arguably we can move this as well?

sigs.k8s.io/yaml v1.4.0 // indirect
)

replace github.com/smartcontractkit/chainlink-ccip => ..
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This always imports latest chainlink-ccip - I think we need to do the same with the solana go.mod?

Copy link

github-actions bot commented Jun 3, 2025

Metric mk/move-over-e2e-tests main
Coverage 72.4% 72.3%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant