feat: update depositTo to use standard deposit

.github/workflows/build-test.yml

@@ -161,8 +161,6 @@ jobs:
       - name: Set up the local node
         uses: OffchainLabs/actions/run-nitro-test-node@main
         with:
-          nitro-testnode-ref: release
-          nitro-contracts-branch: validator-wallet-fix
           l3-node: ${{ matrix.orbit-test == '1' }}
           args: ${{ matrix.custom-fee == '1' && '--l3-fee-token' || '' }}
 

src/lib/assetBridger/ethBridger.ts

@@ -352,10 +352,12 @@ export class EthBridger extends AssetBridger<
           l1Provider: params.l1Signer.provider!,
         })
 
-    const tx = await params.l1Signer.sendTransaction({
-      ...retryableTicketRequest.txRequest,
-      ...params.overrides,
-    })
+    const l1ToL2MessageCreator = new L1ToL2MessageCreator(params.l1Signer)
+
+    const tx = await l1ToL2MessageCreator.createRetryableTicket(
+      retryableTicketRequest,
+      params.l2Provider
+    )
 
     return L1TransactionReceipt.monkeyPatchContractCallWait(tx)
   }

tests/integration/eth.test.ts

@@ -21,6 +21,7 @@
 
 import { Wallet } from '@ethersproject/wallet'
 import { parseEther } from '@ethersproject/units'
+import { constants } from 'ethers'
 
 import {
   fundL1,
@@ -32,11 +33,15 @@
 import { L2ToL1Message } from '../../src/lib/message/L2ToL1Message'
 import { L2ToL1MessageStatus } from '../../src/lib/dataEntities/message'
 import { L2TransactionReceipt } from '../../src/lib/message/L2Transaction'
-import { L1ToL2MessageStatus } from '../../src/lib/message/L1ToL2Message'
+import {
+  L1ToL2MessageStatus,
+  L1ToL2MessageWriter,
+} from '../../src/lib/message/L1ToL2Message'
 import { testSetup } from '../../scripts/testSetup'
 import { isL2NetworkWithCustomFeeToken } from './custom-fee-token/customFeeTokenTestHelpers'
 import { ERC20__factory } from '../../src/lib/abi/factories/ERC20__factory'
 import { itOnlyWhenEth } from './custom-fee-token/mochaExtensions'
+import { L1TransactionReceipt } from '../../src'
 
 dotenv.config()
 
@@ -211,6 +216,60 @@
     )
   })
 
+  it('deposit ether to a specific L2 address and check balance before auto-redeem', async () => {
+    const { ethBridger, l1Signer, l2Signer } = await testSetup()
+
+    await fundL1(l1Signer)
+    const destWallet = Wallet.createRandom()
+
+    const ethToDeposit = parseEther('0.0002')
+    const res = await ethBridger.depositTo({
+      amount: ethToDeposit,
+      l1Signer: l1Signer,
+      destinationAddress: destWallet.address,
+      l2Provider: l2Signer.provider!,
+      retryableGasOverrides: {
+        gasLimit: {
+          // causes auto-redeem to fail which allows us to check balances before it happens
+          base: constants.Zero,
+        },
+      },
+    })
+    const rec = await res.wait()
+    const l1ToL2Messages = await rec.getL1ToL2Messages(l2Signer.provider!)
+    expect(l1ToL2Messages.length).to.eq(1, 'failed to find 1 l1 to l2 message')
+    let l1ToL2Message = l1ToL2Messages[0]
+
+    const retryableTicketResult = await l1ToL2Message.waitForStatus()
+
+    expect(retryableTicketResult.status).to.eq(
+      L1ToL2MessageStatus.FUNDS_DEPOSITED_ON_L2,
+      'unexpected status, expected auto-redeem to fail'
+    )
+
+    const testWalletL2EthBalance = await l2Signer.provider!.getBalance(
+      destWallet.address
+    )
+    expect(
+      testWalletL2EthBalance.lt(ethToDeposit),
+      'balance before auto-redeem'
+    ).to.be.true
+
+    const l1TxHash = await l1Signer.provider!.getTransactionReceipt(res.hash)
+    const l1Receipt = new L1TransactionReceipt(l1TxHash)
+    l1ToL2Message = l1Receipt.getL1ToL2Messages(l2Signer)[0]
+
+    if (l1ToL2Message instanceof L1ToL2MessageWriter) {
+      const res = await l1ToL2Message.redeem()
+      await res.wait()
+    }
+
+    expect(
+      testWalletL2EthBalance.toString(),
+      'balance after manual redeem'
+    ).to.eq(ethToDeposit.toString())
+  })
+
   it('withdraw Ether transaction succeeds', async () => {
     const { l2Signer, l1Signer, ethBridger } = await testSetup()
     await fundL2(l2Signer)