Parameterize MAX_ALLOWANCE in Clock lib

Author: guidanoli

prt/contracts/src/tournament/abstracts/Tournament.sol

@@ -357,8 +357,8 @@ abstract contract Tournament {
             Clock.State storage _firstClock = clocks[_danglingCommitment];
 
             // grant extra match effort for both clocks
-            _firstClock.addMatchEffort(matchEffort);
-            _newClock.addMatchEffort(matchEffort);
+            _firstClock.addMatchEffort(matchEffort, maxAllowance);
+            _newClock.addMatchEffort(matchEffort, maxAllowance);
 
             _firstClock.advanceClock();
 

prt/contracts/src/tournament/libs/Clock.sol

@@ -128,17 +128,15 @@ library Clock {
     }
 
     /// @notice Add matchEffort to a clock and set it to paused.
-    /// The new clock allowance is capped by `MAX_ALLOWANCE`.
-    function addMatchEffort(State storage state, Time.Duration matchEffort)
-        internal
-    {
+    /// The new clock allowance is capped by maxAllowance.
+    function addMatchEffort(
+        State storage state,
+        Time.Duration matchEffort,
+        Time.Duration maxAllowance
+    ) internal {
         Time.Duration _timeLeft = timeLeft(state);
 
-        Time.Duration _allowance = _timeLeft.add(matchEffort);
-
-        if (_allowance.gt(ArbitrationConstants.MAX_ALLOWANCE)) {
-            _allowance = ArbitrationConstants.MAX_ALLOWANCE;
-        }
+        Time.Duration _allowance = _timeLeft.add(matchEffort).min(maxAllowance);
 
         _setNewPaused(state, _allowance);
     }