Help on testing the "liquidation" function

[arbdev95]

hello guys, i'm testing the function "liquidate", i m going to put the two functions, to be clear:

What I'm working on

I'm working with a DeFi protocol's collateral and minting function:

function liquidate(address collateral, address user, uint256 debtToCover)
        public
        moreThanZero(debtToCover)
        nonReentrant
    {
        // need to check the health factor of the user
        uint256 startingUserHealthFactor = _healthFactor(user);
        if (startingUserHealthFactor >= MIN_HEALTH_FACTOR) {
            revert DSCEngine__HealthFactorOk();
        }
        
        uint256 tokenAmountFromDebtCovered = getTokenAmountFromUsd(collateral, debtToCover);
       
        uint256 bonusCollateral = (tokenAmountFromDebtCovered * LIQUIDATION_BONUS) / LIQUIDATION_PRECISION;
        
        uint256 totalCollateralToRedeem = tokenAmountFromDebtCovered + bonusCollateral;
        _redeemCollateral(user, msg.sender, collateral, totalCollateralToRedeem);
        _burnDsc(debtToCover, user, msg.sender);

        uint256 endingUserHealthFactor = _healthFactor(user);
        if (endingUserHealthFactor <= startingUserHealthFactor) {
            revert DSCEngine__HealthFactorNotImproved();
        }
        _revertIfHealthFactorIsBroken(msg.sender);
    }

function testLiquidationImproveHealthFactor() public {
        vm.startPrank(USER);
        ERC20Mock(weth).mint(USER, AMOUNT_COLLATERAL);
        ERC20Mock(weth).approve(address(dscengine),AMOUNT_COLLATERAL);
        dscengine.depositCollateralAndMintDsc(address(weth),AMOUNT_COLLATERAL,AMOUNT_TO_MINT);
        vm.stopPrank();

        int256 newPrice = 150e8;

        MockV3Aggregator(ethUsdPriceFeed).updateAnswer(newPrice);

        uint256 startingHealthFactor = dscengine.getHealthFactor(USER);
        console.log("starting health factor:", startingHealthFactor);

        vm.startPrank(LIQUIDATOR);

        uint256 debtToCover = 100e18;
            ERC20Mock(weth).mint(LIQUIDATOR, AMOUNT_COLLATERAL);
            ERC20Mock(weth).approve(address(dscengine), AMOUNT_COLLATERAL);
            dscengine.depositCollateralAndMintDsc(address(weth), AMOUNT_COLLATERAL,debtToCover);
            dsc.approve(address(dscengine), debtToCover);
            dscengine.liquidate(address(weth),USER, debtToCover);

            uint256 endingHealthFactor = dscengine.getHealthFactor(USER);
            console.log("ending health factor", endingHealthFactor);
            assertGt(endingHealthFactor,startingHealthFactor, "Health Factor not improved");
            vm.stopPrank();
    }
​```solidity

but always gives me this error : "[FAIL: DSCEngine__HealthFactorOk()] testLiquidationImproveHealthFactor() (gas: 527982)"

Can you help me please to find where i m wrong?

[EngrPips]

Hello @arbdev95, This error is very clear and explicit. It says DSCEngine__HealthFactorOk(), which means the user you are trying to liquidate is healthy and can not be liquidated, so you should make the user unhealthy. I believe making the user unhealthy will not be tough for you considering you understood everything so far.