dont include solana in ethers (#501)

Author: GregoryElliott

src/api/alchemy-provider.ts

@@ -81,6 +81,9 @@ export class AlchemyProvider
     // ethers. This allows the parent super constructor in JsonRpcProvider to
     // correctly set the network.
     const ethersNetwork = EthersNetwork[alchemyNetwork];
+    if (!ethersNetwork) {
+      throw new Error(`Unsupported network: ${alchemyNetwork}`);
+    }
     super(connection, ethersNetwork);
 
     this.apiKey = config.apiKey;

src/util/const.ts

@@ -58,8 +58,8 @@ export const EthersNetwork = {
   [Network.MATIC_MAINNET]: 'matic',
   [Network.MATIC_MUMBAI]: 'maticmum',
   [Network.MATIC_AMOY]: 'maticamoy',
-  [Network.SOLANA_MAINNET]: 'solana-mainnet',
-  [Network.SOLANA_DEVNET]: 'solana-devnet',
+  [Network.SOLANA_MAINNET]: null, // Solana networks are not supported by ethers.js
+  [Network.SOLANA_DEVNET]: null, // Solana networks are not supported by ethers.js
   [Network.ASTAR_MAINNET]: 'astar-mainnet',
   [Network.POLYGONZKEVM_MAINNET]: 'polygonzkevm-mainnet',
   [Network.POLYGONZKEVM_TESTNET]: 'polygonzkevm-testnet',

test/integration/provider.test.ts

@@ -4,7 +4,8 @@ import {
   Alchemy,
   AlchemyProvider,
   AlchemySubscription,
-  AlchemyWebSocketProvider
+  AlchemyWebSocketProvider,
+  Network
 } from '../../src';
 import { EthersNetwork } from '../../src/util/const';
 import { loadAlchemyEnv } from '../test-util';
@@ -26,6 +27,12 @@ describe('AlchemyProvider', () => {
     alchemy = new Alchemy({
       apiKey: process.env.ALCHEMY_API_KEY
     });
+    if (
+      alchemy.config.network === Network.SOLANA_MAINNET ||
+      alchemy.config.network === Network.SOLANA_DEVNET
+    ) {
+      return;
+    }
     ethersProvider = new EthersAlchemyProvider(
       EthersNetwork[alchemy.config.network],
       alchemy.config.apiKey

test/unit/alchemy.test.ts

@@ -16,7 +16,13 @@ describe('Alchemy class', () => {
         await alchemy.config.getProvider();
       });
     }
-    for (const network of Object.values(Network)) {
+
+    // Filter out Solana networks as they don't exist in ethers
+    const nonSolanaNetworks = Object.values(Network).filter(
+      network => !network.startsWith('solana-')
+    );
+
+    for (const network of nonSolanaNetworks) {
       testNetwork(network);
     }
   });

test/unit/provider.test.ts

@@ -12,7 +12,13 @@ describe('AlchemyProvider', () => {
         await alchemy.config.getWebSocketProvider();
       });
     }
-    for (const network of Object.values(Network)) {
+
+    // Filter out Solana networks as they don't exist in ethers
+    const nonSolanaNetworks = Object.values(Network).filter(
+      network => !network.startsWith('solana-')
+    );
+
+    for (const network of nonSolanaNetworks) {
       testNetwork(network);
     }
   });