chore: use solidity 0.8.27

Signed-off-by: Tomás Migone <[email protected]>

packages/subgraph-service/foundry.toml

@@ -4,14 +4,6 @@ out = 'build'
 libs = ['node_modules', 'lib']
 test = 'test'
 cache_path  = 'cache_forge'
+fs_permissions = [{ access = "read", path = "./"}]
 optimizer = true
 optimizer-runs = 200
-via_ir = true
-fs_permissions = [{ access = "read", path = "./"}]
-
-[profile.lite]
-optimizer = false
-optimizer-runs = 1
-
-[profile.lite.optimizer_details.yulDetails]
-optimizerSteps = ':'

packages/subgraph-service/package.json

@@ -10,7 +10,7 @@
     "lint": "yarn lint:ts && yarn lint:sol",
     "clean": "rm -rf build cache typechain-types",
     "build": "forge build && hardhat compile",
-    "test": "FOUNDRY_PROFILE=lite forge test -vvvv && hardhat test"
+    "test": "forge test && hardhat test"
   },
   "devDependencies": {
     "@graphprotocol/contracts": "workspace:^7.0.0",

packages/subgraph-service/test/disputeManager/DisputeManager.t.sol

@@ -166,54 +166,67 @@ contract DisputeManagerTest is SubgraphServiceSharedTest {
         return _disputeID;
     }
 
+    struct BeforeValues_CreateQueryDisputeConflict {
+        Attestation.State attestation1;
+        Attestation.State attestation2;
+        address indexer1;
+        address indexer2;
+        uint256 stakeSnapshot1;
+        uint256 stakeSnapshot2;
+    }
     function _createQueryDisputeConflict(
         bytes memory attestationData1,
         bytes memory attestationData2
     ) internal returns (bytes32, bytes32) {
         (, address fisherman,) = vm.readCallers();
-        Attestation.State memory attestation1 = Attestation.parse(attestationData1);
-        Attestation.State memory attestation2 = Attestation.parse(attestationData2);
-        address indexer1 = disputeManager.getAttestationIndexer(attestation1);
-        address indexer2 = disputeManager.getAttestationIndexer(attestation2);
+
+        BeforeValues_CreateQueryDisputeConflict memory beforeValues;
+        beforeValues.attestation1 = Attestation.parse(attestationData1);
+        beforeValues.attestation2 = Attestation.parse(attestationData2);
+        beforeValues.indexer1 = disputeManager.getAttestationIndexer(beforeValues.attestation1);
+        beforeValues.indexer2 = disputeManager.getAttestationIndexer(beforeValues.attestation2);
+        beforeValues.stakeSnapshot1 = disputeManager.getStakeSnapshot(beforeValues.indexer1);
+        beforeValues.stakeSnapshot2 = disputeManager.getStakeSnapshot(beforeValues.indexer2);
+
         bytes32 expectedDisputeId1 = keccak256(
             abi.encodePacked(
-                attestation1.requestCID,
-                attestation1.responseCID,
-                attestation1.subgraphDeploymentId,
-                indexer1,
+                beforeValues.attestation1.requestCID,
+                beforeValues.attestation1.responseCID,
+                beforeValues.attestation1.subgraphDeploymentId,
+                beforeValues.indexer1,
                 fisherman
             )
         );
         bytes32 expectedDisputeId2 = keccak256(
             abi.encodePacked(
-                attestation2.requestCID,
-                attestation2.responseCID,
-                attestation2.subgraphDeploymentId,
-                indexer2,
+                beforeValues.attestation2.requestCID,
+                beforeValues.attestation2.responseCID,
+                beforeValues.attestation2.subgraphDeploymentId,
+                beforeValues.indexer2,
                 fisherman
             )
         );
-        uint256 stakeSnapshot1 = disputeManager.getStakeSnapshot(indexer1);
-        uint256 stakeSnapshot2 = disputeManager.getStakeSnapshot(indexer2);
 
+        // createQueryDisputeConflict
         vm.expectEmit(address(disputeManager));
         emit IDisputeManager.QueryDisputeCreated(
             expectedDisputeId1,
-            indexer1,
+            beforeValues.indexer1,
             fisherman,
             0,
-            attestation1.subgraphDeploymentId,
+            beforeValues.attestation1.subgraphDeploymentId,
             attestationData1,
-            stakeSnapshot1
+            beforeValues.stakeSnapshot1
         );
+        vm.expectEmit(address(disputeManager));
         emit IDisputeManager.QueryDisputeCreated(
             expectedDisputeId2,
-            indexer2,
+            beforeValues.indexer2,
             fisherman,
             0,
-            attestation2.subgraphDeploymentId,
+            beforeValues.attestation2.subgraphDeploymentId,
             attestationData2,
-            stakeSnapshot2
+            beforeValues.stakeSnapshot2
         );
 
         (bytes32 _disputeId1, bytes32 _disputeId2) = disputeManager.createQueryDisputeConflict(attestationData1, attestationData2);
@@ -226,24 +239,24 @@ contract DisputeManagerTest is SubgraphServiceSharedTest {
 
         // Check dispute values
         IDisputeManager.Dispute memory dispute1 = _getDispute(_disputeId1);
-        assertEq(dispute1.indexer, indexer1, "Indexer 1 should match");
+        assertEq(dispute1.indexer, beforeValues.indexer1, "Indexer 1 should match");
         assertEq(dispute1.fisherman, fisherman, "Fisherman 1 should match");
         assertEq(dispute1.deposit, 0, "Deposit 1 should match");
         assertEq(dispute1.relatedDisputeId, _disputeId2, "Related dispute ID 1 should be the id of the other dispute");
         assertEq(uint8(dispute1.disputeType), uint8(IDisputeManager.DisputeType.QueryDispute), "Dispute type 1 should be query");
         assertEq(uint8(dispute1.status), uint8(IDisputeManager.DisputeStatus.Pending), "Dispute status 1 should be pending");
         assertEq(dispute1.createdAt, block.timestamp, "Created at 1 should match");
-        assertEq(dispute1.stakeSnapshot, stakeSnapshot1, "Stake snapshot 1 should match");
+        assertEq(dispute1.stakeSnapshot, beforeValues.stakeSnapshot1, "Stake snapshot 1 should match");
 
         IDisputeManager.Dispute memory dispute2 = _getDispute(_disputeId2);
-        assertEq(dispute2.indexer, indexer2, "Indexer 2 should match");
+        assertEq(dispute2.indexer, beforeValues.indexer2, "Indexer 2 should match");
         assertEq(dispute2.fisherman, fisherman, "Fisherman 2 should match");
         assertEq(dispute2.deposit, 0, "Deposit 2 should match");
         assertEq(dispute2.relatedDisputeId, _disputeId1, "Related dispute ID 2 should be the id of the other dispute");
         assertEq(uint8(dispute2.disputeType), uint8(IDisputeManager.DisputeType.QueryDispute), "Dispute type 2 should be query");
         assertEq(uint8(dispute2.status), uint8(IDisputeManager.DisputeStatus.Pending), "Dispute status 2 should be pending");
         assertEq(dispute2.createdAt, block.timestamp, "Created at 2 should match");
-        assertEq(dispute2.stakeSnapshot, stakeSnapshot2, "Stake snapshot 2 should match");
+        assertEq(dispute2.stakeSnapshot, beforeValues.stakeSnapshot2, "Stake snapshot 2 should match");
 
         return (_disputeId1, _disputeId2);
     }

packages/subgraph-service/test/disputeManager/disputes/disputes.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/indexing/accept.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/indexing/cancel.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/indexing/create.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/indexing/draw.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/indexing/reject.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/query/accept.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/query/cancel.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/query/create.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/query/draw.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/query/reject.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/queryConflict/accept.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/queryConflict/cancel.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/queryConflict/create.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/queryConflict/draw.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/disputes/queryConflict/reject.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/governance/arbitrator.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/governance/disputeDeposit.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/disputeManager/governance/maxSlashingCut.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";
 

packages/subgraph-service/test/shared/HorizonStakingShared.t.sol

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-pragma solidity 0.8.26;
+pragma solidity 0.8.27;
 
 import "forge-std/Test.sol";