change in testRevertsIfCollateralIsNotAllowed in advaned Foundry

[0xshubhs]

theres a change nw we have to use abi.encodeWithSelector for expected reverts for Token Not Allowed

for eg this doesnt work

    function testRevertsIfCollateralIsNotAllowed() public {
        ERC20Mock meowToken = new ERC20Mock("MeowToken", "meow", USER, 1234567);
        vm.startPrank(USER);
        meowToken.approve(address(dscEngine), 1234567);
        vm.expectRevert(DSCEngine.DSCEngine__TokenNotAllowed.selector);
        dscEngine.depositCollateral(address(meowToken),1234567);
        vm.stopPrank();
    }

because the error is defined with a parameter address token. When using vm.expectRevert with a custom error that has parameters, you need to encode the full error with its parameters, not just use the selector, so changing vm.expectRevert(DSCEngine.DSCEngine__TokenNotAllowed.selector) to vm.expectRevert(abi.encodeWithSelector(DSCEngine.DSCEngine__TokenNotAllowed.selector, address(meowToken))) since the error includes the token address as a parameter.

    function testRevertsIfCollateralIsNotAllowed() public {
        ERC20Mock meowToken = new ERC20Mock("MeowToken", "meow", USER, 1234567);
        vm.startPrank(USER);
        meowToken.approve(address(dscEngine), 1234567);
        vm.expectRevert(abi.encodeWithSelector(DSCEngine.DSCEngine__TokenNotAllowed.selector, address(meowToken)));
        dscEngine.depositCollateral(address(meowToken),1234567);
        vm.stopPrank();
    }

PS: meowToken is ranToken lol , this is in latest update of foundry

[EngrPips]

Hey @0xshubhs, Thank you so much for this heads-up.