crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Added gas field so tests pass on Geth;

Browse Source
This commit is contained in:
Greg Hysen
2018-12-14 11:22:52 -08:00
parent 1de92659ee
commit 00f5b94d0a
1 changed files with 30 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
contracts/extensions/test/extensions/balance_threshold_filter.ts
View File

@@ -428,7 +428,13 @@ describe(ContractName.BalanceThresholdFilter, () => {
const txReceipt = await erc721TakerBalanceThresholdWrapper.batchFillOrdersNoThrowAsync(
orders,
validTakerAddress,
{ takerAssetFillAmounts },
{
takerAssetFillAmounts,
// HACK(albrow): We need to hardcode the gas estimate here because
// the Geth gas estimator doesn't work with the way we use
// delegatecall and swallow errors.
gas: 600000,
},
);
// Assert validated addresses
const expectedValidatedAddresseses = [
@@ -704,7 +710,13 @@ describe(ContractName.BalanceThresholdFilter, () => {
const txReceipt = await erc721TakerBalanceThresholdWrapper.fillOrderNoThrowAsync(
validSignedOrder,
validTakerAddress,
{ takerAssetFillAmount },
{
takerAssetFillAmount,
// HACK(albrow): We need to hardcode the gas estimate here because
// the Geth gas estimator doesn't work with the way we use
// delegatecall and swallow errors.
gas: 600000,
},
);
// Assert validated addresses
const expectedValidatedAddresseses = [validSignedOrder.makerAddress, validTakerAddress];
@@ -963,7 +975,13 @@ describe(ContractName.BalanceThresholdFilter, () => {
const txReceipt = await erc721TakerBalanceThresholdWrapper.marketSellOrdersNoThrowAsync(
orders,
validTakerAddress,
{ takerAssetFillAmount: cumulativeTakerAssetFillAmount },
{
takerAssetFillAmount: cumulativeTakerAssetFillAmount,
// HACK(albrow): We need to hardcode the gas estimate here because
// the Geth gas estimator doesn't work with the way we use
// delegatecall and swallow errors.
gas: 600000,
},
);
// Assert validated addresses
const expectedValidatedAddresseses = [
@@ -1163,7 +1181,13 @@ describe(ContractName.BalanceThresholdFilter, () => {
const txReceipt = await erc721TakerBalanceThresholdWrapper.marketBuyOrdersNoThrowAsync(
orders,
validTakerAddress,
{ makerAssetFillAmount: cumulativeMakerAssetFillAmount },
{
makerAssetFillAmount: cumulativeMakerAssetFillAmount,
// HACK(albrow): We need to hardcode the gas estimate here because
// the Geth gas estimator doesn't work with the way we use
// delegatecall and swallow errors.
gas: 600000,
},
);
// Assert validated addresses
const expectedValidatedAddresseses = [
@@ -1493,7 +1517,7 @@ describe(ContractName.BalanceThresholdFilter, () => {
// Cancel
const txReceipt = await erc721MakerBalanceThresholdWrapper.batchCancelOrdersAsync(
validSignedOrders,
validSignedOrder.makerAddress,
validSignedOrders[0].makerAddress,
);
// Assert validated addresses
const expectedValidatedAddresseses: string[] = [];
@@ -1560,7 +1584,7 @@ describe(ContractName.BalanceThresholdFilter, () => {
const cancelOrdersUpToThisSalt = new BigNumber(1);
const txReceipt = await erc721MakerBalanceThresholdWrapper.cancelOrdersUpToAsync(
cancelOrdersUpToThisSalt,
validSignedOrder.makerAddress,
validSignedOrders[0].makerAddress,
);
// Assert validated addresses
const expectedValidatedAddresseses: string[] = [];