Skip to content

Implement integration test for UPDATE_GER note consumption#1854

Closed
Fumuran wants to merge 28 commits intonextfrom
andrew-e2e-bridging-in-n-out
Closed

Implement integration test for UPDATE_GER note consumption#1854
Fumuran wants to merge 28 commits intonextfrom
andrew-e2e-bridging-in-n-out

Conversation

@Fumuran
Copy link
Copy Markdown
Contributor

@Fumuran Fumuran commented Mar 4, 2026

WiP

Copy link
Copy Markdown
Contributor

@mmagician mmagician left a comment

Choose a reason for hiding this comment

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

Looks good!

Comment thread bin/integration-tests/src/tests/agglayer/ger.rs Outdated
Comment thread bin/integration-tests/src/tests/mod.rs Outdated
@Fumuran Fumuran added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Mar 4, 2026
Copy link
Copy Markdown
Contributor

@mmagician mmagician left a comment

Choose a reason for hiding this comment

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

LGTM

Let's not merge this yet - instead, once client crates v0.14.0-alpha.1 is released, we will have a release/v0.14.0-alpha branch, and we should target that.

The reason is that I want to hand over these tests to Gateway ASAP (can be from a branch compatible with the release), but before merging, we should have all this agglayer integration work at least looked at by Ignacio & co.

feat: create agglayer accounts at genesis
Base automatically changed from andrew-migrate-to-next-base to next March 5, 2026 15:13
@Fumuran Fumuran marked this pull request as ready for review March 5, 2026 15:50
partylikeits1983 and others added 2 commits March 8, 2026 16:03
* feat: add agglayer e2e bridge in / out tests

* feat: add foundry install to CI

* refactor: add multiple client instances & refactor

* fix: forge install command fix

* fix: update CI install process

* fix: merge conflicts

* fix: test.yml

* fix: init dir

* refactor: make agglayer_test_utils non-public

* empty commit test for CI

* feat: add wait_for_note logic in test

* fix: clippy warning

* feat: setup accounts or load from genesis config

* chore: re-use setup structure

* refactor: destructure CoreAccountIds into tuple

* refactor: replace ClientWithKeystore tuple with Actor struct

* refactor: extract destination_network variable in bridge-out

* style: apply formatting from make lint

* refactor: rename Actor to ClientPair

* lint

* chore: lint

---------

Co-authored-by: riemann <aleqvids@gmail.com>
Co-authored-by: Claude (Opus) <noreply@anthropic.com>
@mmagician
Copy link
Copy Markdown
Contributor

closing in favor of #1854 (cleaner branch name)

@mmagician mmagician closed this Mar 10, 2026
@mmagician mmagician deleted the andrew-e2e-bridging-in-n-out branch April 7, 2026 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants