Add allowed value to AllowedZoneAlreadySet error

Author: lfportal

contracts/trading/seaport16/ImmutableSeaport.sol

@@ -26,7 +26,7 @@ contract ImmutableSeaport is Consideration, Ownable, ImmutableSeaportEvents {
     mapping(address => bool) public allowedZones;
 
     error OrderNotRestricted(uint8 orderType);
-    error AllowedZoneAlreadySet(address zone);
+    error AllowedZoneAlreadySet(address zone, bool allowed);
     error InvalidZone(address zone);
 
     /**
@@ -52,7 +52,7 @@ contract ImmutableSeaport is Consideration, Ownable, ImmutableSeaportEvents {
         require(zone != address(0), "ImmutableSeaport: zone is the zero address");
 
         if (allowedZones[zone] == allowed) {
-            revert AllowedZoneAlreadySet(zone);
+            revert AllowedZoneAlreadySet(zone, allowed);
         }
 
         allowedZones[zone] = allowed;

test/trading/seaport16/ImmutableSeaportConfig.t.sol

@@ -21,14 +21,22 @@ contract ImmutableSeaportConfigTest is ImmutableSeaportBaseTest {
         immutableSeaport.setAllowedZone(address(0), true);
     }
 
-    function testRejectAllowedZoneAlreadySet() public {
+    function testRejectAllowedZoneAlreadySetToTrue() public {
         address zone = makeAddr("zone");
 
         vm.prank(owner);
         immutableSeaport.setAllowedZone(zone, true);
 
         vm.prank(owner);
-        vm.expectRevert(abi.encodeWithSelector(ImmutableSeaport.AllowedZoneAlreadySet.selector, zone));
+        vm.expectRevert(abi.encodeWithSelector(ImmutableSeaport.AllowedZoneAlreadySet.selector, zone, true));
         immutableSeaport.setAllowedZone(zone, true);
     }
+
+    function testRejectAllowedZoneAlreadySetToFalse() public {
+        address zone = makeAddr("zone");
+
+        vm.prank(owner);
+        vm.expectRevert(abi.encodeWithSelector(ImmutableSeaport.AllowedZoneAlreadySet.selector, zone, false));
+        immutableSeaport.setAllowedZone(zone, false);
+    }
 }