Skip to content

Commit

Permalink
Chore/vrf 585 add ctf test vrfv2plus migration (#10450)
Browse files Browse the repository at this point in the history
* fix VRF V3 setup script

* VRF-585: add VRFV2Plus CTF test for migration

* VRF-585: add VRFV2Plus Coordinator with onMigration method

* VRF-585: reverting VRFV2PlusClient contract changes

* remove unnecessary deregister (#10487)

* VRF-585: adding new steps

* VRF-585: updating VRFConsumerBaseV2Plus.sol to have public coordinator; and VRFV2PlusLoadTestWithMetrics.sol to use that coordinator

* VRF-585: update

* set s_consumers in onMigration()

* VRF-585: updating test

* VRF-585: updating test

* VRF-585: updating test

* VRF-585: fixing solidity lint issues

* VRF-585: removing code from upgraded coordinator in order to resolve max code size exceeded issue

* VRF-585: changing MinimumConfirmations

* VRF-585: enable CL Client debug

* VRF-585: reusing env for the test

* VRF-585: disabling CL node http debug mode

* VRF-585: enabling back waitforevents for contract deployment

* VRF-585: add err checks

* VRF-585: add total balance verification

* VRF-585: refactoring test; updating Coordinator contract to update subIds array when migrating

* VRF-585: removing commented code

* VRF-585: reverting

---------

Co-authored-by: Makram <[email protected]>
Co-authored-by: jinhoonbang <[email protected]>
  • Loading branch information
3 people authored Sep 18, 2023
1 parent 78c8214 commit aadc467
Show file tree
Hide file tree
Showing 22 changed files with 5,602 additions and 231 deletions.
1 change: 1 addition & 0 deletions contracts/scripts/native_solc_compile_all_vrf
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ compileContract dev/vrf/libraries/VRFV2PlusClient.sol
compileContract dev/vrf/testhelpers/VRFCoordinatorV2Plus_V2Example.sol
compileContract dev/vrf/TrustedBlockhashStore.sol
compileContract dev/vrf/testhelpers/VRFV2PlusLoadTestWithMetrics.sol
compileContractAltOpts dev/vrf/testhelpers/VRFCoordinatorV2PlusUpgradedVersion.sol 5

# VRF V2 Wrapper
compileContract vrf/VRFV2Wrapper.sol
Expand Down
2 changes: 1 addition & 1 deletion contracts/src/v0.8/dev/vrf/VRFConsumerBaseV2Plus.sol
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ abstract contract VRFConsumerBaseV2Plus is IVRFMigratableConsumerV2Plus, Confirm
error OnlyOwnerOrCoordinator(address have, address owner, address coordinator);
error ZeroAddress();

IVRFMigratableCoordinatorV2Plus internal s_vrfCoordinator;
IVRFMigratableCoordinatorV2Plus public s_vrfCoordinator;

/**
* @param _vrfCoordinator address of VRFCoordinator contract
Expand Down
Loading

0 comments on commit aadc467

Please sign in to comment.