contracts icon indicating copy to clipboard operation
contracts copied to clipboard

feat: add e2e deployment tests for L2s

Open tmigone opened this issue 3 years ago • 1 comments
trafficstars

This PR adds task changes and e2e deployment tests for L1/L2. This allows us to validate steps 1.1 through 1.12 with tests (except 1.9 & 1.10 which are pending GRE support for multiple layers).

Changes

  • Add chainId as optional parameter when initializing GRE. This gives some flexibility when choosing where to connect to.
  • hh task e2e no longer runs scenarios, they can be run separately if needed with e2e:scenario.
  • hh task migrate:unpause will unpause the bridge as well as the protocol.
  • Fix bug where migrate:ownership would fail if run on L2.
  • Add L2 e2e deployment tests
  • Add bridge (L1 & L2) e2e tests

tmigone avatar Aug 05 '22 15:08 tmigone

Codecov Report

Merging #676 (b8ea9e6) into l2-testnet (19de7a2) will not change coverage. The diff coverage is n/a.

@@             Coverage Diff             @@
##           l2-testnet     #676   +/-   ##
===========================================
  Coverage       92.13%   92.13%           
===========================================
  Files              44       44           
  Lines            2137     2137           
  Branches          370      370           
===========================================
  Hits             1969     1969           
  Misses            168      168           
Flag Coverage Δ
unittests 92.13% <ø> (ø)

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.

codecov[bot] avatar Aug 05 '22 16:08 codecov[bot]

https://github.com/graphprotocol/contracts/pull/704 cherry picks commits from here into pcv/l2-bridge. Follow up PR will get the reservoir changes into the drip-rewards branch

tmigone avatar Sep 07 '22 09:09 tmigone