Fix geth tests
This commit is contained in:
parent
e480e08aa4
commit
f6b6619c08
@ -272,8 +272,12 @@ describe('AssetProxyOwner', () => {
|
|||||||
await multiSigWrapper.confirmTransactionAsync(erc721AddAuthorizedAddressTxId, owners[1]);
|
await multiSigWrapper.confirmTransactionAsync(erc721AddAuthorizedAddressTxId, owners[1]);
|
||||||
await increaseTimeAndMineBlockAsync(SECONDS_TIME_LOCKED.toNumber());
|
await increaseTimeAndMineBlockAsync(SECONDS_TIME_LOCKED.toNumber());
|
||||||
await multiSigWrapper.executeTransactionAsync(registerAssetProxyTxId, owners[0]);
|
await multiSigWrapper.executeTransactionAsync(registerAssetProxyTxId, owners[0]);
|
||||||
await multiSigWrapper.executeTransactionAsync(erc20AddAuthorizedAddressTxId, owners[0]);
|
await multiSigWrapper.executeTransactionAsync(erc20AddAuthorizedAddressTxId, owners[0], {
|
||||||
await multiSigWrapper.executeTransactionAsync(erc721AddAuthorizedAddressTxId, owners[0]);
|
gas: constants.MAX_EXECUTE_TRANSACTION_GAS,
|
||||||
|
});
|
||||||
|
await multiSigWrapper.executeTransactionAsync(erc721AddAuthorizedAddressTxId, owners[0], {
|
||||||
|
gas: constants.MAX_EXECUTE_TRANSACTION_GAS,
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('validRemoveAuthorizedAddressAtIndexTx', () => {
|
describe('validRemoveAuthorizedAddressAtIndexTx', () => {
|
||||||
|
@ -6,7 +6,6 @@ import * as _ from 'lodash';
|
|||||||
import { AssetProxyOwnerContract } from '../../generated_contract_wrappers/asset_proxy_owner';
|
import { AssetProxyOwnerContract } from '../../generated_contract_wrappers/asset_proxy_owner';
|
||||||
import { MultiSigWalletContract } from '../../generated_contract_wrappers/multi_sig_wallet';
|
import { MultiSigWalletContract } from '../../generated_contract_wrappers/multi_sig_wallet';
|
||||||
|
|
||||||
import { constants } from './constants';
|
|
||||||
import { LogDecoder } from './log_decoder';
|
import { LogDecoder } from './log_decoder';
|
||||||
|
|
||||||
export class MultiSigWrapper {
|
export class MultiSigWrapper {
|
||||||
@ -41,10 +40,14 @@ export class MultiSigWrapper {
|
|||||||
const tx = await this._logDecoder.getTxWithDecodedLogsAsync(txHash);
|
const tx = await this._logDecoder.getTxWithDecodedLogsAsync(txHash);
|
||||||
return tx;
|
return tx;
|
||||||
}
|
}
|
||||||
public async executeTransactionAsync(txId: BigNumber, from: string): Promise<TransactionReceiptWithDecodedLogs> {
|
public async executeTransactionAsync(
|
||||||
|
txId: BigNumber,
|
||||||
|
from: string,
|
||||||
|
opts: { gas?: number } = {},
|
||||||
|
): Promise<TransactionReceiptWithDecodedLogs> {
|
||||||
const txHash = await this._multiSig.executeTransaction.sendTransactionAsync(txId, {
|
const txHash = await this._multiSig.executeTransaction.sendTransactionAsync(txId, {
|
||||||
from,
|
from,
|
||||||
gas: constants.MAX_EXECUTE_TRANSACTION_GAS,
|
gas: opts.gas,
|
||||||
});
|
});
|
||||||
const tx = await this._logDecoder.getTxWithDecodedLogsAsync(txHash);
|
const tx = await this._logDecoder.getTxWithDecodedLogsAsync(txHash);
|
||||||
return tx;
|
return tx;
|
||||||
@ -57,7 +60,6 @@ export class MultiSigWrapper {
|
|||||||
const txHash = await (this
|
const txHash = await (this
|
||||||
._multiSig as AssetProxyOwnerContract).executeRemoveAuthorizedAddressAtIndex.sendTransactionAsync(txId, {
|
._multiSig as AssetProxyOwnerContract).executeRemoveAuthorizedAddressAtIndex.sendTransactionAsync(txId, {
|
||||||
from,
|
from,
|
||||||
gas: constants.MAX_EXECUTE_TRANSACTION_GAS,
|
|
||||||
});
|
});
|
||||||
const tx = await this._logDecoder.getTxWithDecodedLogsAsync(txHash);
|
const tx = await this._logDecoder.getTxWithDecodedLogsAsync(txHash);
|
||||||
return tx;
|
return tx;
|
||||||
|
Loading…
x
Reference in New Issue
Block a user