Skip to content

Commit

Permalink
test: updated estimation assertion improve failing noop tests
Browse files Browse the repository at this point in the history
  • Loading branch information
hui-an-yang committed Sep 20, 2024
1 parent 53dad47 commit 85077a7
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 7 deletions.
8 changes: 4 additions & 4 deletions integration-tests/__tests__/contract/estimation-tests.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ CONFIGS().forEach(({ lib, setup, knownBaker, createAddress, rpc }) => {
expect(estimate.minimalFeeMutez).toEqual(374);
expect(estimate.totalCost).toEqual(374);
expect(estimate.usingBaseFeeMutez).toEqual(374);
expect(estimate.consumedMilligas).toEqual(1455884);
expect(estimate.consumedMilligas).toEqual(1455970);
});

it('Verify .estimate.transfer for multiple internal transfers to unallocated account', async () => {
Expand All @@ -119,7 +119,7 @@ CONFIGS().forEach(({ lib, setup, knownBaker, createAddress, rpc }) => {
expect(estimate.minimalFeeMutez).toEqual(445);
expect(estimate.totalCost).toEqual(133945);
expect(estimate.usingBaseFeeMutez).toEqual(445);
expect(estimate.consumedMilligas).toEqual(1570413);
expect(estimate.consumedMilligas).toEqual(1570499);
});

it('Verify .estimate.transfer for internal origination', async () => {
Expand All @@ -132,7 +132,7 @@ CONFIGS().forEach(({ lib, setup, knownBaker, createAddress, rpc }) => {
expect(estimate.minimalFeeMutez).toEqual(421);
expect(estimate.totalCost).toEqual(84671);
expect(estimate.usingBaseFeeMutez).toEqual(421);
expect(estimate.consumedMilligas).toEqual(1866508);
expect(estimate.consumedMilligas).toEqual(1866594);
});

it('Verify .estimate.transfer for multiple internal originations', async () => {
Expand All @@ -145,7 +145,7 @@ CONFIGS().forEach(({ lib, setup, knownBaker, createAddress, rpc }) => {
expect(estimate.minimalFeeMutez).toEqual(539);
expect(estimate.totalCost).toEqual(164039);
expect(estimate.usingBaseFeeMutez).toEqual(539);
expect(estimate.consumedMilligas).toEqual(2391661);
expect(estimate.consumedMilligas).toEqual(2391747);
// Do the actual operation
const op2 = await contract.methods.do(originate2()).send();
await op2.confirmation();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ CONFIGS().forEach(({ setup, rpc, lib }) => {
let signerAlice = new InMemorySigner('edsk3QoqBuvdamxouPhin7swCvkQNgq4jP5KZPbwWNnwdZpSpJiEbq')

describe(`Test failing_noop through wallet api using: ${rpc}`, () => {
let Tezos = lib
const Tezos = lib
beforeAll(async () => {
setup(true)
if (rpc.includes('paris')) {
Expand Down
2 changes: 1 addition & 1 deletion integration-tests/__tests__/wallet/failing-noop.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ CONFIGS().forEach(({ setup, rpc, lib }) => {
let signerAlice = new InMemorySigner('edsk3QoqBuvdamxouPhin7swCvkQNgq4jP5KZPbwWNnwdZpSpJiEbq')

describe(`Test failing_noop through wallet api using: ${rpc}`, () => {
let Tezos = lib
const Tezos = lib
beforeAll(async () => {
setup(true)
if (rpc.includes('paris')) {
Expand Down
2 changes: 1 addition & 1 deletion packages/taquito/src/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ export enum ChainIds {
OXFORDNET2 = 'NetXxWsskGahzQB',
PARISBNET = 'NetXo8SqH1c38SS',
PARISCNET = 'NetXXWAHLEvre9b',
QUEBECANET = 'NetXsTfFgyGbHTz',
QUEBECANET = 'NetXQWuX99cUDUQ',
}

// A fixed fee reveal operation gasLimit accepted by both simulate and injection endpoint is between 1.2-5 times of actual gas consumption (3.5 fails occasionally with gas exhausted; 4 fails occasionally with fee too low)
Expand Down

0 comments on commit 85077a7

Please sign in to comment.