fix: apply consistent mapping order ProvisionManagerNotAuthorized (OZ…

…_N-05)
graphprotocol · Oct 3, 2024 · 6265da0 · 6265da0
1 parent 0256641
commit 6265da0
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 15 deletions.
diff --git a/packages/subgraph-service/test/subgraphService/allocation/start.t.sol b/packages/subgraph-service/test/subgraphService/allocation/start.t.sol
@@ -60,9 +60,10 @@ contract SubgraphServiceAllocationStartTest is SubgraphServiceTest {
         bytes memory data = _generateData(tokens);
         vm.expectRevert(abi.encodeWithSelector(
             ProvisionManager.ProvisionManagerNotAuthorized.selector,
-            users.operator,
-            users.indexer
-        ));
+            users.indexer,
+            users.operator
+            )
+        );
         subgraphService.startService(users.indexer, data);
     }
 

diff --git a/packages/subgraph-service/test/subgraphService/allocation/stop.t.sol b/packages/subgraph-service/test/subgraphService/allocation/stop.t.sol
@@ -48,9 +48,9 @@ contract SubgraphServiceAllocationStopTest is SubgraphServiceTest {
         bytes memory data = abi.encode(allocationID);
         vm.expectRevert(
             abi.encodeWithSelector(
-                ProvisionManager.ProvisionManagerNotAuthorized.selector,
-                users.operator,
-                users.indexer
+            ProvisionManager.ProvisionManagerNotAuthorized.selector,
+            users.indexer,
+            users.operator
             )
         );
         subgraphService.stopService(users.indexer, data);

diff --git a/packages/subgraph-service/test/subgraphService/collect/query/query.t.sol b/packages/subgraph-service/test/subgraphService/collect/query/query.t.sol
@@ -111,9 +111,9 @@ contract SubgraphServiceRegisterTest is SubgraphServiceTest {
         resetPrank(users.operator);
         vm.expectRevert(
             abi.encodeWithSelector(
-                ProvisionManager.ProvisionManagerNotAuthorized.selector,
-                users.operator,
-                users.indexer
+            ProvisionManager.ProvisionManagerNotAuthorized.selector,
+            users.indexer,
+            users.operator
             )
         );
         subgraphService.collect(users.indexer, paymentType, data);

diff --git a/packages/subgraph-service/test/subgraphService/provider/register.t.sol b/packages/subgraph-service/test/subgraphService/provider/register.t.sol
@@ -41,9 +41,10 @@ contract SubgraphServiceProviderRegisterTest is SubgraphServiceTest {
         resetPrank(users.operator);
         vm.expectRevert(abi.encodeWithSelector(
             ProvisionManager.ProvisionManagerNotAuthorized.selector,
-            users.operator,
-            users.indexer
-        ));
+            users.indexer,
+            users.operator
+            )
+        );
         bytes memory data = abi.encode("url", "geoHash", users.rewardsDestination);
         subgraphService.register(users.indexer, data);
     }

diff --git a/packages/subgraph-service/test/subgraphService/provision/accept.t.sol b/packages/subgraph-service/test/subgraphService/provision/accept.t.sol
@@ -47,9 +47,10 @@ contract SubgraphServiceProvisionAcceptTest is SubgraphServiceTest {
         resetPrank(users.operator);
         vm.expectRevert(abi.encodeWithSelector(
             ProvisionManager.ProvisionManagerNotAuthorized.selector,
-            users.operator,
-            users.indexer
-        ));
+            users.indexer,
+            users.operator
+            )
+        );
         subgraphService.acceptProvision(users.indexer, "");
     }