Skip to content
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

Make router destination/source chain specific #1242

Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
b670fe2
make source router configurable per chain on onRamp
RyanRHall Jul 29, 2024
955b29e
make source router configurable per chain on offRamp
RyanRHall Jul 31, 2024
4a0fef6
rename sourceRouter => router
RyanRHall Jul 31, 2024
3925ca6
update gas snapshot & wrappers
RyanRHall Jul 31, 2024
17369b0
fix integration test
RyanRHall Jul 31, 2024
d4bb04d
make wrappers
RyanRHall Jul 31, 2024
0f95beb
improve comment layouts
RyanRHall Aug 1, 2024
40af04e
gas golf forwardFromRouter
RyanRHall Aug 1, 2024
579dd5a
add test for revert case of applyDestChainConfigUpdates()
RyanRHall Aug 1, 2024
b703bf1
add empty list test case to applyDestChainConfigUpdates() test
RyanRHall Aug 1, 2024
1eeb738
add line breaks to test
RyanRHall Aug 1, 2024
0c52ebb
add router comparison in config equality assessment
RyanRHall Aug 1, 2024
f9c01f9
add missing vm.expectEmit()
RyanRHall Aug 1, 2024
0fc3c24
change address => IRouter in OnRamp; add getRouter() func
RyanRHall Aug 2, 2024
498c4c1
update wrappers & snapshot
RyanRHall Aug 2, 2024
430e19d
Merge branch 'ccip-develop' into CCIP-2837-multi-on-ramp-make-router-…
RyanRHall Aug 2, 2024
0f1ae75
Merge branch 'ccip-develop' into CCIP-2837-multi-on-ramp-make-router-…
RyanRHall Aug 5, 2024
f33493e
wip
RyanRHall Aug 5, 2024
3b85277
fix integration tests
RyanRHall Aug 5, 2024
8c07f16
Merge branch 'ccip-develop' into CCIP-2837-multi-on-ramp-make-router-…
RyanRHall Aug 5, 2024
b69ebba
fix misleading comments
RyanRHall Aug 5, 2024
c7f4682
Merge branch 'ccip-develop' into CCIP-2837-multi-on-ramp-make-router-…
RyanRHall Aug 5, 2024
603f269
Merge branch 'ccip-develop' into CCIP-2837-multi-on-ramp-make-router-…
RyanRHall Aug 6, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add missing vm.expectEmit()
  • Loading branch information
RyanRHall committed Aug 1, 2024
commit f9c01f98d43a542be52432791a282a86e9505f8e
Original file line number Diff line number Diff line change
Expand Up @@ -765,6 +765,7 @@ contract EVM2EVMMultiOnRamp_applyDestChainConfigUpdates is EVM2EVMMultiOnRampSet
emit EVM2EVMMultiOnRamp.DestChainConfigSet(
DEST_CHAIN_SELECTOR, EVM2EVMMultiOnRamp.DestChainConfig(0, address(s_sourceRouter))
);
vm.expectEmit();
emit EVM2EVMMultiOnRamp.DestChainConfigSet(9999, EVM2EVMMultiOnRamp.DestChainConfig(0, address(9999)));
RyanRHall marked this conversation as resolved.
Show resolved Hide resolved
s_onRamp.applyDestChainConfigUpdates(configArgs);
assertEq(address(s_sourceRouter), s_onRamp.getDestChainConfig(DEST_CHAIN_SELECTOR).router);
Expand Down
Loading