Make migrations deterministic

1 files changed, 45 insertions, 57 deletions

diff --git a/packages/contracts/deploy/migrations/migrate.ts b/packages/contracts/deploy/migrations/migrate.ts
index c3d38875e..43c1fa035 100644
--- a/packages/contracts/deploy/migrations/migrate.ts
+++ b/packages/contracts/deploy/migrations/migrate.ts
@@ -17,43 +17,26 @@ export const migrator = {
         const web3Wrapper: Web3Wrapper = deployer.web3Wrapper;
         const accounts: string[] = await web3Wrapper.getAvailableAddressesAsync();
 
-        const independentContracts: Web3.ContractInstance[] = await Promise.all([
-            deployer.deployAndSaveAsync('TokenTransferProxy'),
-            deployer.deployAndSaveAsync('ZRXToken'),
-            deployer.deployAndSaveAsync('EtherToken'),
-            deployer.deployAndSaveAsync('TokenRegistry'),
-        ]);
-        const [tokenTransferProxy, zrxToken, etherToken, tokenReg] = independentContracts;
+        const tokenTransferProxy = await deployer.deployAndSaveAsync('TokenTransferProxy');
+        const zrxToken = await deployer.deployAndSaveAsync('ZRXToken');
+        const etherToken = await deployer.deployAndSaveAsync('EtherToken');
+        const tokenReg = await deployer.deployAndSaveAsync('TokenRegistry');
 
         const exchangeArgs = [zrxToken.address, tokenTransferProxy.address];
         const owners = [accounts[0], accounts[1]];
         const confirmationsRequired = new BigNumber(2);
         const secondsRequired = new BigNumber(0);
         const multiSigArgs = [owners, confirmationsRequired, secondsRequired, tokenTransferProxy.address];
-        const dependentContracts: Web3.ContractInstance[] = await Promise.all([
-            deployer.deployAndSaveAsync('Exchange', exchangeArgs),
-            deployer.deployAndSaveAsync('MultiSigWalletWithTimeLockExceptRemoveAuthorizedAddress', multiSigArgs),
-        ]);
-        const [exchange, multiSig] = dependentContracts;
+        const exchange = await deployer.deployAndSaveAsync('Exchange', exchangeArgs);
+        const multiSig = await deployer.deployAndSaveAsync(
+            'MultiSigWalletWithTimeLockExceptRemoveAuthorizedAddress', multiSigArgs,
+        );
 
         const owner = accounts[0];
         await tokenTransferProxy.addAuthorizedAddress.sendTransactionAsync(exchange.address, {from: owner});
         await tokenTransferProxy.transferOwnership.sendTransactionAsync(multiSig.address, {from: owner});
-
-        const tokensToRegister: Web3.ContractInstance[] = await Promise.all(
-            _.map(tokenInfo, async (token: Token): Promise<Web3.ContractInstance> => {
-                const totalSupply = new BigNumber(0);
-                const args = [
-                    token.name,
-                    token.symbol,
-                    token.decimals,
-                    totalSupply,
-                ];
-                return deployer.deployAsync('DummyToken', args);
-            }),
-        );
         const addTokenGasEstimate = await tokenReg.addToken.estimateGasAsync(
-            tokensToRegister[0].address,
+            zrxToken.address,
             tokenInfo[0].name,
             tokenInfo[0].symbol,
             tokenInfo[0].decimals,
@@ -61,35 +44,41 @@ export const migrator = {
             tokenInfo[0].swarmHash,
             {from: owner},
         );
-        const addTokenPromises = [
-            tokenReg.addToken.sendTransactionAsync(
-                zrxToken.address,
-                '0x Protocol Token',
-                'ZRX',
-                18,
-                constants.NULL_BYTES,
-                constants.NULL_BYTES,
-                {
-                    from: owner,
-                    gas: addTokenGasEstimate,
-                },
-            ),
-            tokenReg.addToken.sendTransactionAsync(
-                etherToken.address,
-                'Ether Token',
-                'WETH',
-                18,
-                constants.NULL_BYTES,
-                constants.NULL_BYTES,
-                {
-                    from: owner,
-                    gas: addTokenGasEstimate,
-                },
-            ),
-        ];
-        const addDummyTokenPromises = _.map(tokenInfo, async (token: Token, i: number): Promise<void> => {
-            return tokenReg.addToken.sendTransactionAsync(
-                tokensToRegister[i].address,
+        await tokenReg.addToken.sendTransactionAsync(
+            zrxToken.address,
+            '0x Protocol Token',
+            'ZRX',
+            18,
+            constants.NULL_BYTES,
+            constants.NULL_BYTES,
+            {
+                from: owner,
+                gas: addTokenGasEstimate,
+            },
+        );
+        await tokenReg.addToken.sendTransactionAsync(
+            etherToken.address,
+            'Ether Token',
+            'WETH',
+            18,
+            constants.NULL_BYTES,
+            constants.NULL_BYTES,
+            {
+                from: owner,
+                gas: addTokenGasEstimate,
+            },
+        );
+        for (const token of tokenInfo) {
+            const totalSupply = new BigNumber(0);
+            const args = [
+                token.name,
+                token.symbol,
+                token.decimals,
+                totalSupply,
+            ];
+            const dummyToken = await deployer.deployAsync('DummyToken', args);
+            await tokenReg.addToken.sendTransactionAsync(
+                dummyToken.address,
                 token.name,
                 token.symbol,
                 token.decimals,
@@ -100,7 +89,6 @@ export const migrator = {
                     gas: addTokenGasEstimate,
                 },
             );
-        });
-        await Promise.all([...addDummyTokenPromises, ...addTokenPromises]);
+        }
     },
 };