Refactor contracts-* exports so none in test dir so npmignore works as intended

contracts/exchange/src/exchange_data_encoder.ts

@@ -2,7 +2,7 @@ import { constants, ExchangeFunctionName, provider } from '@0x/contracts-test-ut
 import { orderHashUtils } from '@0x/order-utils';
 import { SignedOrder } from '@0x/types';
 
-import { IExchangeContract } from '../wrappers';
+import { IExchangeContract } from './wrappers';
 
 export const exchangeDataEncoder = {
     encodeOrdersToExchangeData(fnName: ExchangeFunctionName, orders: SignedOrder[] = []): string {

contracts/exchange/src/index.ts

@@ -1,6 +1,5 @@
 export * from './artifacts';
 export * from './wrappers';
-export { BlockchainBalanceStore } from '../test/balance_stores/blockchain_balance_store';
-export { LocalBalanceStore } from '../test/balance_stores/local_balance_store';
-export { TokenOwnersByName } from '../test/balance_stores/types';
-export * from '../test/utils/exchange_data_encoder';
+export { BlockchainBalanceStore } from './balance_stores/blockchain_balance_store';
+export { LocalBalanceStore } from './balance_stores/local_balance_store';
+export { exchangeDataEncoder } from './exchange_data_encoder';

contracts/exchange/test/assertion_wrappers/fill_order_wrapper.ts

@@ -16,14 +16,10 @@ import * as _ from 'lodash';
 
 import { ExchangeContract } from '../wrappers';
 
-import {
-    BalanceStore,
-    BlockchainBalanceStore,
-    LocalBalanceStore,
-    TokenContractsByName,
-    TokenIds,
-    TokenOwnersByName,
-} from './../balance_stores';
+import { BalanceStore } from '../../src/balance_stores/balance_store';
+import { BlockchainBalanceStore } from '../../src/balance_stores/blockchain_balance_store';
+import { LocalBalanceStore } from '../../src/balance_stores/local_balance_store';
+import { TokenContractsByName, TokenIds, TokenOwnersByName } from '../../src/balance_stores/types';
 
 export class FillOrderWrapper {
     private readonly _blockchainBalanceStore: BlockchainBalanceStore;

contracts/exchange/test/balance_stores/index.ts

@@ -1,4 +0,0 @@
-export { BalanceStore } from './balance_store';
-export { LocalBalanceStore } from './local_balance_store';
-export { BlockchainBalanceStore } from './blockchain_balance_store';
-export * from './types';

contracts/exchange/test/core.ts

@@ -40,7 +40,8 @@ import { Web3Wrapper } from '@0x/web3-wrapper';
 import { LogWithDecodedArgs } from 'ethereum-types';
 import * as _ from 'lodash';
 
-import { ExchangeWrapper, ValidatorWalletAction } from './utils';
+import { ValidatorWalletAction } from './utils/constants';
+import { ExchangeWrapper } from './utils/exchange_wrapper';
 
 import { artifacts } from './artifacts';
 import { FillOrderWrapper } from './assertion_wrappers/fill_order_wrapper';

contracts/exchange/test/signature_validator.ts

@@ -25,7 +25,7 @@ import {
     TestValidatorWalletContract,
 } from './wrappers';
 
-import { ValidatorWalletAction } from './utils';
+import { ValidatorWalletAction } from './utils/constants';
 
 // tslint:disable:no-unnecessary-type-assertion
 blockchainTests.resets('MixinSignatureValidator', env => {

contracts/exchange/test/transactions.ts

@@ -19,8 +19,10 @@ import { LogWithDecodedArgs, MethodAbi } from 'ethereum-types';
 import * as ethUtil from 'ethereumjs-util';
 import * as _ from 'lodash';
 
+import { exchangeDataEncoder } from '../src/exchange_data_encoder';
+
 import { artifacts as localArtifacts } from './artifacts';
-import { exchangeDataEncoder, ExchangeWrapper } from './utils';
+import { ExchangeWrapper } from './utils/exchange_wrapper';
 import {
     ExchangeCancelEventArgs,
     ExchangeCancelUpToEventArgs,

contracts/exchange/test/utils/index.ts

@@ -1,4 +0,0 @@
-export * from './exchange_wrapper';
-export * from './exchange_data_encoder';
-export * from './types';
-export * from './constants';