fix: update failing tests for HorizonStakingNotAuthorized (OZ_N-05)

graphprotocol · Oct 4, 2024 · 17cfefd · 17cfefd
1 parent 00a36dd
commit 17cfefd
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 12 deletions.
diff --git a/packages/horizon/test/staking/provision/deprovision.t.sol b/packages/horizon/test/staking/provision/deprovision.t.sol
@@ -75,9 +75,9 @@ contract HorizonStakingDeprovisionTest is HorizonStakingTest {
         vm.startPrank(users.operator);
         bytes memory expectedError = abi.encodeWithSignature(
             "HorizonStakingNotAuthorized(address,address,address)",
-            users.indexer,
+            subgraphDataServiceAddress,
             users.operator,
-            subgraphDataServiceAddress
+            users.indexer
         );
         vm.expectRevert(expectedError);
         staking.deprovision(users.indexer, subgraphDataServiceAddress, 0);

diff --git a/packages/horizon/test/staking/provision/locked.t.sol b/packages/horizon/test/staking/provision/locked.t.sol
@@ -66,9 +66,9 @@ contract HorizonStakingProvisionLockedTest is HorizonStakingTest {
         vm.startPrank(users.operator);
         bytes memory expectedError = abi.encodeWithSignature(
             "HorizonStakingNotAuthorized(address,address,address)",
-            users.indexer,
+            subgraphDataServiceAddress,
             users.operator,
-            subgraphDataServiceAddress
+            users.indexer
         );
         vm.expectRevert(expectedError);
         staking.provisionLocked(

diff --git a/packages/horizon/test/staking/provision/parameters.t.sol b/packages/horizon/test/staking/provision/parameters.t.sol
@@ -42,9 +42,9 @@ contract HorizonStakingProvisionParametersTest is HorizonStakingTest {
         vm.expectRevert(
             abi.encodeWithSignature(
                 "HorizonStakingNotAuthorized(address,address,address)",
-                users.indexer,
+                subgraphDataServiceAddress,
                 msg.sender,
-                subgraphDataServiceAddress
+                users.indexer
             )
         );
         staking.setProvisionParameters(users.indexer, subgraphDataServiceAddress, maxVerifierCut, thawingPeriod);

diff --git a/packages/horizon/test/staking/provision/provision.t.sol b/packages/horizon/test/staking/provision/provision.t.sol
@@ -102,9 +102,9 @@ contract HorizonStakingProvisionTest is HorizonStakingTest {
         vm.startPrank(users.operator);
         bytes memory expectedError = abi.encodeWithSignature(
             "HorizonStakingNotAuthorized(address,address,address)",
-            users.indexer,
+            subgraphDataServiceAddress,
             users.operator,
-            subgraphDataServiceAddress
+            users.indexer,
         );
         vm.expectRevert(expectedError);
         staking.provision(users.indexer, subgraphDataServiceAddress, amount, maxVerifierCut, thawingPeriod);

diff --git a/packages/horizon/test/staking/provision/reprovision.t.sol b/packages/horizon/test/staking/provision/reprovision.t.sol
@@ -80,9 +80,9 @@ contract HorizonStakingReprovisionTest is HorizonStakingTest {
         vm.startPrank(users.operator);
         bytes memory expectedError = abi.encodeWithSignature(
             "HorizonStakingNotAuthorized(address,address,address)",
-            users.indexer,
+            newDataService,
             users.operator,
-            newDataService
+            users.indexer
         );
         vm.expectRevert(expectedError);
         staking.reprovision(users.indexer, subgraphDataServiceAddress, newDataService, provisionAmount, 0);

diff --git a/packages/horizon/test/staking/provision/thaw.t.sol b/packages/horizon/test/staking/provision/thaw.t.sol
@@ -49,9 +49,9 @@ contract HorizonStakingThawTest is HorizonStakingTest {
         vm.startPrank(users.operator);
         bytes memory expectedError = abi.encodeWithSignature(
             "HorizonStakingNotAuthorized(address,address,address)",
-            users.indexer,
+            subgraphDataServiceAddress,
             users.operator,
-            subgraphDataServiceAddress
+            users.indexer
         );
         vm.expectRevert(expectedError);
         staking.thaw(users.indexer, subgraphDataServiceAddress, amount);