Skip to content

Commit 69c3bca

Browse files
wedamijaclaude
andcommitted
test(node): Skip cron integration suite on bun run
Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 054232b commit 69c3bca

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ const NO_FETCH_INSTRUMENTATION = [
6262
// instrumentation creates no spans. Apps must be built with `@sentry/bun/plugin`.
6363
// See https://github.com/getsentry/sentry-javascript/issues/23882
6464
export const NO_AUTO_INSTRUMENTATION = [
65+
'suites/cron/cron-integration/test.ts',
6566
'suites/cron/node-cron/integration/test.ts',
6667
'suites/cron/node-schedule-integration/test.ts',
6768
'suites/express/**',

‎packages/server-utils/test/orchestrion/moduleInjectedTransform.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ describe('channel integration definitions', () => {
4444

4545
// `@nestjs/*` and `@remix-run/*` are covered by listeners in their own SDK packages.
4646
// Crons integrations are opt-in, so bundler-only SDKs must not install them for every app.
47-
const optIn = new Set(['node-cron', 'node-schedule']);
47+
const optIn = new Set(['cron', 'node-cron', 'node-schedule']);
4848
const uncovered = [...configured].filter(
4949
name => !defined.has(name) && !optIn.has(name) && !name.startsWith('@nestjs/') && !name.startsWith('@remix-run/'),
5050
);

0 commit comments

Comments
 (0)