L2 public testnet deployment and related fixes by pcarranzav · Pull Request #751 · graphprotocol/contracts

pcarranzav · 2022-11-23T17:06:38Z

No description provided.

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

* feat: add optional expression logging to GRE Closes: #702 Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

test: print gas report, and ensure evm is killed

* chore: update GRE to use secure accounts Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

* feat(e2e): run L1 and L2 together in e2e tests Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

- feat(gre): add hardhat-secure-accounts to GRE (#696) - feat: add publish and mint to the CLI - feat: refactor GRE to support L1 and L2 simultaneously - fix: prevent underflow when subgraphs go under the minimum signal threshold

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

codecov · 2022-11-23T17:23:06Z

Codecov Report

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

Coverage data is based on head (51c0441) compared to base (d8fed24).
Patch has no changes to coverable lines.

❗ Current head 51c0441 differs from pull request most recent head aa85104. Consider uploading reports for the commit aa85104 to get more accurate results

Additional details and impacted files

@@              Coverage Diff               @@
##           pcv/l2-bridge     #751   +/-   ##
==============================================
  Coverage          91.59%   91.59%           
==============================================
  Files                 42       42           
  Lines               1999     1999           
  Branches             361      361           
==============================================
  Hits                1831     1831           
  Misses               168      168

Flag	Coverage Δ
unittests	`91.59% <ø> (ø)`

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.

abarmat

🥇 🥇 🥇

abarmat and others added 30 commits September 27, 2022 11:48

fix: use path.join to manage path concat

5e36583

feat: add publish and mint to the CLI

0ca43ff

chore: add reports for recent audits (#710)

45ce81e

chore: run GitHub actions when it makes sense (#707)

b6219d3

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

chore: provide sane defaults for graph config (#721)

04cfd3a

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

chore: remove rinkeby and kovan references (#729)

fdc0bae

feat: add optional expression logging to GRE (#728)

f1c5fe7

* feat: add optional expression logging to GRE Closes: #702 Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

test: print gas report, and ensure evm is killed

0b7055a

Merge pull request #732 from graphprotocol/pcv/test-gas-cat-report

3f16e5a

test: print gas report, and ensure evm is killed

feat(gre): add hardhat-secure-accounts to GRE (#696)

7e5ac05

* chore: update GRE to use secure accounts Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

chore: update arbitrum sdk to 3.0-rc1

598ecc3

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix(task): accept ownership only if pending owner matches

a7ba535

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

feat(task): split unpause task into protocol and bridge tasks

0aed5eb

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

feat(e2e): run L1 and L2 together in e2e tests

a3fa719

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

feat(e2e): fetch deployment data from testnodes

ad11bdf

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

chore: update README.md

e9cab36

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix(e2e): allow skipping bridge tests

7eb7ba2

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix(e2e): update github action to multi layer e2e tests

1436a0c

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix(e2e): bump gas limit for l2 scenario txs

d9977b5

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

chore(e2e): run nitro addressbook setup only on localnitro networks

6a9a999

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix: allow e2e to run only on L2

9729f43

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix(e2e): use fixed gasLimit for CI send to L2

85c2964

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix(e2e): use fixed gasLimit for redeem in CI

ee8c7f3

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

e2e: add bridge e2e tests (#720)

967e085

* feat(e2e): run L1 and L2 together in e2e tests Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix: gateways now unpaused on e2e

ae1d70b

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

Merge branch 'dev' into l2-public-testnet

60afb9e

chore: fix automerge wrong import

b1be78b

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

fix: remove unused task folder

ccaf653

fix: wrong imports and params

dff1845

fix: additional post merge fixes

5f1a3c4

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

tmigone and others added 5 commits November 3, 2022 16:03

fix: update nitro task for nitro v2.0.8

32fc123

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

chore: add deployed addresses for L2 public testnet

5daa3c1

fix: minor tweaks to e2e

fb40819

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

v1.17.0

3605081

- feat(gre): add hardhat-secure-accounts to GRE (#696) - feat: add publish and mint to the CLI - feat: refactor GRE to support L1 and L2 simultaneously - fix: prevent underflow when subgraphs go under the minimum signal threshold

chore: bump npm package version to v2.0.0-testnet

1dd6f06

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>

pcarranzav added 8 commits November 23, 2022 14:55

Merge branch 'pcv/l2-bridge' into l2-public-testnet

93f6939

fix(e2e): no need to explicitly disable secure accounts

dcc2cec

Merge remote-tracking branch 'origin/dev' into l2-public-testnet

f45173c

chore: bump @arbitrum/sdk to 3.0.0

12e54e5

chore: bump package version to 2.0.0

43e719b

fix: set flat bonding curve in all L2 configs

4e2872b

test(e2e): reenable finalizeInboundTransfer revert check

51c0441

fix: use correct governance addresses on Arbitrum One

aa85104

abarmat approved these changes Nov 23, 2022

View reviewed changes

pcarranzav merged commit d35acb6 into pcv/l2-bridge Nov 23, 2022

pcarranzav deleted the l2-public-testnet branch November 23, 2022 22:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

L2 public testnet deployment and related fixes#751

L2 public testnet deployment and related fixes#751
pcarranzav merged 43 commits intopcv/l2-bridgefrom
l2-public-testnet

pcarranzav commented Nov 23, 2022

Uh oh!

codecov bot commented Nov 23, 2022 •

edited

Loading

Uh oh!

abarmat left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

pcarranzav commented Nov 23, 2022

Uh oh!

codecov bot commented Nov 23, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

abarmat left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

codecov bot commented Nov 23, 2022 •

edited

Loading