contracts icon indicating copy to clipboard operation
contracts copied to clipboard

scenario1: implement testnet validation step 1.2

Open tmigone opened this issue 1 year ago • 1 comments

This PR implements step 1.2 on the L2 validation plan with a script plus a test file that can be run to assert the scenario.

This scenario sets up and tests the following:

  • two indexers with stake
  • each indexer has two or three allocations, one of which is open
  • four subgraphs, with 0, 400, 10000 and 12500 tokens signalled, with signal coming from three different curators

tmigone avatar Aug 04 '22 09:08 tmigone

Codecov Report

Merging #670 (9395f65) into l2-testnet (3a22a27) will not change coverage. The diff coverage is n/a.

@@             Coverage Diff             @@
##           l2-testnet     #670   +/-   ##
===========================================
  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 04 '22 10:08 codecov[bot]

@abarmat I'll work on addressing your comments but I think you didn't review the latest commit, I think you were on 6087247 (see https://github.com/graphprotocol/contracts/pull/670/commits). I only noticed because there are two later commits that refactor the scenario1 into more generic and re-usable parts.

tmigone avatar Aug 23 '22 12:08 tmigone

@abarmat, I'll reply to your comments now. Note that you didn't review the latest commit but an intermediate one. Some of your comments were addressed by later commits.

In any case, since this PR is L1 exclusive and not using any of the new L1 features (reservoir, gateway) I'm cherry picking these commits into dev on a separate PR so I'll close this one.

tmigone avatar Sep 01 '22 15:09 tmigone

Commits from this PR have been cherry picked into https://github.com/graphprotocol/contracts/pull/703 Closing PR.

tmigone avatar Sep 02 '22 10:09 tmigone