smartcontractkit / ccip

Cross Chain Interoperability Protocol
Other
119 stars 47 forks source link

Updating MockRouter to align with EVM2EVMOffRamp. #1425

Closed zeuslawyer closed 1 month ago

zeuslawyer commented 1 month ago

Specifically, allow for empty message with zero gas.

Motivation

EVM2EVMOffRamp checks the message for content and gaslimit and skips calling ccip receive if they're not present.
Consequently the mock errors (ReceiverError) when sending only tokens to a smart contract. This update will help chainlink-local correctly process fork-based tests too.

Solution

The MockRouter implementation of _routeMessage was last updated in https://github.com/smartcontractkit/ccip/pull/669/ and is out of sync with the logic in EVM2EVMOffRamp.

zeuslawyer commented 1 month ago

moved to https://github.com/smartcontractkit/ccip/pull/1427