diff options
| author | Alex Browne <stephenalexbrowne@gmail.com> | 2018-06-30 04:12:46 +0800 |
|---|---|---|
| committer | Alex Browne <stephenalexbrowne@gmail.com> | 2018-06-30 04:46:42 +0800 |
| commit | 6cf39896f1106f414f83cda1dc42472117e872ae (patch) | |
| tree | e5a5dd1137d8c089a08af37f09f86251956cabc1 /packages/contracts/test | |
| parent | 44b6adaa2912e4c21ae93441fde410610f817b1d (diff) | |
| download | dexon-sol-tools-6cf39896f1106f414f83cda1dc42472117e872ae.tar dexon-sol-tools-6cf39896f1106f414f83cda1dc42472117e872ae.tar.gz dexon-sol-tools-6cf39896f1106f414f83cda1dc42472117e872ae.tar.bz2 dexon-sol-tools-6cf39896f1106f414f83cda1dc42472117e872ae.tar.lz dexon-sol-tools-6cf39896f1106f414f83cda1dc42472117e872ae.tar.xz dexon-sol-tools-6cf39896f1106f414f83cda1dc42472117e872ae.tar.zst dexon-sol-tools-6cf39896f1106f414f83cda1dc42472117e872ae.zip | |
Update expectRevertReasonOrAlwaysFailingTransactionAsync to check status codes
Diffstat (limited to 'packages/contracts/test')
| -rw-r--r-- | packages/contracts/test/exchange/match_orders.ts | 24 |
1 files changed, 3 insertions, 21 deletions
diff --git a/packages/contracts/test/exchange/match_orders.ts b/packages/contracts/test/exchange/match_orders.ts index e23ab9851..7bdb9c385 100644 --- a/packages/contracts/test/exchange/match_orders.ts +++ b/packages/contracts/test/exchange/match_orders.ts @@ -651,13 +651,7 @@ describe('matchOrders', () => { }); // Match orders return expectRevertReasonOrAlwaysFailingTransactionAsync( - matchOrderTester.matchOrdersAndVerifyBalancesAsync( - signedOrderLeft, - signedOrderRight, - takerAddress, - erc20BalancesByOwner, - erc721TokenIdsByOwner, - ), + exchangeWrapper.matchOrdersAsync(signedOrderLeft, signedOrderRight, takerAddress), RevertReason.NegativeSpreadRequired, ); }); @@ -680,13 +674,7 @@ describe('matchOrders', () => { }); // Match orders return expectRevertReasonOrAlwaysFailingTransactionAsync( - matchOrderTester.matchOrdersAndVerifyBalancesAsync( - signedOrderLeft, - signedOrderRight, - takerAddress, - erc20BalancesByOwner, - erc721TokenIdsByOwner, - ), + exchangeWrapper.matchOrdersAsync(signedOrderLeft, signedOrderRight, takerAddress), // We are assuming assetData fields of the right order are the // reverse of the left order, rather than checking equality. This // saves a bunch of gas, but as a result if the assetData fields are @@ -715,13 +703,7 @@ describe('matchOrders', () => { }); // Match orders return expectRevertReasonOrAlwaysFailingTransactionAsync( - matchOrderTester.matchOrdersAndVerifyBalancesAsync( - signedOrderLeft, - signedOrderRight, - takerAddress, - erc20BalancesByOwner, - erc721TokenIdsByOwner, - ), + exchangeWrapper.matchOrdersAsync(signedOrderLeft, signedOrderRight, takerAddress), RevertReason.InvalidOrderSignature, ); }); |