feat: remove all references to L1Network

scripts/genNetwork.ts

@@ -3,7 +3,6 @@ dotenv.config()
 import { execSync } from 'child_process'
 import * as fs from 'fs'
 import { L2Network } from '../src'
-import { ARB_MINIMUM_BLOCK_TIME_IN_SECONDS } from '../src/lib/dataEntities/constants'
 import { IERC20Bridge__factory } from '../src/lib/abi/factories/IERC20Bridge__factory'
 import { ethers } from 'ethers'
 
@@ -41,12 +40,10 @@ async function patchNetworks(
   l2Provider: ethers.providers.Provider | undefined
 ) {
   l2Network.parentChainId = (l2Network as any).partnerChainID
-  l2Network.blockTime = ARB_MINIMUM_BLOCK_TIME_IN_SECONDS
 
   // native token for l3
   if (l3Network && l2Provider) {
     l3Network.parentChainId = (l3Network as any).partnerChainID
-    l3Network.blockTime = ARB_MINIMUM_BLOCK_TIME_IN_SECONDS
 
     try {
       l3Network.nativeToken = await IERC20Bridge__factory.connect(

scripts/instantiate_bridge.ts

@@ -22,12 +22,7 @@ import { Wallet } from '@ethersproject/wallet'
 import dotenv from 'dotenv'
 import args from './getCLargs'
 import { EthBridger, InboxTools, Erc20Bridger } from '../src'
-import {
-  l1Networks,
-  l2Networks,
-  L1Network,
-  L2Network,
-} from '../src/lib/dataEntities/networks'
+import { L2Network, getL2Network } from '../src/lib/dataEntities/networks'
 import { Signer } from 'ethers'
 import { AdminErc20Bridger } from '../src/lib/assetBridger/erc20Bridger'
 import { isDefined } from '../src/lib/utils/lib'
@@ -39,19 +34,18 @@ const ethKey = process.env['ETH_KEY'] as string
 
 const defaultNetworkId = 421614
 
-export const instantiateBridge = (
+export const instantiateBridge = async (
   l1PkParam?: string,
   l2PkParam?: string
-): {
-  l1Network: L1Network
+): Promise<{
   l2Network: L2Network
   l1Signer: Signer
   l2Signer: Signer
   erc20Bridger: Erc20Bridger
   ethBridger: EthBridger
   adminErc20Bridger: AdminErc20Bridger
   inboxTools: InboxTools
-} => {
+}> => {
   if (!l1PkParam && !ethKey) {
     throw new Error('need ARB_KEY var')
   }
@@ -68,21 +62,8 @@ export const instantiateBridge = (
 
     l2NetworkID = defaultNetworkId
   }
-  const isL1 = isDefined(l1Networks[l2NetworkID])
-  const isL2 = isDefined(l2Networks[l2NetworkID])
-  if (!isL1 && !isL2) {
-    throw new Error(`Unrecognized network ID: ${l2NetworkID}`)
-  }
-  if (!isL2) {
-    throw new Error(`Tests must specify an L2 network ID: ${l2NetworkID}`)
-  }
 
-  const l2Network = l2Networks[l2NetworkID]
-  const l1Network = l1Networks[l2Network.parentChainId]
-
-  if (!l1Network) {
-    throw new Error(`Unrecognised parent chain id: ${l2Network.parentChainId}`)
-  }
+  const l2Network = await getL2Network(l2NetworkID)
 
   const l1Rpc = (() => {
     if (l2NetworkID === 42161) return process.env['MAINNET_RPC'] as string
@@ -138,7 +119,6 @@ export const instantiateBridge = (
   const inboxTools = new InboxTools(l1Signer, l2Network)
 
   return {
-    l1Network,
     l2Network,
     l1Signer,
     l2Signer,

scripts/testSetup.ts

@@ -23,9 +23,7 @@ import dotenv from 'dotenv'
 
 import { EthBridger, InboxTools, Erc20Bridger } from '../src'
 import {
-  L1Network,
   ArbitrumNetwork,
-  getL1Network,
   getArbitrumNetwork,
   addCustomNetwork,
 } from '../src/lib/dataEntities/networks'
@@ -72,7 +70,6 @@ export const getSigner = (provider: JsonRpcProvider, key?: string) => {
 }
 
 export const testSetup = async (): Promise<{
-  parentChain: L1Network | ArbitrumNetwork
   childChain: ArbitrumNetwork
   parentSigner: Signer
   childSigner: Signer
@@ -95,53 +92,21 @@ export const testSetup = async (): Promise<{
   const parentSigner = seed.connect(ethProvider)
   const childSigner = seed.connect(arbProvider)
 
-  let setParentChain: L1Network | ArbitrumNetwork,
-    setChildChain: ArbitrumNetwork
+  let setChildChain: ArbitrumNetwork
+
   try {
-    const l1Network = isTestingOrbitChains
-      ? await getArbitrumNetwork(parentDeployer)
-      : await getL1Network(parentDeployer)
     const l2Network = await getArbitrumNetwork(childDeployer)
-    setParentChain = l1Network
     setChildChain = l2Network
   } catch (err) {
     // the networks havent been added yet
 
     // check if theres an existing network available
     const localNetworkFile = getLocalNetworksFromFile()
 
-    const { l1Network: parentChain, l2Network: childChain } = localNetworkFile
-
-    if (isTestingOrbitChains) {
-      const _parentChain = parentChain as ArbitrumNetwork
-      const ethLocal: L1Network = {
-        blockTime: 10,
-        chainID: _parentChain.parentChainId,
-        isCustom: true,
-        name: 'EthLocal',
-        isArbitrum: false,
-      }
-
-      addCustomNetwork({
-        customL1Network: ethLocal,
-        customArbitrumNetwork: _parentChain,
-      })
-
-      addCustomNetwork({
-        customArbitrumNetwork: childChain,
-      })
-
-      setParentChain = parentChain
-      setChildChain = childChain
-    } else {
-      addCustomNetwork({
-        customL1Network: parentChain as L1Network,
-        customArbitrumNetwork: childChain,
-      })
-
-      setParentChain = parentChain
-      setChildChain = childChain
-    }
+    const { l2Network: childChain } = localNetworkFile
+
+    addCustomNetwork(childChain)
+    setChildChain = childChain
   }
 
   const erc20Bridger = new Erc20Bridger(setChildChain)
@@ -160,7 +125,6 @@ export const testSetup = async (): Promise<{
     childSigner,
     parentProvider: ethProvider,
     childProvider: arbProvider,
-    parentChain: setParentChain,
     childChain: setChildChain,
     erc20Bridger,
     adminErc20Bridger,
@@ -172,7 +136,6 @@ export const testSetup = async (): Promise<{
 }
 
 export function getLocalNetworksFromFile(): {
-  l1Network: L1Network | ArbitrumNetwork
   l2Network: ArbitrumNetwork
 } {
   const pathToLocalNetworkFile = path.join(__dirname, '..', 'localNetwork.json')

src/index.ts

@@ -43,11 +43,7 @@ export { ParentToChildMessageGasEstimator } from './lib/message/ParentToChildMes
 export { argSerializerConstructor } from './lib/utils/byte_serialize_params'
 export { CallInput, MultiCaller } from './lib/utils/multicall'
 export {
-  L1Networks,
-  L2Networks,
-  L1Network,
   L2Network,
-  getL1Network,
   getL2Network,
   addCustomNetwork,
   addDefaultLocalNetwork,

src/lib/assetBridger/assetBridger.ts

@@ -21,11 +21,7 @@ import { constants } from 'ethers'
 import { ParentContractTransaction } from '../message/ParentTransaction'
 import { ChildContractTransaction } from '../message/ChildTransaction'
 
-import {
-  L1Network,
-  ArbitrumNetwork,
-  getParentForNetwork,
-} from '../dataEntities/networks'
+import { ArbitrumNetwork } from '../dataEntities/networks'
 import {
   SignerOrProvider,
   SignerProviderUtils,
@@ -35,11 +31,6 @@ import {
  * Base for bridging assets from parent-to-child and back
  */
 export abstract class AssetBridger<DepositParams, WithdrawParams> {
-  /**
-   * Parent chain for the given Arbitrum chain, can be an L1 or an L2
-   */
-  public readonly parentChain: L1Network | ArbitrumNetwork
-
   /**
    * In case of a chain that uses ETH as its native/gas token, this is either `undefined` or the zero address
    *
@@ -48,7 +39,6 @@ export abstract class AssetBridger<DepositParams, WithdrawParams> {
   public readonly nativeToken?: string
 
   public constructor(public readonly childChain: ArbitrumNetwork) {
-    this.parentChain = getParentForNetwork(childChain)
     this.nativeToken = childChain.nativeToken
   }
 
@@ -57,7 +47,10 @@ export abstract class AssetBridger<DepositParams, WithdrawParams> {
    * @param sop
    */
   protected async checkParentChain(sop: SignerOrProvider): Promise<void> {
-    await SignerProviderUtils.checkNetworkMatches(sop, this.parentChain.chainID)
+    await SignerProviderUtils.checkNetworkMatches(
+      sop,
+      this.childChain.parentChainId
+    )
   }
 
   /**