Mark parsePaymasterAndData as internal (#12)

Author: mdehoog

src/Paymaster.sol

@@ -84,7 +84,7 @@ contract Paymaster is BasePaymaster {
     }
 
     function parsePaymasterAndData(bytes calldata paymasterAndData)
-    public pure returns(uint48 validUntil, uint48 validAfter, bytes calldata signature) {
+    internal pure returns(uint48 validUntil, uint48 validAfter, bytes calldata signature) {
         (validUntil, validAfter) = abi.decode(paymasterAndData[VALID_TIMESTAMP_OFFSET:SIGNATURE_OFFSET],(uint48, uint48));
         signature = paymasterAndData[SIGNATURE_OFFSET:];
     }

test/Paymaster.t.sol

@@ -46,14 +46,6 @@ contract PaymasterTest is Test {
         paymaster.renounceOwnership();
     }
 
-    function test_parsePaymasterAndData() public {
-        bytes memory paymasterAndData = abi.encodePacked(address(paymaster), abi.encode(MOCK_VALID_UNTIL, MOCK_VALID_AFTER), MOCK_SIG);
-        (uint48 validUntil, uint48 validAfter, bytes memory signature) = paymaster.parsePaymasterAndData(paymasterAndData);
-        assertEq(validUntil, MOCK_VALID_UNTIL);
-        assertEq(validAfter, MOCK_VALID_AFTER);
-        assertEq(signature, MOCK_SIG);
-    }
-
     function test_getHash() public {
         UserOperation memory userOp = createUserOp();
         userOp.initCode = "initCode";