From 342432dc76d1bfc2531fb2edef77afc523eacefe Mon Sep 17 00:00:00 2001 From: Amir Bandeali Date: Tue, 29 May 2018 22:09:57 -0700 Subject: Update Exchange statuses, revert instead of emmitting event on fill/cancel failures, and remove redundant logic in matchOrders --- packages/contracts/test/exchange/transactions.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'packages/contracts/test/exchange/transactions.ts') diff --git a/packages/contracts/test/exchange/transactions.ts b/packages/contracts/test/exchange/transactions.ts index 9af8b522b..6480b7e0b 100644 --- a/packages/contracts/test/exchange/transactions.ts +++ b/packages/contracts/test/exchange/transactions.ts @@ -194,9 +194,9 @@ describe('Exchange transactions', () => { it('should cancel the order when signed by maker and called by sender', async () => { await exchangeWrapper.executeTransactionAsync(signedTx, senderAddress); - const res = await exchangeWrapper.fillOrderAsync(signedOrder, senderAddress); - const newBalances = await erc20Wrapper.getBalancesAsync(); - expect(newBalances).to.deep.equal(erc20Balances); + return expect(exchangeWrapper.fillOrderAsync(signedOrder, senderAddress)).to.be.rejectedWith( + constants.REVERT, + ); }); }); }); -- cgit