Increase the challenge timeout period

contracts/HATVaultsRegistry.sol

@@ -137,7 +137,7 @@ contract HATVaultsRegistry is IHATVaultsRegistry, Ownable {
         defaultBountyHackerHATVested = _bountyHackerHATVested;
         defaultArbitrator = _defaultArbitrator;
         defaultChallengePeriod = 3 days;
-        defaultChallengeTimeOutPeriod = 5 weeks;
+        defaultChallengeTimeOutPeriod = 125 days;
         emit RegistryCreated(
             _hatVaultImplementation,
             _hatClaimsManagerImplementation,
@@ -420,7 +420,7 @@ contract HATVaultsRegistry is IHATVaultsRegistry, Ownable {
     /** @notice See {IHATVaultsRegistry-validateChallengeTimeOutPeriod}. */
     function validateChallengeTimeOutPeriod(uint32 _challengeTimeOutPeriod) public pure {
         if (_challengeTimeOutPeriod < 2 days) revert ChallengeTimeOutPeriodTooShort();
-        if (_challengeTimeOutPeriod > 85 days) revert ChallengeTimeOutPeriodTooLong();
+        if (_challengeTimeOutPeriod > 125 days) revert ChallengeTimeOutPeriodTooLong();
     }
 
     /**

contracts/interfaces/IHATVaultsRegistry.sol

@@ -154,7 +154,7 @@ interface IHATVaultsRegistry {
 
     /**
      * @notice Raised on {setDefaultChallengeTimeOutPeriod} if the challenge
-     * timeout period to be set is longer than 85 days
+     * timeout period to be set is longer than 125 days
      */
     error ChallengeTimeOutPeriodTooLong();
 
@@ -426,7 +426,7 @@ interface IHATVaultsRegistry {
 
     /**
      * @notice Check that the given challenge timeout period is legal, meaning
-     * that it is greater than 2 days and less than 85 days.
+     * that it is greater than 2 days and less than 125 days.
      * @param _challengeTimeOutPeriod The challenge timeout period to check
      */
     function validateChallengeTimeOutPeriod(uint32 _challengeTimeOutPeriod) external pure;

docs/dodoc/HATVaultsRegistry.md

@@ -1249,7 +1249,7 @@ Raised on {setDefaultChallengePeriod} if the challenge period  to be set is shor
 error ChallengeTimeOutPeriodTooLong()
 ```
 
-Raised on {setDefaultChallengeTimeOutPeriod} if the challenge timeout period to be set is longer than 85 days
+Raised on {setDefaultChallengeTimeOutPeriod} if the challenge timeout period to be set is longer than 125 days
 
 
 

docs/dodoc/interfaces/IHATVaultsRegistry.md

@@ -579,7 +579,7 @@ Check that the given challenge period is legal, meaning that it is greater than
 function validateChallengeTimeOutPeriod(uint32 _challengeTimeOutPeriod) external pure
 ```
 
-Check that the given challenge timeout period is legal, meaning that it is greater than 2 days and less than 85 days.
+Check that the given challenge timeout period is legal, meaning that it is greater than 2 days and less than 125 days.
 
 
 
@@ -990,7 +990,7 @@ Raised on {setDefaultChallengePeriod} if the challenge period  to be set is shor
 error ChallengeTimeOutPeriodTooLong()
 ```
 
-Raised on {setDefaultChallengeTimeOutPeriod} if the challenge timeout period to be set is longer than 85 days
+Raised on {setDefaultChallengeTimeOutPeriod} if the challenge timeout period to be set is longer than 125 days
 
 
 

test/arbitrator.js

@@ -73,8 +73,8 @@ contract("Registry Arbitrator", (accounts) => {
   it("Set default challengeTimeOutPeriod", async () => {
     const { registry, claimsManager } = await setup(accounts);
 
-    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 35);
-    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 35);
+    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
+    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
 
     await assertFunctionRaisesException(
       registry.setDefaultChallengeTimeOutPeriod(60 * 60 * 24 * 2, { from: accounts[1] }),
@@ -87,7 +87,7 @@ contract("Registry Arbitrator", (accounts) => {
     );
 
     await assertFunctionRaisesException(
-      registry.setDefaultChallengeTimeOutPeriod(60 * 60 * 24 * 85 + 1),
+      registry.setDefaultChallengeTimeOutPeriod(60 * 60 * 24 * 125 + 1),
       "ChallengeTimeOutPeriodTooLong"
     );
 
@@ -98,12 +98,12 @@ contract("Registry Arbitrator", (accounts) => {
     assert.equal(tx.logs[0].event, "SetDefaultChallengeTimeOutPeriod");
     assert.equal(tx.logs[0].args._defaultChallengeTimeOutPeriod, 60 * 60 * 24 * 2);
 
-    tx = await registry.setDefaultChallengeTimeOutPeriod(60 * 60 * 24 * 85);
+    tx = await registry.setDefaultChallengeTimeOutPeriod(60 * 60 * 24 * 125);
 
-    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 85);
-    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 85);
+    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
+    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
     assert.equal(tx.logs[0].event, "SetDefaultChallengeTimeOutPeriod");
-    assert.equal(tx.logs[0].args._defaultChallengeTimeOutPeriod, 60 * 60 * 24 * 85);
+    assert.equal(tx.logs[0].args._defaultChallengeTimeOutPeriod, 60 * 60 * 24 * 125);
   });
 
   it("Set vault arbitration parameters", async () => {
@@ -117,8 +117,8 @@ contract("Registry Arbitrator", (accounts) => {
     assert.equal((await registry.defaultChallengePeriod()).toString(), 60 * 60 * 24 * 3);
     assert.equal(await claimsManager.getChallengePeriod(), 60 * 60 * 24 * 3);
 
-    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 35);
-    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 35);
+    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
+    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
 
     assert.equal(await claimsManager.arbitratorCanChangeBounty(), true);
     assert.equal(await claimsManager.arbitratorCanChangeBeneficiary(), false);
@@ -172,7 +172,7 @@ contract("Registry Arbitrator", (accounts) => {
     );
 
     await assertFunctionRaisesException(
-      claimsManager.setChallengeTimeOutPeriod(60 * 60 * 24 * 85 + 1),
+      claimsManager.setChallengeTimeOutPeriod(60 * 60 * 24 * 125 + 1),
       "ChallengeTimeOutPeriodTooLong"
     );
 
@@ -200,7 +200,7 @@ contract("Registry Arbitrator", (accounts) => {
     assert.equal(await registry.defaultChallengePeriod(), 60 * 60 * 24 * 3);
     assert.equal(await claimsManager.getChallengePeriod(), 60 * 60 * 24);
 
-    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 35);
+    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
     assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 2);
 
     assert.equal(await claimsManager.arbitratorCanChangeBounty(), false);
@@ -217,9 +217,9 @@ contract("Registry Arbitrator", (accounts) => {
     assert.equal(tx.logs[0].event, "SetChallengePeriod");
     assert.equal(tx.logs[0].args._challengePeriod, 60 * 60 * 24 * 5);
 
-    tx = await claimsManager.setChallengeTimeOutPeriod(60 * 60 * 24 * 85);
+    tx = await claimsManager.setChallengeTimeOutPeriod(60 * 60 * 24 * 125);
     assert.equal(tx.logs[0].event, "SetChallengeTimeOutPeriod");
-    assert.equal(tx.logs[0].args._challengeTimeOutPeriod, 60 * 60 * 24 * 85);
+    assert.equal(tx.logs[0].args._challengeTimeOutPeriod, 60 * 60 * 24 * 125);
 
     tx = await claimsManager.setArbitratorOptions(true, false, false);
     assert.equal(tx.logs[0].event, "SetArbitratorOptions");
@@ -233,8 +233,8 @@ contract("Registry Arbitrator", (accounts) => {
     assert.equal(await registry.defaultChallengePeriod(), 60 * 60 * 24 * 3);
     assert.equal(await claimsManager.getChallengePeriod(), 60 * 60 * 24 * 5);
 
-    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 35);
-    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 85);
+    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
+    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
 
     assert.equal(await claimsManager.arbitratorCanChangeBounty(), true);
 
@@ -266,8 +266,8 @@ contract("Registry Arbitrator", (accounts) => {
     assert.equal(await registry.defaultChallengePeriod(), 60 * 60 * 24 * 3);
     assert.equal(await claimsManager.getChallengePeriod(), 60 * 60 * 24 * 3);
 
-    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 35);
-    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 35);
+    assert.equal(await registry.defaultChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
+    assert.equal(await claimsManager.getChallengeTimeOutPeriod(), 60 * 60 * 24 * 125);
 
     assert.equal(await claimsManager.arbitratorCanChangeBounty(), true);
 

test/hatvaults.js

@@ -391,7 +391,7 @@ contract("HatVaults", (accounts) => {
     assert.equal(activeClaim.bountyHackerHATVested, "500");
     assert.equal(activeClaim.arbitrator, accounts[0]);
     assert.equal(activeClaim.challengePeriod, "86400");
-    assert.equal(activeClaim.challengeTimeOutPeriod, "3024000");
+    assert.equal(activeClaim.challengeTimeOutPeriod, "10800000");
     assert.equal(activeClaim.arbitratorCanChangeBounty, true);
     assert.equal(activeClaim.arbitratorCanChangeBeneficiary, false);
   });
@@ -981,7 +981,7 @@ contract("HatVaults", (accounts) => {
     await newClaimsManager.setCommittee(accounts[1], { from: accounts[2] });
   });
 
-  it("dismiss can be called by anyone after 5 weeks delay", async () => {
+  it("dismiss can be called by anyone after 125 days delay", async () => {
     var staker = accounts[1];
     await setUpGlobalVars(accounts, 0, 9000, [9000, 0, 1000], [1000, 500], 10, 0, 100, false, 2500000, 60 * 60 * 24 * 3);
 
@@ -1013,7 +1013,7 @@ contract("HatVaults", (accounts) => {
       );
     }
     await utils.increaseTime(1);
-    await utils.increaseTime(5 * 7 * 24 * 60 * 60);
+    await utils.increaseTime(125 * 24 * 60 * 60);
     tx = await claimsManager.dismissClaim(claimId, { from: accounts[1] });
     assert.equal(tx.logs[0].event, "DismissClaim");
   });