Skip to content

Commit 513e42d

Browse files
committed
remove conditional test stuff
1 parent 80d832f commit 513e42d

File tree

18 files changed

+16
-59
lines changed

18 files changed

+16
-59
lines changed

dev-packages/node-integration-tests/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
"build:types": "tsc -p tsconfig.types.json",
1717
"clean": "rimraf -g **/node_modules && run-p clean:script",
1818
"clean:script": "node scripts/clean.js",
19-
"prisma:init": "(cd suites/tracing/prisma-orm && ts-node ./setup.ts)",
19+
"prisma:init": "cd suites/tracing/prisma-orm && yarn && yarn setup",
2020
"lint": "eslint . --format stylish",
2121
"fix": "eslint . --format stylish --fix",
2222
"type-check": "tsc",

dev-packages/node-integration-tests/suites/anr/test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import type { Event } from '@sentry/core';
2-
import { conditionalTest } from '../../utils';
32
import { cleanupChildProcesses, createRunner } from '../../utils/runner';
43

54
const ANR_EVENT = {
@@ -107,7 +106,7 @@ const ANR_EVENT_WITH_DEBUG_META: Event = {
107106
},
108107
};
109108

110-
conditionalTest({ min: 16 })('should report ANR when event loop blocked', () => {
109+
describe('should report ANR when event loop blocked', () => {
111110
afterAll(() => {
112111
cleanupChildProcesses();
113112
});

dev-packages/node-integration-tests/suites/contextLines/test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import { join } from 'path';
2-
import { conditionalTest } from '../../utils';
32
import { createRunner } from '../../utils/runner';
43

5-
conditionalTest({ min: 18 })('ContextLines integration in ESM', () => {
4+
describe('ContextLines integration in ESM', () => {
65
test('reads encoded context lines from filenames with spaces', done => {
76
expect.assertions(1);
87
const instrumentPath = join(__dirname, 'instrument.mjs');

dev-packages/node-integration-tests/suites/esm/import-in-the-middle/test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
import { spawnSync } from 'child_process';
22
import { join } from 'path';
3-
import { conditionalTest } from '../../../utils';
43
import { cleanupChildProcesses } from '../../../utils/runner';
54

65
afterAll(() => {
76
cleanupChildProcesses();
87
});
98

10-
conditionalTest({ min: 18 })('import-in-the-middle', () => {
9+
describe('import-in-the-middle', () => {
1110
test('onlyIncludeInstrumentedModules', () => {
1211
const result = spawnSync('node', [join(__dirname, 'app.mjs')], { encoding: 'utf-8' });
1312
expect(result.stderr).not.toMatch('should be the only hooked modules but we just hooked');

dev-packages/node-integration-tests/suites/esm/modules-integration/test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,10 @@
1-
import { conditionalTest } from '../../../utils';
21
import { cleanupChildProcesses, createRunner } from '../../../utils/runner';
32

43
afterAll(() => {
54
cleanupChildProcesses();
65
});
76

8-
conditionalTest({ min: 18 })('modulesIntegration', () => {
7+
describe('modulesIntegration', () => {
98
test('does not crash ESM setups', done => {
109
createRunner(__dirname, 'app.mjs').ensureNoErrorOutput().start(done);
1110
});

dev-packages/node-integration-tests/suites/no-code/test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import { conditionalTest } from '../../utils';
21
import { cleanupChildProcesses, createRunner } from '../../utils/runner';
32

43
const EVENT = {
@@ -25,7 +24,7 @@ describe('no-code init', () => {
2524
.start(done);
2625
});
2726

28-
conditionalTest({ min: 18 })('--import', () => {
27+
describe('--import', () => {
2928
test('ESM', done => {
3029
createRunner(__dirname, 'app.mjs')
3130
.withFlags('--import=@sentry/node/init')

dev-packages/node-integration-tests/suites/public-api/LocalVariables/test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ const EXPECTED_LOCAL_VARIABLES_EVENT = {
3737
},
3838
};
3939

40-
conditionalTest({ min: 18 })('LocalVariables integration', () => {
40+
describe('LocalVariables integration', () => {
4141
afterAll(() => {
4242
cleanupChildProcesses();
4343
});

dev-packages/node-integration-tests/suites/tracing/ai/test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1-
import { conditionalTest } from '../../../utils';
21
import { cleanupChildProcesses, createRunner } from '../../../utils/runner';
32

43
// `ai` SDK only support Node 18+
5-
conditionalTest({ min: 18 })('ai', () => {
4+
describe('ai', () => {
65
afterAll(() => {
76
cleanupChildProcesses();
87
});

dev-packages/node-integration-tests/suites/tracing/prisma-orm/setup.ts

Lines changed: 0 additions & 18 deletions
This file was deleted.

dev-packages/node-integration-tests/suites/tracing/prisma-orm/test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
1-
import { conditionalTest } from '../../../utils';
21
import { createRunner } from '../../../utils/runner';
32

4-
conditionalTest({ min: 16 })('Prisma ORM Tests', () => {
3+
describe('Prisma ORM Tests', () => {
54
test('CJS - should instrument PostgreSQL queries from Prisma ORM', done => {
65
const EXPECTED_TRANSACTION = {
76
transaction: 'Test Transaction',

0 commit comments

Comments
 (0)