Skip to content

Commit 994c099

Browse files
committed
test(napi/parser): skip slow test (#14424)
The raw transfer range + parent test on `antd.js` fixture is still hitting 5 second timeout on CI. For some reason the `timeout` option isn't increasing the timeout. Just skip this test.
1 parent d8da4a4 commit 994c099

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

napi/parser/test/parse-raw.test.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// Tests for raw transfer.
22

33
import { mkdir, readdir, readFile, stat, writeFile } from 'node:fs/promises';
4-
import { basename, join as pathJoin } from 'node:path';
4+
import { basename, join as pathJoin, sep as pathSep } from 'node:path';
55
import Tinypool from 'tinypool';
66
import { describe, expect, it } from 'vitest';
77

@@ -225,10 +225,12 @@ describe.concurrent('fixtures', () => {
225225
it.each(benchFixturePaths)('%s', path => runCaseInWorker(TEST_TYPE_FIXTURE, path));
226226
});
227227

228-
// `antd.js` test sometimes takes longer than 5 seconds on CI, so increase timeout to 10 seconds
228+
// `antd.js` test sometimes takes longer than 5 seconds on CI, so skip it.
229+
// TODO: Why doesn't `timeout` option work?
229230
describeRangeParent.concurrent('range & parent fixtures', { timeout: 10_000 }, () => {
231+
const paths = benchFixturePaths.filter(path => !path.endsWith(`${pathSep}antd.js`));
230232
// oxlint-disable-next-line jest/expect-expect
231-
it.each(benchFixturePaths)('%s', path => runCaseInWorker(TEST_TYPE_FIXTURE | TEST_TYPE_RANGE_PARENT, path));
233+
it.each(paths)('%s', path => runCaseInWorker(TEST_TYPE_FIXTURE | TEST_TYPE_RANGE_PARENT, path));
232234
});
233235

234236
// Check lazy deserialization doesn't throw

0 commit comments

Comments
 (0)