Skip to content

Commit 1638f49

Browse files
JPeer264claude
andcommitted
fixup! test(bun): Run all Node integration suites on Bun
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 7f4bb42 commit 1638f49

2 files changed

Lines changed: 11 additions & 1 deletion

File tree

‎dev-packages/bun-integration-tests/node-suites/excludes.ts‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,15 @@
11
// Node suites that do not run on Bun, relative to `node-integration-tests`. A single test that
22
// fails on Bun is skipped with `test.skipIf` on `RUNTIME` in the Node suite, not listed here.
33

4-
// Node-only features: ANR and native thread watchdogs, child processes, the AWS Lambda Node runtime.
4+
// Node-only features: ANR and native thread watchdogs, child processes, the AWS Lambda Node runtime,
5+
// and `node:sqlite`, which `flue` needs.
56
const NODE_ONLY = [
67
'suites/anr/test.ts',
78
'suites/aws-serverless/**',
89
'suites/breadcrumbs/**',
910
'suites/child-process/test.ts',
1011
'suites/thread-blocked-native/test.ts',
12+
'suites/tracing/flue/test.ts',
1113
];
1214

1315
// Bun does not publish `http.server.request.start`, so `@sentry/node` creates no `http.server`
@@ -95,6 +97,7 @@ const NO_AUTO_INSTRUMENTATION = [
9597
'suites/tracing/langchain/**',
9698
'suites/tracing/langgraph/test.ts',
9799
'suites/tracing/lru-memoizer/test.ts',
100+
'suites/tracing/mastra/test.ts',
98101
'suites/tracing/mcp-handler-exact-once/test.ts',
99102
'suites/tracing/mcp-server-streamed/test.ts',
100103
'suites/tracing/mistral/test.ts',
@@ -114,6 +117,7 @@ const NO_AUTO_INSTRUMENTATION = [
114117
'suites/tracing/mysql2/test.ts',
115118
'suites/tracing/openai/test.ts',
116119
'suites/tracing/openai/v6/test.ts',
120+
'suites/tracing/openai/v7/test.ts',
117121
'suites/tracing/orchestrion-lazy-registration/test.ts',
118122
'suites/tracing/postgres-streamed/test.ts',
119123
'suites/tracing/postgres/test.ts',
@@ -138,6 +142,7 @@ const NOT_TRIAGED = [
138142
'suites/modules/test.ts',
139143
'suites/proxy/test.ts',
140144
'suites/system-error/test.ts',
145+
'suites/tracing/graphql-tracing-channel/**',
141146
'suites/tracing/tracer-start-active-span-error/test.ts',
142147
];
143148

‎dev-packages/node-integration-tests/utils/index.ts‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,11 @@ export function conditionalTest(allowedVersion: {
5050
min?: number;
5151
max?: number;
5252
}): typeof describe | typeof describe.skip {
53+
// Vitest always runs on Node, so its Node version says nothing about Bun or Deno running the
54+
// scenario. Those runtimes list the suites they cannot run in their own exclude lists.
55+
if (RUNTIME !== 'node') {
56+
return describe;
57+
}
5358
return describe.skipIf(!matchesNodeVersion(allowedVersion));
5459
}
5560

0 commit comments

Comments
 (0)