Merge pull request #1451 from polywrap/fix-ethereum-provider-config

fix: add polywrap infura provider to CLI test-env config
polywrap · Dec 16, 2022 · 3cefe9b · 3cefe9b
2 parents 91342a9 + fc1b8da
commit 3cefe9b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/packages/cli/src/lib/test-env/client-config.ts b/packages/cli/src/lib/test-env/client-config.ts
@@ -43,6 +43,14 @@ export function getTestEnvClientConfig(): Partial<PolywrapClientConfig> {
               testnet: new Connection({
                 provider: ethProvider,
               }),
+              mainnet: new Connection({
+                provider:
+                  "https://mainnet.infura.io/v3/b00b2c2cc09c487685e9fb061256d6a6",
+              }),
+              goerli: new Connection({
+                provider:
+                  "https://goerli.infura.io/v3/b00b2c2cc09c487685e9fb061256d6a6",
+              }),
             },
           }),
         }),

diff --git a/packages/js/plugins/ethereum/src/index.ts b/packages/js/plugins/ethereum/src/index.ts
@@ -442,14 +442,14 @@ export class EthereumPlugin extends Module<EthereumPluginConfig> {
     // This behavior is a consequence of how the ens-resolver uses the Ethereum plugin, always specifying the connection network name (e.g. mainnet)
     if (
       !connection?.node &&
-      this.env.connection &&
-      this.env.connection.networkNameOrChainId ===
+      this.env?.connection &&
+      this.env?.connection.networkNameOrChainId ===
         connection?.networkNameOrChainId
     ) {
-      return this._connections.getConnection(this.env.connection);
+      return this._connections.getConnection(this.env?.connection);
     }
 
-    return this._connections.getConnection(connection || this.env.connection);
+    return this._connections.getConnection(connection || this.env?.connection);
   }
 }