From e1879ef4d91fc58203d9748eca38695b79c43674 Mon Sep 17 00:00:00 2001 From: Fabio Berger Date: Mon, 11 Jun 2018 23:42:30 +0200 Subject: Fix no-unused-variable tslint rule to include parameters and fix issues --- packages/order-utils/test/exchange_transfer_simulator_test.ts | 9 --------- .../utils/simple_erc20_balance_and_proxy_allowance_fetcher.ts | 8 ++++---- 2 files changed, 4 insertions(+), 13 deletions(-) (limited to 'packages/order-utils/test') diff --git a/packages/order-utils/test/exchange_transfer_simulator_test.ts b/packages/order-utils/test/exchange_transfer_simulator_test.ts index 3369901b1..eeae42698 100644 --- a/packages/order-utils/test/exchange_transfer_simulator_test.ts +++ b/packages/order-utils/test/exchange_transfer_simulator_test.ts @@ -2,7 +2,6 @@ import { BlockchainLifecycle, devConstants } from '@0xproject/dev-utils'; import { ExchangeContractErrs } from '@0xproject/types'; import { BigNumber } from '@0xproject/utils'; import * as chai from 'chai'; -import * as _ from 'lodash'; import 'make-promises-safe'; import { artifacts } from '../src/artifacts'; @@ -176,11 +175,3 @@ describe('ExchangeTransferSimulator', async () => { }); }); }); - -function getAddressFromArtifact(artifact: any, networkId: number): string { - if (_.isUndefined(artifact.networks[networkId])) { - throw new Error(`Contract ${artifact.contractName} not deployed to network ${networkId}`); - } - const contractAddress = artifact.networks[networkId].address.toLowerCase(); - return contractAddress; -} diff --git a/packages/order-utils/test/utils/simple_erc20_balance_and_proxy_allowance_fetcher.ts b/packages/order-utils/test/utils/simple_erc20_balance_and_proxy_allowance_fetcher.ts index 29b9a128b..d41eaca40 100644 --- a/packages/order-utils/test/utils/simple_erc20_balance_and_proxy_allowance_fetcher.ts +++ b/packages/order-utils/test/utils/simple_erc20_balance_and_proxy_allowance_fetcher.ts @@ -11,14 +11,14 @@ export class SimpleERC20BalanceAndProxyAllowanceFetcher implements AbstractBalan this._erc20TokenContract = erc20TokenWrapper; this._erc20ProxyAddress = erc20ProxyAddress; } - public async getBalanceAsync(assetData: string, userAddress: string): Promise { - // HACK: We cheat and don't pass in the userData since it's always the same token used + public async getBalanceAsync(_assetData: string, userAddress: string): Promise { + // HACK: We cheat and don't pass in the assetData since it's always the same token used // in our tests. const balance = await this._erc20TokenContract.balanceOf.callAsync(userAddress); return balance; } - public async getProxyAllowanceAsync(assetData: string, userAddress: string): Promise { - // HACK: We cheat and don't pass in the userData since it's always the same token used + public async getProxyAllowanceAsync(_assetData: string, userAddress: string): Promise { + // HACK: We cheat and don't pass in the assetData since it's always the same token used // in our tests. const proxyAllowance = await this._erc20TokenContract.allowance.callAsync(userAddress, this._erc20ProxyAddress); return proxyAllowance; -- cgit v1.2.3