For most tokens, transfer will either return true or revert, so there's no need to check for a true return value. However, for tokens like USDT, these functions do not return anything, causing the following line in the depositCollateral function to fail despite correct execution:
bool success = IERC20(tokenCollateralAddress).transferFrom(msg.sender, address(this), amountCollateral);