fix: linting errors and remove duplicate switch cases

hyperlane-xyz · Dec 12, 2024 · 94920ae · 94920ae
1 parent fae0a67
commit 94920ae
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 46 deletions.
diff --git a/typescript/sdk/src/deploy/verify/ZKSyncContractVerifier.ts b/typescript/sdk/src/deploy/verify/ZKSyncContractVerifier.ts
@@ -137,6 +137,7 @@ export class ZKSyncContractVerifier extends BaseContractVerifier {
     } catch (error) {
       verificationLogger.trace(
         {
+          error,
           failure: response.statusText,
           status: response.status,
           chain,

diff --git a/typescript/sdk/src/deploy/verify/utils.ts b/typescript/sdk/src/deploy/verify/utils.ts
@@ -1,9 +1,9 @@
 import { ethers, utils } from 'ethers';
 
-import { ZKSyncArtifact } from '@hyperlane-xyz/core';
 import {
   ProxyAdmin__factory,
   TransparentUpgradeableProxy__factory,
+  ZKSyncArtifact,
 } from '@hyperlane-xyz/core';
 import { Address, assert, eqAddress } from '@hyperlane-xyz/utils';
 
@@ -99,16 +99,7 @@ export async function getContractVerificationInputForZKSync({
 
 export function encodeArguments(abi: any, constructorArgs: any[]): string {
   const contractInterface = new utils.Interface(abi);
-  let deployArgumentsEncoded;
-  try {
-    deployArgumentsEncoded = contractInterface
-      .encodeDeploy(constructorArgs)
-      .replace('0x', '');
-  } catch (error: any) {
-    throw new Error('Cant encode constructor args');
-  }
-
-  return deployArgumentsEncoded;
+  return contractInterface.encodeDeploy(constructorArgs).replace('0x', '');
 }
 
 /**

diff --git a/typescript/sdk/src/ism/HyperlaneIsmFactory.ts b/typescript/sdk/src/ism/HyperlaneIsmFactory.ts
@@ -25,8 +25,8 @@ import {
   StorageMessageIdMultisigIsm__factory,
   TestIsm__factory,
   TrustedRelayerIsm__factory,
+  ZKSyncArtifact,
 } from '@hyperlane-xyz/core';
-import { ZKSyncArtifact } from '@hyperlane-xyz/core';
 import {
   Address,
   Domain,
@@ -231,16 +231,6 @@ export class HyperlaneIsmFactory extends HyperlaneApp<ProxyFactoryFactories> {
           [config.bridge],
         );
         break;
-      case IsmType.STORAGE_MESSAGE_ID_MULTISIG:
-      case IsmType.STORAGE_MERKLE_ROOT_MULTISIG:
-        assert(
-          this.deployer,
-          `HyperlaneDeployer must be set to deploy ${ismType}`,
-        );
-        return this.deployStorageMultisigIsm({
-          destination,
-          config,
-        });
       default:
         throw new Error(`Unsupported ISM type ${ismType}`);
     }
@@ -263,30 +253,6 @@ export class HyperlaneIsmFactory extends HyperlaneApp<ProxyFactoryFactories> {
     return contract;
   }
 
-  protected async deployStorageMultisigIsm({
-    destination,
-    config,
-  }: {
-    destination: ChainName;
-    config: MultisigIsmConfig;
-  }): Promise<IMultisigIsm> {
-    const signer = this.multiProvider.getSigner(destination);
-
-    const factory =
-      config.type === IsmType.STORAGE_MERKLE_ROOT_MULTISIG
-        ? new StorageMerkleRootMultisigIsm__factory()
-        : new StorageMessageIdMultisigIsm__factory();
-
-    const contract = await this.deployer!.deployContractFromFactory(
-      destination,
-      factory,
-      config.type,
-      [config.validators, config.threshold],
-    );
-
-    return IMultisigIsm__factory.connect(contract.address, signer);
-  }
-
   protected async deployMultisigIsm(
     destination: ChainName,
     config: MultisigIsmConfig,