Skip to content

Commit 621407d

Browse files
committed
keep server alive
1 parent 68fee73 commit 621407d

File tree

6 files changed

+21
-11
lines changed

6 files changed

+21
-11
lines changed

dev-packages/node-integration-tests/suites/tracing/langchain/v1/scenario-init-chat-model.mjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,9 @@ async function run() {
104104

105105
await Sentry.flush(2000);
106106

107-
server.close();
107+
await new Promise((resolve, reject) => {
108+
server.close(err => (err ? reject(err) : resolve()));
109+
});
108110
}
109111

110112
run();

dev-packages/node-integration-tests/suites/tracing/langchain/v1/scenario-message-truncation.mjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,9 @@ async function run() {
6666

6767
await Sentry.flush(2000);
6868

69-
server.close();
69+
await new Promise((resolve, reject) => {
70+
server.close(err => (err ? reject(err) : resolve()));
71+
});
7072
}
7173

7274
run();

dev-packages/node-integration-tests/suites/tracing/langchain/v1/scenario-openai-before-langchain.mjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,9 @@ async function run() {
8989
});
9090

9191
await Sentry.flush(2000);
92-
server.close();
92+
await new Promise((resolve, reject) => {
93+
server.close(err => (err ? reject(err) : resolve()));
94+
});
9395
}
9496

9597
run();

dev-packages/node-integration-tests/suites/tracing/langchain/v1/scenario-tools.mjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,9 @@ async function run() {
8484

8585
await Sentry.flush(2000);
8686

87-
server.close();
87+
await new Promise((resolve, reject) => {
88+
server.close(err => (err ? reject(err) : resolve()));
89+
});
8890
}
8991

9092
run();

dev-packages/node-integration-tests/suites/tracing/langchain/v1/scenario.mjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,9 @@ async function run() {
104104

105105
await Sentry.flush(2000);
106106

107-
server.close();
107+
await new Promise((resolve, reject) => {
108+
server.close(err => (err ? reject(err) : resolve()));
109+
});
108110
}
109111

110112
run();

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ describe('LangChain integration (v1)', () => {
146146
'scenario.mjs',
147147
'instrument.mjs',
148148
(createRunner, test) => {
149-
test('creates langchain related spans with sendDefaultPii: false', { timeout: 30_000 }, async () => {
149+
test('creates langchain related spans with sendDefaultPii: false', async () => {
150150
await createRunner()
151151
.ignore('event')
152152
.expect({ transaction: EXPECTED_TRANSACTION_DEFAULT_PII_FALSE })
@@ -168,7 +168,7 @@ describe('LangChain integration (v1)', () => {
168168
'scenario.mjs',
169169
'instrument-with-pii.mjs',
170170
(createRunner, test) => {
171-
test('creates langchain related spans with sendDefaultPii: true', { timeout: 30_000 }, async () => {
171+
test('creates langchain related spans with sendDefaultPii: true', async () => {
172172
await createRunner()
173173
.ignore('event')
174174
.expect({ transaction: EXPECTED_TRANSACTION_DEFAULT_PII_TRUE })
@@ -218,7 +218,7 @@ describe('LangChain integration (v1)', () => {
218218
'scenario-tools.mjs',
219219
'instrument.mjs',
220220
(createRunner, test) => {
221-
test('creates langchain spans with tool calls', { timeout: 30_000 }, async () => {
221+
test('creates langchain spans with tool calls', async () => {
222222
await createRunner()
223223
.ignore('event')
224224
.expect({ transaction: EXPECTED_TRANSACTION_TOOL_CALLS })
@@ -276,7 +276,7 @@ describe('LangChain integration (v1)', () => {
276276
'scenario-message-truncation.mjs',
277277
'instrument-with-pii.mjs',
278278
(createRunner, test) => {
279-
test('truncates messages when they exceed byte limit', { timeout: 30_000 }, async () => {
279+
test('truncates messages when they exceed byte limit', async () => {
280280
await createRunner()
281281
.ignore('event')
282282
.expect({ transaction: EXPECTED_TRANSACTION_MESSAGE_TRUNCATION })
@@ -298,7 +298,7 @@ describe('LangChain integration (v1)', () => {
298298
'scenario-openai-before-langchain.mjs',
299299
'instrument.mjs',
300300
(createRunner, test) => {
301-
test('demonstrates timing issue with duplicate spans (ESM only)', { timeout: 30_000 }, async () => {
301+
test('demonstrates timing issue with duplicate spans (ESM only)', async () => {
302302
await createRunner()
303303
.ignore('event')
304304
.expect({
@@ -427,7 +427,7 @@ describe('LangChain integration (v1)', () => {
427427
'scenario-init-chat-model.mjs',
428428
'instrument.mjs',
429429
(createRunner, test) => {
430-
test('creates langchain spans using initChatModel with OpenAI', { timeout: 30_000 }, async () => {
430+
test('creates langchain spans using initChatModel with OpenAI', async () => {
431431
await createRunner()
432432
.ignore('event')
433433
.expect({ transaction: EXPECTED_TRANSACTION_INIT_CHAT_MODEL })

0 commit comments

Comments
 (0)