fix(contracts): update mock to also create subscription in constructor

zmalatrax · zmalatrax · commit 98569412cc77 · 2023-06-21T08:40:18.000+01:00
diff --git a/contracts/src/rng/mock/VRFSubscriptionManagerV2Mock.sol b/contracts/src/rng/mock/VRFSubscriptionManagerV2Mock.sol
@@ -60,6 +60,7 @@ contract VRFSubscriptionManagerV2Mock {
     constructor(address _governor, address _vrfCoordinator) {
         vrfCoordinator = VRFCoordinatorV2InterfaceMock(_vrfCoordinator);
         governor = _governor;
+        createNewSubscription();
     }
 
     // ************************************* //
@@ -81,7 +82,7 @@ contract VRFSubscriptionManagerV2Mock {
     /**
      * @dev Creates a new subscription, overriding the previous one to be manageable by the contract.
      */
-    function createNewSubscription() external onlyByGovernor {
+    function createNewSubscription() public onlyByGovernor {
         subscriptionId = vrfCoordinator.createSubscription();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -60,6 +60,7 @@ contract VRFSubscriptionManagerV2Mock {`
`60`	`60`	`constructor(address _governor, address _vrfCoordinator) {`
`61`	`61`	`vrfCoordinator = VRFCoordinatorV2InterfaceMock(_vrfCoordinator);`
`62`	`62`	`governor = _governor;`
	`63`	`+ createNewSubscription();`
`63`	`64`	`}`
`64`	`65`
`65`	`66`	`// ************************************* //`
`@@ -81,7 +82,7 @@ contract VRFSubscriptionManagerV2Mock {`
`81`	`82`	`/**`
`82`	`83`	`* @dev Creates a new subscription, overriding the previous one to be manageable by the contract.`
`83`	`84`	`*/`
`84`		`- function createNewSubscription() external onlyByGovernor {`
	`85`	`+ function createNewSubscription() public onlyByGovernor {`
`85`	`86`	`subscriptionId = vrfCoordinator.createSubscription();`
`86`	`87`	`}`
`87`	`88`