Fix no-unused-variable tslint rule to include parameters and fix issues

2 files changed, 4 insertions, 13 deletions

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<BigNumber> {
-        // 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<BigNumber> {
+        // 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<BigNumber> {
-        // 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<BigNumber> {
+        // 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;