Skip to content

Commit a8e10f4

Browse files
committed
Fix linter
1 parent db55a48 commit a8e10f4

File tree

9 files changed

+4
-28
lines changed

9 files changed

+4
-28
lines changed

dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/mocks.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,4 +52,3 @@ export class MockAnthropic {
5252
};
5353
}
5454
}
55-

dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/subject.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
import * as Sentry from '@sentry/browser';
22
import { MockAnthropic } from './mocks.js';
33

4-
window.Sentry = Sentry;
5-
64
Sentry.init({
75
dsn: 'https://[email protected]/1337',
86
tracesSampleRate: 1,
@@ -22,4 +20,3 @@ client.messages.create({
2220
temperature: 0.7,
2321
max_tokens: 100,
2422
});
25-

dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/test.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,13 @@
11
import { expect } from '@playwright/test';
22
import { sentryTest } from '../../../../utils/fixtures';
3-
import { envelopeRequestParser, shouldSkipTracingTest, waitForTransactionRequest } from '../../../../utils/helpers';
3+
import { envelopeRequestParser, waitForTransactionRequest } from '../../../../utils/helpers';
44

55
// These tests are not exhaustive because the instrumentation is
66
// already tested in the node integration tests and we merely
77
// want to test that the instrumentation does not crash in the browser
88
// and that gen_ai transactions are sent.
99

1010
sentryTest('manual Anthropic instrumentation sends gen_ai transactions', async ({ getLocalTestUrl, page }) => {
11-
if (shouldSkipTracingTest()) {
12-
sentryTest.skip();
13-
}
14-
1511
const transactionPromise = waitForTransactionRequest(page, event => {
1612
return !!event.transaction?.includes('claude-3-haiku-20240307');
1713
});

dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/mocks.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,4 +115,3 @@ export class MockGoogleGenAI {
115115
};
116116
}
117117
}
118-

dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/subject.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
import * as Sentry from '@sentry/browser';
22
import { MockGoogleGenAI } from './mocks.js';
33

4-
window.Sentry = Sentry;
5-
64
Sentry.init({
75
dsn: 'https://[email protected]/1337',
86
tracesSampleRate: 1,
@@ -34,5 +32,4 @@ const chat = client.chats.create({
3432

3533
chat.sendMessage({
3634
message: 'Tell me a joke',
37-
})
38-
35+
});

dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/test.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,13 @@
11
import { expect } from '@playwright/test';
22
import { sentryTest } from '../../../../utils/fixtures';
3-
import { envelopeRequestParser, shouldSkipTracingTest, waitForTransactionRequest } from '../../../../utils/helpers';
3+
import { envelopeRequestParser, waitForTransactionRequest } from '../../../../utils/helpers';
44

55
// These tests are not exhaustive because the instrumentation is
66
// already tested in the node integration tests and we merely
77
// want to test that the instrumentation does not crash in the browser
88
// and that gen_ai transactions are sent.
99

1010
sentryTest('manual Google GenAI instrumentation sends gen_ai transactions', async ({ getLocalTestUrl, page }) => {
11-
if (shouldSkipTracingTest()) {
12-
sentryTest.skip();
13-
}
14-
1511
const transactionPromise = waitForTransactionRequest(page, event => {
1612
return !!event.transaction?.includes('gemini-1.5-pro');
1713
});

dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/mocks.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,4 +44,3 @@ export class MockOpenAi {
4444
};
4545
}
4646
}
47-

dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/subject.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
import * as Sentry from '@sentry/browser';
22
import { MockOpenAi } from './mocks.js';
33

4-
window.Sentry = Sentry;
5-
64
Sentry.init({
75
dsn: 'https://[email protected]/1337',
86
tracesSampleRate: 1,
@@ -25,4 +23,3 @@ client.chat.completions.create({
2523
temperature: 0.7,
2624
max_tokens: 100,
2725
});
28-

dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/test.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,13 @@
11
import { expect } from '@playwright/test';
22
import { sentryTest } from '../../../../utils/fixtures';
3-
import { envelopeRequestParser, shouldSkipTracingTest, waitForTransactionRequest } from '../../../../utils/helpers';
3+
import { envelopeRequestParser, waitForTransactionRequest } from '../../../../utils/helpers';
44

55
// These tests are not exhaustive because the instrumentation is
66
// already tested in the node integration tests and we merely
77
// want to test that the instrumentation does not crash in the browser
88
// and that gen_ai transactions are sent.
99

1010
sentryTest('manual OpenAI instrumentation sends gen_ai transactions', async ({ getLocalTestUrl, page }) => {
11-
if (shouldSkipTracingTest()) {
12-
sentryTest.skip();
13-
}
14-
1511
const transactionPromise = waitForTransactionRequest(page, event => {
1612
return !!event.transaction?.includes('gpt-3.5-turbo');
1713
});

0 commit comments

Comments
 (0)