Fix role for lazyMint (#253)

* fix role for lazyMint

* run prettier

* remove redundant role check

Author: kumaryash90

contracts/signature-drop/SignatureDrop.sol

@@ -142,7 +142,7 @@ contract SignatureDrop is
         uint256 _amount,
         string calldata _baseURIForTokens,
         bytes calldata _data
-    ) public override onlyRole(minterRole) returns (uint256 batchId) {
+    ) public override returns (uint256 batchId) {
         if (_data.length > 0) {
             (bytes memory encryptedURI, bytes32 provenanceHash) = abi.decode(_data, (bytes, bytes32));
             if (encryptedURI.length != 0 && provenanceHash != "") {
@@ -306,7 +306,7 @@ contract SignatureDrop is
 
     /// @dev Returns whether lazy minting can be done in the given execution context.
     function _canLazyMint() internal view virtual override returns (bool) {
-        return hasRole(DEFAULT_ADMIN_ROLE, _msgSender());
+        return hasRole(minterRole, _msgSender());
     }
 
     /*///////////////////////////////////////////////////////////////

lib/forge-std

@@ -462,14 +462,16 @@ contract SignatureDropTest is BaseTest {
      *  note: Testing revert condition; an address without MINTER_ROLE calls lazyMint function.
      */
     function test_revert_lazyMint_MINTER_ROLE() public {
-        bytes memory errorMessage = abi.encodePacked(
-            "Permissions: account ",
-            Strings.toHexString(uint160(address(this)), 20),
-            " is missing role ",
-            Strings.toHexString(uint256(keccak256("MINTER_ROLE")), 32)
-        );
+        bytes32 _minterRole = keccak256("MINTER_ROLE");
 
-        vm.expectRevert(errorMessage);
+        vm.prank(deployerSigner);
+        sigdrop.grantRole(_minterRole, address(0x345));
+
+        vm.prank(address(0x345));
+        sigdrop.lazyMint(100, "ipfs://", emptyEncodedBytes);
+
+        vm.prank(address(0x567));
+        vm.expectRevert("Not authorized");
         sigdrop.lazyMint(100, "ipfs://", emptyEncodedBytes);
     }