Skip to content

e2e: add bridge e2e tests #720

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

Merged
merged 5 commits into from
Oct 31, 2022
Merged

Conversation

tmigone
Copy link
Member

@tmigone tmigone commented Sep 28, 2022

This PR adds e2e tests to the bridge contracts (cross checks addresses configured in L1GraphTokenGateway, L2GraphTokenGateway, L1GraphToken, L2GraphToken and BridgeEscrow match the deployed contract addresses).

@tmigone tmigone changed the base branch from dev to tmigone/e2e-l2-bridge September 28, 2022 21:40
@codecov
Copy link

codecov bot commented Sep 28, 2022

Codecov Report

Base: 91.53% // Head: 91.53% // No change to project coverage 👍

Coverage data is based on head (a1c3dcd) compared to base (6de2ea9).
Patch has no changes to coverable lines.

Additional details and impacted files
@@                  Coverage Diff                   @@
##           tmigone/e2e-l2-bridge     #720   +/-   ##
======================================================
  Coverage                  91.53%   91.53%           
======================================================
  Files                         42       42           
  Lines                       1984     1984           
  Branches                     346      346           
======================================================
  Hits                        1816     1816           
  Misses                       168      168           
Flag Coverage Δ
unittests 91.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Signed-off-by: Tomás Migone <[email protected]>
@tmigone tmigone merged commit af67ed0 into tmigone/e2e-l2-bridge Oct 31, 2022
@tmigone tmigone deleted the tmigone/bridge-e2e branch October 31, 2022 18:23
pcarranzav pushed a commit that referenced this pull request Oct 31, 2022
* feat(e2e): run L1 and L2 together in e2e tests

Signed-off-by: Tomás Migone <[email protected]>
tmigone added a commit that referenced this pull request Nov 2, 2022
* feat(e2e): run L1 and L2 together in e2e tests

Signed-off-by: Tomás Migone <[email protected]>
tmigone added a commit that referenced this pull request Nov 2, 2022
* feat(e2e): run L1 and L2 together in e2e tests

Signed-off-by: Tomás Migone <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant