From 6beaf425005be0a39aaee8f7d745a248d39d56ae Mon Sep 17 00:00:00 2001 From: Vladimir Sheremet Date: Fri, 28 Oct 2022 09:18:20 +0200 Subject: [PATCH] test: improve benchmark test stability --- test/benchmark/test.mjs | 10 +++++----- test/benchmark/test/base.bench.ts | 6 +++--- test/benchmark/test/only.bench.ts | 10 +++++----- 3 files changed, 13 insertions(+), 13 deletions(-) diff --git a/test/benchmark/test.mjs b/test/benchmark/test.mjs index 411a5046920b..1e4740574877 100644 --- a/test/benchmark/test.mjs +++ b/test/benchmark/test.mjs @@ -13,6 +13,11 @@ await execa('npx', ['vitest', 'bench', 'base.bench', 'mode.bench', 'only.bench'] error = e }) +if (error) { + console.error(error) + process.exit(1) +} + const benchResult = await readFile('./bench.json', 'utf-8') if (benchResult.includes('skip')) @@ -26,9 +31,4 @@ const todoBenches = ['unimplemented suite', 'unimplemented test'] if (todoBenches.some(b => benchResult.includes(b))) process.exit(1) -if (error) { - console.error(error) - process.exit(1) -} - process.exit(0) diff --git a/test/benchmark/test/base.bench.ts b/test/benchmark/test/base.bench.ts index 421cde41c6f3..3e86a64fd040 100644 --- a/test/benchmark/test/base.bench.ts +++ b/test/benchmark/test/base.bench.ts @@ -6,17 +6,17 @@ describe('sort', () => { x.sort((a, b) => { return a - b }) - }) + }, { iterations: 5, time: 0 }) bench('reverse', () => { const x = [1, 5, 4, 2, 3] x.reverse().sort((a, b) => { return a - b }) - }) + }, { iterations: 5, time: 0 }) // TODO: move to failed tests - // should not be collect + // should not be collected // it('test', () => { // expect(1 + 1).toBe(3) // }) diff --git a/test/benchmark/test/only.bench.ts b/test/benchmark/test/only.bench.ts index 09bcee8e0218..bf012cace8b5 100644 --- a/test/benchmark/test/only.bench.ts +++ b/test/benchmark/test/only.bench.ts @@ -6,7 +6,7 @@ const run = [false, false, false, false, false] describe('a0', () => { bench.only('0', () => { run[0] = true - }) + }, { iterations: 1, time: 0 }) bench('s0', () => { expect(true).toBe(false) }) @@ -17,7 +17,7 @@ describe('a1', () => { describe('c1', () => { bench.only('1', () => { run[1] = true - }) + }, { iterations: 1, time: 0 }) }) bench('s1', () => { expect(true).toBe(false) @@ -28,7 +28,7 @@ describe('a1', () => { describe.only('a2', () => { bench('2', () => { run[2] = true - }) + }, { iterations: 1, time: 0 }) }) bench('s2', () => { @@ -39,7 +39,7 @@ describe.only('a3', () => { describe('b3', () => { bench('3', () => { run[3] = true - }) + }, { iterations: 1, time: 0 }) }) bench.skip('s3', () => { expect(true).toBe(false) @@ -50,7 +50,7 @@ describe('a4', () => { describe.only('b4', () => { bench('4', () => { run[4] = true - }) + }, { iterations: 1, time: 0 }) }) describe('sb4', () => { bench('s4', () => {