fix types for unsafeDeposit and add test

[wilsoncusack]

There was an issue with the types for writeUnsafeDepositTransaction was not allowing value to be passed. I solved this by making the types less generic, which I think is good.

WriteContractParameters<
    typeof optimismPortalABI,
    'depositTransaction',
    TChain,
    TAccount,
    TChainOverride
  >

Note: now value is required to be passed. We should think about how to distinguish the args.value from the top level value. The latter being the amount that will be minted on L2, the former being the mint that will be sent as msg.value in the call

[zencephalon]

Type narrowing here seems good to me