Updating tests after fixing log decoding

0xProject · Jan 14, 2020 · 7a7f70e · 7a7f70e
1 parent b3c3ec1
commit 7a7f70e
Showing 1 changed file with 17 additions and 18 deletions.
diff --git a/contracts/exchange/test/wrapper_unit_tests.ts b/contracts/exchange/test/wrapper_unit_tests.ts
@@ -121,26 +121,25 @@ blockchainTests('Exchange wrapper functions unit tests.', env => {
         }
     }
 
-    function assertSameOrderFromEvent(actual: any[], expected: Order): void {
-        expect(actual.length === 14);
-        expect(actual[0].toLowerCase()).to.be.eq(expected.makerAddress);
-        expect(actual[1].toLowerCase()).to.be.eq(expected.takerAddress);
-        expect(actual[2].toLowerCase()).to.be.eq(expected.feeRecipientAddress);
-        expect(actual[3].toLowerCase()).to.be.eq(expected.senderAddress);
-        expect(actual[4]).to.be.bignumber.eq(expected.makerAssetAmount);
-        expect(actual[5]).to.be.bignumber.eq(expected.takerAssetAmount);
-        expect(actual[6]).to.be.bignumber.eq(expected.makerFee);
-        expect(actual[7]).to.be.bignumber.eq(expected.takerFee);
-        expect(actual[8]).to.be.bignumber.eq(expected.expirationTimeSeconds);
-        expect(actual[9]).to.be.bignumber.eq(expected.salt);
-        expect(actual[10]).to.be.eq(expected.makerAssetData);
-        expect(actual[11]).to.be.eq(expected.takerAssetData);
-        expect(actual[12]).to.be.eq(expected.makerFeeAssetData);
-        expect(actual[13]).to.be.eq(expected.takerFeeAssetData);
+    function assertSameOrderFromEvent(actual: any, expected: Order): void {
+        expect(actual.makerAddress).to.be.eq(expected.makerAddress);
+        expect(actual.takerAddress).to.be.eq(expected.takerAddress);
+        expect(actual.feeRecipientAddress).to.be.eq(expected.feeRecipientAddress);
+        expect(actual.senderAddress).to.be.eq(expected.senderAddress);
+        expect(actual.makerAssetAmount).to.bignumber.eq(expected.makerAssetAmount);
+        expect(actual.takerAssetAmount).to.bignumber.eq(expected.takerAssetAmount);
+        expect(actual.makerFee).to.bignumber.eq(expected.makerFee);
+        expect(actual.takerFee).to.bignumber.eq(expected.takerFee);
+        expect(actual.expirationTimeSeconds).to.bignumber.eq(expected.expirationTimeSeconds);
+        expect(actual.salt).to.bignumber.eq(expected.salt);
+        expect(actual.makerAssetData).to.eq(expected.makerAssetData);
+        expect(actual.takerAssetData).to.eq(expected.takerAssetData);
+        expect(actual.makerFeeAssetData).to.eq(expected.makerFeeAssetData);
+        expect(actual.takerFeeAssetData).to.eq(expected.takerFeeAssetData);
     }
 
-    describe.only('fillOrKillOrder', () => {
-        it.only('works if the order is filled by exactly `takerAssetFillAmount`', async () => {
+    describe('fillOrKillOrder', () => {
+        it('works if the order is filled by exactly `takerAssetFillAmount`', async () => {
             const fillAmount = randomAmount();
             const order = randomOrder({
                 // `_fillOrder()` is overridden to always return `order.takerAssetAmount` as