Allow a timelock delay of zero

Author: maurelian

packages/contracts-bedrock/src/safe/TimelockGuard.sol

@@ -67,7 +67,7 @@ contract TimelockGuard is IGuard, ISemver {
     /// @param _timelockDelay The timelock delay in seconds
     function configureTimelockGuard(uint256 _timelockDelay) external {
         // Validate timelock delay - must be non-zero and not longer than 1 year
-        if (_timelockDelay > 365 days) {
+        if (_timelockDelay == 0 || _timelockDelay > 365 days) {
             revert TimelockGuard_InvalidTimelockDelay();
         }
 

packages/contracts-bedrock/test/safe/TimelockGuard.t.sol

@@ -120,12 +120,6 @@ contract TimelockGuard_ConfigureTimelockGuard_Test is TimelockGuard_TestInit {
         timelockGuard.configureTimelockGuard(tooLongDelay);
     }
 
-    function test_configureTimelockGuard_revertsIfDelayZero_reverts() external {
-        vm.expectRevert(TimelockGuard.TimelockGuard_InvalidTimelockDelay.selector);
-        vm.prank(address(safeInstance.safe));
-        timelockGuard.configureTimelockGuard(0);
-    }
-
     function test_configureTimelockGuard_acceptsMaxValidDelay_succeeds() external {
         vm.expectEmit(true, true, true, true);
         emit GuardConfigured(address(safeInstance.safe), ONE_YEAR);