aboutsummaryrefslogtreecommitdiffstats
path: root/packages/contracts/test
diff options
context:
space:
mode:
authorFabio Berger <me@fabioberger.com>2018-06-27 00:49:10 +0800
committerFabio Berger <me@fabioberger.com>2018-06-27 00:49:10 +0800
commit1bc742aed1e7f10b79b5ef23ebb57b6c93d64e3c (patch)
treece2edd713186dab60b313d50b1df64c8f4bec667 /packages/contracts/test
parentf780e6f235be0216eb37dbbdc148a9c19521cbaf (diff)
downloaddexon-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/contracts/test')
-rw-r--r--packages/contracts/test/exchange/core.ts4
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,
);
});