From 375d667144e26253f8ff6b0ad1beb45d5b3ef228 Mon Sep 17 00:00:00 2001
From: Fabio Berger <me@fabioberger.com>
Date: Wed, 27 Jun 2018 10:43:12 +0200
Subject: Move combinatorial fillOrder tests into the `fill_ordere` test file

---
 .../contracts/test/exchange/combinatorial_tests.ts | 48 ----------------------
 packages/contracts/test/exchange/fill_order.ts     | 21 ++++++++++
 2 files changed, 21 insertions(+), 48 deletions(-)
 delete mode 100644 packages/contracts/test/exchange/combinatorial_tests.ts

diff --git a/packages/contracts/test/exchange/combinatorial_tests.ts b/packages/contracts/test/exchange/combinatorial_tests.ts
deleted file mode 100644
index f5f795bbe..000000000
--- a/packages/contracts/test/exchange/combinatorial_tests.ts
+++ /dev/null
@@ -1,48 +0,0 @@
-import { BlockchainLifecycle } from '@0xproject/dev-utils';
-import * as _ from 'lodash';
-
-import { chaiSetup } from '../../src/utils/chai_setup';
-import { CoreCombinatorialUtils, coreCombinatorialUtilsFactoryAsync } from '../../src/utils/core_combinatorial_utils';
-import { provider, txDefaults, web3Wrapper } from '../../src/utils/web3_wrapper';
-
-import { FillScenario } from '../../src/utils/types';
-
-chaiSetup.configure();
-const blockchainLifecycle = new BlockchainLifecycle(web3Wrapper);
-
-describe('Combinatorial tests', () => {
-    let coreCombinatorialUtils: CoreCombinatorialUtils;
-
-    before(async () => {
-        await blockchainLifecycle.startAsync();
-        coreCombinatorialUtils = await coreCombinatorialUtilsFactoryAsync(web3Wrapper, txDefaults);
-    });
-    after(async () => {
-        await blockchainLifecycle.revertAsync();
-    });
-    beforeEach(async () => {
-        await blockchainLifecycle.startAsync();
-    });
-    afterEach(async () => {
-        await blockchainLifecycle.revertAsync();
-    });
-    const test = (fillScenarios: FillScenario[]) => {
-        _.forEach(fillScenarios, fillScenario => {
-            const orderScenario = fillScenario.orderScenario;
-            const description = `Combinatorial OrderFill: ${orderScenario.feeRecipientScenario} ${
-                orderScenario.makerAssetAmountScenario
-            } ${orderScenario.takerAssetAmountScenario} ${orderScenario.makerFeeScenario} ${
-                orderScenario.takerFeeScenario
-            } ${orderScenario.expirationTimeSecondsScenario} ${orderScenario.makerAssetDataScenario} ${
-                orderScenario.takerAssetDataScenario
-            }`;
-            it(description, async () => {
-                await coreCombinatorialUtils.testFillOrderScenarioAsync(provider, fillScenario);
-            });
-        });
-    };
-
-    const allFillScenarios = CoreCombinatorialUtils.generateFillOrderCombinations();
-
-    describe('Fills orders', () => test(allFillScenarios));
-});
diff --git a/packages/contracts/test/exchange/fill_order.ts b/packages/contracts/test/exchange/fill_order.ts
index 60fe777c3..d65ab2f9a 100644
--- a/packages/contracts/test/exchange/fill_order.ts
+++ b/packages/contracts/test/exchange/fill_order.ts
@@ -1,4 +1,5 @@
 import { BlockchainLifecycle } from '@0xproject/dev-utils';
+import * as _ from 'lodash';
 
 import { chaiSetup } from '../../src/utils/chai_setup';
 import { CoreCombinatorialUtils, coreCombinatorialUtilsFactoryAsync } from '../../src/utils/core_combinatorial_utils';
@@ -10,6 +11,7 @@ import {
     BalanceAmountScenario,
     ExpirationTimeSecondsScenario,
     FeeRecipientAddressScenario,
+    FillScenario,
     OrderAssetAmountScenario,
     TakerAssetFillAmountScenario,
     TakerScenario,
@@ -62,6 +64,25 @@ describe('FillOrder Tests', () => {
         await blockchainLifecycle.revertAsync();
     });
     describe('fillOrder', () => {
+        const test = (fillScenarios: FillScenario[]) => {
+            _.forEach(fillScenarios, fillScenario => {
+                const orderScenario = fillScenario.orderScenario;
+                const description = `Combinatorial OrderFill: ${orderScenario.feeRecipientScenario} ${
+                    orderScenario.makerAssetAmountScenario
+                } ${orderScenario.takerAssetAmountScenario} ${orderScenario.makerFeeScenario} ${
+                    orderScenario.takerFeeScenario
+                } ${orderScenario.expirationTimeSecondsScenario} ${orderScenario.makerAssetDataScenario} ${
+                    orderScenario.takerAssetDataScenario
+                }`;
+                it(description, async () => {
+                    await coreCombinatorialUtils.testFillOrderScenarioAsync(provider, fillScenario);
+                });
+            });
+        };
+
+        const allFillScenarios = CoreCombinatorialUtils.generateFillOrderCombinations();
+        describe('Combinatorially generated fills orders', () => test(allFillScenarios));
+
         it('should transfer the correct amounts when makerAssetAmount === takerAssetAmount', async () => {
             const fillScenario = {
                 ...defaultFillScenario,
-- 
cgit v1.2.3