chore: hardhat bump, draw test edge case

Author: jaybuidl

contracts/package.json

@@ -129,14 +129,14 @@
     "gluegun": "^5.2.0",
     "graphql": "^16.9.0",
     "graphql-request": "^7.1.2",
-    "hardhat": "2.26.2",
-    "hardhat-contract-sizer": "^2.10.0",
+    "hardhat": "2.26.3",
+    "hardhat-contract-sizer": "^2.10.1",
     "hardhat-deploy": "^1.0.4",
     "hardhat-deploy-ethers": "^0.4.2",
     "hardhat-deploy-tenderly": "^0.2.1",
     "hardhat-docgen": "^1.3.0",
     "hardhat-gas-reporter": "^2.3.0",
-    "hardhat-tracer": "^3.2.1",
+    "hardhat-tracer": "^3.4.0",
     "hardhat-watcher": "^2.5.0",
     "pino": "^8.21.0",
     "pino-pretty": "^10.3.1",

contracts/test/foundry/KlerosCore_Drawing.t.sol

@@ -5,6 +5,7 @@ import {KlerosCore_TestBase} from "./KlerosCore_TestBase.sol";
 import {KlerosCore} from "../../src/arbitration/KlerosCore.sol";
 import {SortitionModule} from "../../src/arbitration/SortitionModule.sol";
 import {ISortitionModule} from "../../src/arbitration/interfaces/ISortitionModule.sol";
+import {Vm} from "forge-std/Vm.sol";
 import "../../src/libraries/Constants.sol";
 
 /// @title KlerosCore_DrawingTest
@@ -46,6 +47,22 @@ contract KlerosCore_DrawingTest is KlerosCore_TestBase {
             assertEq(choice, 0, "Choice should be empty");
             assertEq(voted, false, "Voted should be false");
         }
+
+        // Try drawing even more...
+        vm.recordLogs();
+        core.draw(disputeID, DEFAULT_NB_OF_JURORS); // Should not revert, nor draw any juror, nor change state.
+        Vm.Log[] memory entries = vm.getRecordedLogs();
+
+        // Verify no Draw events were emitted
+        bytes32 drawEventSignature = keccak256("Draw(address,uint256,uint256,uint256)");
+        for (uint256 i = 0; i < entries.length; i++) {
+            assertFalse(entries[i].topics[0] == drawEventSignature, "Draw event should not be emitted");
+        }
+
+        assertEq(totalStaked, 1500, "Wrong amount total staked");
+        assertEq(totalLocked, 3000, "Wrong amount locked");
+        assertEq(stakedInCourt, 1500, "Wrong amount staked in court");
+        assertEq(sortitionModule.disputesWithoutJurors(), 0, "Wrong disputesWithoutJurors count");
     }
 
     function test_draw_noEmptyAddresses() public {

yarn.lock

@@ -6625,14 +6625,14 @@ __metadata:
     gluegun: "npm:^5.2.0"
     graphql: "npm:^16.9.0"
     graphql-request: "npm:^7.1.2"
-    hardhat: "npm:2.26.2"
-    hardhat-contract-sizer: "npm:^2.10.0"
+    hardhat: "npm:2.26.3"
+    hardhat-contract-sizer: "npm:^2.10.1"
     hardhat-deploy: "npm:^1.0.4"
     hardhat-deploy-ethers: "npm:^0.4.2"
     hardhat-deploy-tenderly: "npm:^0.2.1"
     hardhat-docgen: "npm:^1.3.0"
     hardhat-gas-reporter: "npm:^2.3.0"
-    hardhat-tracer: "npm:^3.2.1"
+    hardhat-tracer: "npm:^3.4.0"
     hardhat-watcher: "npm:^2.5.0"
     isomorphic-fetch: "npm:^3.0.0"
     pino: "npm:^8.21.0"
@@ -22033,16 +22033,16 @@ __metadata:
   languageName: node
   linkType: hard
 
-"hardhat-contract-sizer@npm:^2.10.0":
-  version: 2.10.0
-  resolution: "hardhat-contract-sizer@npm:2.10.0"
+"hardhat-contract-sizer@npm:^2.10.1":
+  version: 2.10.1
+  resolution: "hardhat-contract-sizer@npm:2.10.1"
   dependencies:
     chalk: "npm:^4.0.0"
     cli-table3: "npm:^0.6.0"
     strip-ansi: "npm:^6.0.0"
   peerDependencies:
     hardhat: ^2.0.0
-  checksum: 10/2b3011fe5333c2f1dbcfa6c73dcb1c61da9e2e045775c24bb773fe5f3ac14e9923907fef0e61fc2897e82a61997ce74e73baadb7f69dfdeccc86ae878cf67792
+  checksum: 10/521e985f0e6b5c1d06e38be8b374ae30f26ed6a72740ecfeeea1d8d5ee1341b98f699ed5587cc429e6de84da2c22df612e394ce492d393a4922702e82cc051f9
   languageName: node
   linkType: hard
 
@@ -22143,9 +22143,9 @@ __metadata:
   languageName: node
   linkType: hard
 
-"hardhat-tracer@npm:^3.2.1":
-  version: 3.2.1
-  resolution: "hardhat-tracer@npm:3.2.1"
+"hardhat-tracer@npm:^3.4.0":
+  version: 3.4.0
+  resolution: "hardhat-tracer@npm:3.4.0"
   dependencies:
     chalk: "npm:^4.1.2"
     debug: "npm:^4.3.4"
@@ -22154,7 +22154,7 @@ __metadata:
   peerDependencies:
     chai: 4.x
     hardhat: ">=2.22.5 <3.x"
-  checksum: 10/963afeb42c1ed30704877c3b521afc748e60042d218cef333fc11d3246f544267c303df0effc1e2a5cac9cc8a9ec744fda6ceb4e3fc156d3e17f4cfb766f6ab1
+  checksum: 10/cdbab3bc60b73156866b5fdc27486ef75894094f67382dad28f0868ab167116787c61d9ab430a61ac49fb1787a47da9281fb421e8dc6fefc9dd4fa7660ad3f60
   languageName: node
   linkType: hard
 
@@ -22169,9 +22169,9 @@ __metadata:
   languageName: node
   linkType: hard
 
-"hardhat@npm:2.26.2":
-  version: 2.26.2
-  resolution: "hardhat@npm:2.26.2"
+"hardhat@npm:2.26.3":
+  version: 2.26.3
+  resolution: "hardhat@npm:2.26.3"
   dependencies:
     "@ethereumjs/util": "npm:^9.1.0"
     "@ethersproject/abi": "npm:^5.1.2"
@@ -22222,7 +22222,7 @@ __metadata:
       optional: true
   bin:
     hardhat: internal/cli/bootstrap.js
-  checksum: 10/ef9f5f232264ed45a406a7053ccce71e67b0ce084de2de6fa2c24ff0bb1ec0d7b69f61769b3ac94a50445709b25bcf0d8ee135e1509e53331a3fea44d01cbb63
+  checksum: 10/77fabed0a9f5a5871bd071cd1eae3ce120ee878370a0d58e1a762fd49d01b68385b955c111d4a55147b1540333712ef36796fd5ee142e504aad17e2b2f746d6d
   languageName: node
   linkType: hard