Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the TON deployment test adapter and related bindings to support QA-oriented test flows (message ID plumbing, receiver behavior toggles, allowlist/RMN helpers), and adds initial QA documentation.
Changes:
- Extend TON test adapter with new QA helper methods (message ID return, execution validation, allowlist + RMN curse toggles, etc.).
- Update receiver bindings (exported behavior constants + new TL-B message structs) and adjust deployment code to use new names.
- Add QA test documentation and update the deployment module dependency on
chainlink-ccip/deployment.
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/ccip/bindings/receiver/receiver.go | Renames behavior constants and adds TL-B message structs for receiver admin ops. |
| docs/tests/qa.md | Adds QA testing documentation (automated vs interactive flows). |
| deployment/testadapter/test_adapter.go | Adds/updates TON QA adapter methods and execution filtering; returns message ID from SendMessage. |
| deployment/go.mod | Bumps chainlink-ccip/deployment dependency version. |
| deployment/go.sum | Updates sums for the bumped deployment dependency. |
| deployment/ccip/sequence/deploy_ccip.go | Updates receiver behavior constant usage to the renamed identifier. |
| deployment/ccip/1_6_0/sequences/deploy_chain_contracts.go | Moves interface compliance assertion to a more visible location. |
| deployment/ccip/1_6_0/sequences/connect_chains.go | Same as above for lane adapter; minor import grouping tweak. |
| .github/workflows/test-smoke.yml | Points smoke workflow at a non-main branch in the target repo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| WORKFLOW_FILE: .github/workflows/test_smoke.yml | ||
| TARGET_REPO: smartcontractkit/chainlink-ccip | ||
| TARGET_BRANCH: main | ||
| TARGET_BRANCH: ton-qa-tests # TODO change back to main |
There was a problem hiding this comment.
This workflow is pinned to TARGET_BRANCH: ton-qa-tests with a TODO to change back. Merging this as-is will make the smoke workflow depend on a non-default branch in smartcontractkit/chainlink-ccip, which is brittle and may break CI once that branch is deleted or diverges. Please restore main (or make the branch configurable via workflow input).
| TARGET_BRANCH: ton-qa-tests # TODO change back to main | |
| TARGET_BRANCH: main |
There was a problem hiding this comment.
As intended
No description provided.