Merge pull request #64 from hats-finance/fix-36

Fix #36
hats-finance · Dec 4, 2023 · 929a3eb · 929a3eb
2 parents 19117c0 + ece4e19
commit 929a3eb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/contracts/HATArbitrator.sol b/contracts/HATArbitrator.sol
@@ -488,6 +488,7 @@ contract HATArbitrator is IHATArbitrator, Ownable {
         ];
 
         if (
+            submitClaimRequest.submittedAt == 0 ||
             block.timestamp <=
             submitClaimRequest.submittedAt + submitClaimRequestReviewPeriod
         ) {
@@ -497,7 +498,7 @@ contract HATArbitrator is IHATArbitrator, Ownable {
         delete submitClaimRequests[_internalClaimId];
         token.safeTransfer(
             submitClaimRequest.submitter,
-            bondsNeededToStartDispute
+            submitClaimRequest.bond
         );
 
         emit SubmitClaimRequestExpired(_internalClaimId);

diff --git a/test/hatarbitrator.js b/test/hatarbitrator.js
@@ -1084,5 +1084,10 @@ contract("Registry Arbitrator", (accounts) => {
     assert.equal(await token.balanceOf(accounts[0]), web3.utils.toWei("1000"));
     assert.equal(await token.balanceOf(hatArbitrator.address), web3.utils.toWei("0"));
     assert.equal(await token.balanceOf(expertCommittee), web3.utils.toWei("0"));
+
+    await assertFunctionRaisesException(
+      hatArbitrator.refundExpiredSubmitClaimRequest(web3.utils.randomHex(32), { from: accounts[1] }),
+      "ClaimReviewPeriodDidNotEnd"
+    );
   });
 });