Skip to content

Commit

Permalink
Skip flaky metering test (Agoric#3537)
Browse files Browse the repository at this point in the history
* fix: skip possibly flaky metering test
  • Loading branch information
erights authored Jul 27, 2021
1 parent 890affd commit 0afd86b
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions packages/SwingSet/test/metering/test-dynamic-vat-metered.js
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,8 @@ test('exceed stack', t => {
return overflowCrank(t, 'stack');
});

test('meter decrements', async t => {
// See the TODO comment below about `test.skip`.
test.skip('meter decrements', async t => {
const managerType = 'xs-worker';
const { kernelBundles, dynamicVatBundle, bootstrapBundle } = t.context.data;
const config = {
Expand Down Expand Up @@ -290,8 +291,13 @@ test('meter decrements', async t => {
remaining = await getMeter();
console.log(remaining);

// It looks like #3499 might reduce this to two, in which case comment out
// the following three lines.
// It looks like #3499 has made this flaky, reducing it to two on
// my (markm) local development env but not under CI. Locally, if
// I comment out the following three lines, it passes locally
// but not under CI. Leaving them uncommented passes on CI but
// not locally.
// TODO Marking this `test.skip` until this flakiness is cleared up.
// See https://github.com/Agoric/agoric-sdk/issues/3538
await consume(true);
remaining = await getMeter();
console.log(remaining);
Expand Down

0 comments on commit 0afd86b

Please sign in to comment.