Wrong calculation of `balanceBefore` and `balanceAfter` in deposit method
Deposit method calculate net amount transferred from user. It use `reservedFunds` also in consideration when calculating `balanceBefore` and `balanceAfter` but it is not actually require.\\n```\\n    uint256 balanceBefore = getVaultBalance() - reservedFunds;\\n    vaultCurrency.safeTransferFrom(msg.sender, address(this), _amount);\\n    uint256 balanceAfter = getVaultBalance() - reservedFunds;\\n    uint256 amount = balanceAfter - balanceBefore;\\n```\\n\\nDeposit may fail when `reservedFunds` is greater than `getVaultBalance()`
Issue Wrong calculation of `balanceBefore` and `balanceAfter` in deposit method\\nUse below code. This is correct way of finding net amount transfer by depositor\\n```\\n    uint256 balanceBefore = getVaultBalance();\\n    vaultCurrency.safeTransferFrom(msg.sender, address(this), _amount);\\n    uint256 balanceAfter = getVaultBalance();\\n    uint256 amount = balanceAfter - balanceBefore;\\n```\\n
Deposit may fail when `reservedFunds` is greater than `getVaultBalance()`
```\\n    uint256 balanceBefore = getVaultBalance() - reservedFunds;\\n    vaultCurrency.safeTransferFrom(msg.sender, address(this), _amount);\\n    uint256 balanceAfter = getVaultBalance() - reservedFunds;\\n    uint256 amount = balanceAfter - balanceBefore;\\n```\\n