Minor changes

Author: sunbreak1211

deploy/PASInit.sol

@@ -141,6 +141,8 @@ library PASInit {
         address[] memory rateLimits,
         address[] memory controllers
     ) internal {
+        require(hop > 0, "PASInit/hop-is-zero");
+
         BeamStateLike beamState = BeamStateLike(pasInstance.beamState);
 
         beamState.setHop(address(0), hop);

src/BeamState.sol

@@ -178,7 +178,7 @@ contract BeamState {
     }
 
     function setMaxChange(address rateLimits_, uint256 value) external roleAuth {
-        require(value >= WAD, "BeamState/maxChange-below-1x");
+        require(value == 0 || value >= WAD, "BeamState/maxChange-below-1x");
         maxChange[rateLimits_] = value;
         emit SetMaxChange(rateLimits_, value);
     }

test/BeamState.t.sol

@@ -316,6 +316,15 @@ contract BeamStateTest is DssTest {
         assertEq(beamState.maxChange(TARGET1), WAD, "maxChange should be WAD");
     }
 
+    function testSetMaxChangeZero() public {
+        beamState.setMaxChange(TARGET1, 2 * WAD);
+        assertEq(beamState.maxChange(TARGET1), 2 * WAD, "maxChange should be set");
+
+        // Setting to 0 clears the specific override
+        beamState.setMaxChange(TARGET1, 0);
+        assertEq(beamState.maxChange(TARGET1), 0, "maxChange should be cleared");
+    }
+
     function testSetMaxChangeBelowWad() public {
         vm.expectRevert("BeamState/maxChange-below-1x");
         beamState.setMaxChange(TARGET1, WAD - 1);

test/Integration.t.sol

@@ -255,6 +255,16 @@ contract IntegrationTest is DssTest {
         assertEq(freshBeamState.controllers(testControllers[1]), 1, "second controller should be added");
     }
 
+    function initExtras() external {
+        PASInstance memory freshPas = PASDeploy.deploy(address(this), address(this), MIN_DELAY);
+        PASInit.initExtras(freshPas, 0, 1.5 ether, new address[](0), new address[](0), new address[](0));
+    }
+
+    function testInitExtrasRevertsWhenHopIsZero() public {
+        vm.expectRevert("PASInit/hop-is-zero");
+        this.initExtras();
+    }
+
     // ============================================================================
     // CoreCouncil Direct Actions (IMMEDIATE Role) Tests
     // ============================================================================