## Description 

Currently only the depositERC20 checks that the conversion is greater
than 0. Moving the requirement to the conversion function itself to
capture both ERC20 and ETH deposits

## Test plan 

Added unit test coverage to account for both depositEth and depositERC20