fix: use _checkRole and remove unused unauth error

l2-contracts/src/ZkCappedMinterV2.sol

@@ -23,9 +23,6 @@ contract ZkCappedMinterV2 is AccessControl {
   /// @notice Error for when the cap is exceeded.
   error ZkCappedMinterV2__CapExceeded(address minter, uint256 amount);
 
-  /// @notice Error for when the account is unauthorized.
-  error ZkCappedMinterV2__Unauthorized(address account);
-
   /// @notice Constructor for a new ZkCappedMinterV2 contract
   /// @param _token The token contract where tokens will be minted.
   /// @param _admin The address that will be granted the admin role.
@@ -41,19 +38,12 @@ contract ZkCappedMinterV2 is AccessControl {
   /// @param _to The address that will receive the new tokens.
   /// @param _amount The quantity of tokens, in raw decimals, that will be created.
   function mint(address _to, uint256 _amount) external {
-    _revertIfUnauthorized();
+    _checkRole(MINTER_ROLE, msg.sender);
     _revertIfCapExceeded(_amount);
     minted += _amount;
     TOKEN.mint(_to, _amount);
   }
 
-  /// @notice Reverts if the account is unauthorized.
-  function _revertIfUnauthorized() internal view {
-    if (!hasRole(MINTER_ROLE, msg.sender)) {
-      revert ZkCappedMinterV2__Unauthorized(msg.sender);
-    }
-  }
-
   /// @notice Reverts if the amount of new tokens will increase the minted tokens beyond the mint cap.
   /// @param _amount The quantity of tokens, in raw decimals, that will checked against the cap.
   function _revertIfCapExceeded(uint256 _amount) internal view {

l2-contracts/test/ZkCappedMinterV2.t.sol

@@ -5,6 +5,7 @@ import {ZkTokenTest} from "test/utils/ZkTokenTest.sol";
 import {IMintableAndDelegatable} from "src/interfaces/IMintableAndDelegatable.sol";
 import {ZkCappedMinterV2} from "src/ZkCappedMinterV2.sol";
 import {console2} from "forge-std/Test.sol";
+import {Strings} from "lib/openzeppelin-contracts/contracts/utils/Strings.sol";
 
 contract ZkCappedMinterV2Test is ZkTokenTest {
   function setUp() public virtual override {
@@ -99,7 +100,16 @@ contract Mint is ZkCappedMinterV2Test {
     vm.assume(_nonMinter != address(0));
     vm.assume(!cappedMinter.hasRole(MINTER_ROLE, _nonMinter));
 
-    vm.expectRevert(abi.encodeWithSelector(ZkCappedMinterV2.ZkCappedMinterV2__Unauthorized.selector, _nonMinter));
+    vm.expectRevert(
+      bytes(
+        string.concat(
+          "AccessControl: account ",
+          Strings.toHexString(uint160(_nonMinter), 20),
+          " is missing role ",
+          Strings.toHexString(uint256(MINTER_ROLE))
+        )
+      )
+    );
     vm.prank(_nonMinter);
     cappedMinter.mint(_nonMinter, _cap);
   }
@@ -137,7 +147,16 @@ contract Mint is ZkCappedMinterV2Test {
 
     ZkCappedMinterV2 cappedMinter = createCappedMinter(_admin, _cap);
 
-    vm.expectRevert(abi.encodeWithSelector(ZkCappedMinterV2.ZkCappedMinterV2__Unauthorized.selector, _admin));
+    vm.expectRevert(
+      bytes(
+        string.concat(
+          "AccessControl: account ",
+          Strings.toHexString(uint160(_admin), 20),
+          " is missing role ",
+          Strings.toHexString(uint256(MINTER_ROLE))
+        )
+      )
+    );
     vm.prank(_admin);
     cappedMinter.mint(_receiver, _amount);
   }