fix: token registry dry run gas limit (#191)

fix: token-registry dry run - roles error
Open-Attestation · Jan 27, 2022 · c62cf7a · c62cf7a
1 parent caabd91
commit c62cf7a
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/implementations/deploy/token-registry/token-registry.ts b/src/implementations/deploy/token-registry/token-registry.ts
@@ -16,19 +16,19 @@ export const deployTokenRegistry = async ({
   dryRun,
   ...rest
 }: DeployTokenRegistryCommand): Promise<TransactionReceipt> => {
+  const wallet = await getWalletOrSigner({ network, ...rest });
+  const factory = new TradeTrustErc721Factory(wallet);
   if (dryRun) {
-    // TODO this does not work ?
-    const factory = new TradeTrustErc721Factory();
+    const unsignedTx = factory.getDeployTransaction(registryName, registrySymbol, {});
+    const tx = await wallet.populateTransaction(unsignedTx);
     await dryRunMode({
       network,
       gasPriceScale: gasPriceScale,
-      transaction: factory.getDeployTransaction(registryName, registrySymbol, {}),
+      transaction: tx,
     });
     process.exit(0);
   }
-  const wallet = await getWalletOrSigner({ network, ...rest });
   const gasPrice = await wallet.provider.getGasPrice();
-  const factory = new TradeTrustErc721Factory(wallet);
   signale.await(`Sending transaction to pool`);
   const transaction = await factory.deploy(registryName, registrySymbol, { gasPrice: gasPrice.mul(gasPriceScale) });
   trace(`Tx hash: ${transaction.deployTransaction.hash}`);