Can remove as this causes a double count

[0xVonBismarck]

https://github.com/CanzaApps/Baki-contracts/blob/a30a237c6468ffd4b45eca6f125c6f487b305cce/contracts/Vault.sol#L607-L627

Since we are already applying all the changes in the depositMint and RepayWithdraw functions before we trigger the test impact, we do not need to calculate the adjusted netMint since the netMint is already updated for the movements. Therefore if we do it again here we are double counting the movements.

i.e. if the NetMint is increased by 10 the Mint function will increase it by ten and then when we do the test we will be increasing that netMint once again by 10, causing the test to use 20 rather than 10.

[David405]

Not sure I understand this, the testImpact does NOT make any changes to state variables.

I think I see what you mean now.