Merge branch 'v3'

src/lib/assetBridger/assetBridger.ts

@@ -16,12 +16,13 @@
 /* eslint-env node */
 'use strict'
 
-import { constants } from 'ethers'
-
 import { ParentContractTransaction } from '../message/ParentTransaction'
 import { ChildContractTransaction } from '../message/ChildTransaction'
 
-import { ArbitrumNetwork } from '../dataEntities/networks'
+import {
+  ArbitrumNetwork,
+  isArbitrumNetworkNativeTokenEther,
+} from '../dataEntities/networks'
 import {
   SignerOrProvider,
   SignerProviderUtils,
@@ -69,7 +70,7 @@ export abstract class AssetBridger<DepositParams, WithdrawParams> {
    * @returns {boolean}
    */
   protected get nativeTokenIsEth() {
-    return !this.nativeToken || this.nativeToken === constants.AddressZero
+    return isArbitrumNetworkNativeTokenEther(this.childNetwork)
   }
 
   /**

src/lib/dataEntities/networks.ts

@@ -17,6 +17,7 @@
 ;('use strict')
 
 import { Provider } from '@ethersproject/abstract-provider'
+import { constants } from 'ethers'
 
 import { SignerOrProvider, SignerProviderUtils } from './signerOrProvider'
 import { ArbSdkError } from '../dataEntities/errors'
@@ -551,6 +552,15 @@ export function assertArbitrumNetworkHasTokenBridge<T extends ArbitrumNetwork>(
   }
 }
 
+export function isArbitrumNetworkNativeTokenEther(
+  network: ArbitrumNetwork
+): boolean {
+  return (
+    typeof network.nativeToken === 'undefined' ||
+    network.nativeToken === constants.AddressZero
+  )
+}
+
 const { resetNetworksToDefault } = createNetworkStateHandler()
 
 export { resetNetworksToDefault }

src/lib/message/ParentToChildMessageCreator.ts

@@ -11,7 +11,10 @@ import {
   ParentTransactionReceipt,
 } from './ParentTransaction'
 import { Inbox__factory } from '../abi/factories/Inbox__factory'
-import { getArbitrumNetwork } from '../dataEntities/networks'
+import {
+  getArbitrumNetwork,
+  isArbitrumNetworkNativeTokenEther,
+} from '../dataEntities/networks'
 import { ERC20Inbox__factory } from '../abi/factories/ERC20Inbox__factory'
 import { PayableOverrides } from '@ethersproject/contracts'
 import { SignerProviderUtils } from '../dataEntities/signerOrProvider'
@@ -156,7 +159,7 @@ export class ParentToChildMessageCreator {
     )
 
     const childChain = await getArbitrumNetwork(childProvider)
-    const nativeTokenIsEth = typeof childChain.nativeToken === 'undefined'
+    const nativeTokenIsEth = isArbitrumNetworkNativeTokenEther(childChain)
 
     const data = ParentToChildMessageCreator.getTicketCreationRequestCallData(
       params,

tests/unit/network.test.ts

@@ -7,8 +7,11 @@ import {
   getChildrenForNetwork,
   isParentNetwork,
   getMulticallAddress,
+  isArbitrumNetworkNativeTokenEther,
+  ArbitrumNetwork,
 } from '../../src/lib/dataEntities/networks'
 import { SignerOrProvider } from '../../src/lib/dataEntities/signerOrProvider'
+import { constants } from 'ethers'
 
 const ethereumMainnetChainId = 1
 const arbitrumOneChainId = 42161
@@ -220,4 +223,30 @@ describe('Networks', async () => {
       expect(network.chainId).to.equal(arbitrumOneChainId)
     })
   })
+
+  describe('isArbitrumNetworkNativeTokenEther', () => {
+    it('returns `true` when `nativeToken` is undefined', () => {
+      expect(
+        isArbitrumNetworkNativeTokenEther({
+          nativeToken: undefined,
+        } as ArbitrumNetwork)
+      ).to.equal(true)
+    })
+
+    it('returns `true` when `nativeToken` is zero address', () => {
+      expect(
+        isArbitrumNetworkNativeTokenEther({
+          nativeToken: constants.AddressZero,
+        } as ArbitrumNetwork)
+      ).to.equal(true)
+    })
+
+    it('returns `false` when `nativeToken` is a valid address', () => {
+      expect(
+        isArbitrumNetworkNativeTokenEther({
+          nativeToken: '0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48',
+        } as ArbitrumNetwork)
+      ).to.equal(false)
+    })
+  })
 })