diff options
author | Fabio Berger <me@fabioberger.com> | 2018-06-27 00:49:10 +0800 |
---|---|---|
committer | Fabio Berger <me@fabioberger.com> | 2018-06-27 00:49:10 +0800 |
commit | 1bc742aed1e7f10b79b5ef23ebb57b6c93d64e3c (patch) | |
tree | ce2edd713186dab60b313d50b1df64c8f4bec667 /packages | |
parent | f780e6f235be0216eb37dbbdc148a9c19521cbaf (diff) | |
download | dexon-sol-tools-1bc742aed1e7f10b79b5ef23ebb57b6c93d64e3c.tar.gz dexon-sol-tools-1bc742aed1e7f10b79b5ef23ebb57b6c93d64e3c.tar.zst dexon-sol-tools-1bc742aed1e7f10b79b5ef23ebb57b6c93d64e3c.zip |
Temporarily switch revert reasons to `TransferFailed`. Should be `InvalidAmount` but because of an oversight in the assembly implementation of `dispatchTransferFrom`, it always throws `TransferFailed`
Diffstat (limited to 'packages')
-rw-r--r-- | packages/contracts/test/exchange/core.ts | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/packages/contracts/test/exchange/core.ts b/packages/contracts/test/exchange/core.ts index 59756fe9a..99d2bc157 100644 --- a/packages/contracts/test/exchange/core.ts +++ b/packages/contracts/test/exchange/core.ts @@ -817,7 +817,7 @@ describe('Exchange core', () => { const takerAssetFillAmount = signedOrder.takerAssetAmount; return expectRevertReasonOrAlwaysFailingTransactionAsync( exchangeWrapper.fillOrderAsync(signedOrder, takerAddress, { takerAssetFillAmount }), - RevertReason.InvalidAmount, + RevertReason.TransferFailed, ); }); @@ -840,7 +840,7 @@ describe('Exchange core', () => { const takerAssetFillAmount = signedOrder.takerAssetAmount; return expectRevertReasonOrAlwaysFailingTransactionAsync( exchangeWrapper.fillOrderAsync(signedOrder, takerAddress, { takerAssetFillAmount }), - RevertReason.InvalidAmount, + RevertReason.TransferFailed, ); }); |