Fix #36

hats-finance · Nov 7, 2023 · ece4e19 · ece4e19
1 parent f2a4d00
commit ece4e19
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
@@ -482,6 +482,7 @@ contract HATArbitrator is IHATArbitrator, Ownable {
         ];
 
         if (
+            submitClaimRequest.submittedAt == 0 ||
             block.timestamp <=
             submitClaimRequest.submittedAt + submitClaimRequestReviewPeriod
         ) {
@@ -491,7 +492,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
@@ -1079,5 +1079,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"
+    );
   });
 });