Skip to content

Commit a6c4654

Browse files
committed
Fix tests
1 parent af7cea0 commit a6c4654

File tree

3 files changed

+33
-37
lines changed
  • dev-packages
    • browser-integration-tests/suites/integrations/supabase/db-operations
    • e2e-tests/test-applications/supabase-nextjs
  • packages/core/src/integrations

3 files changed

+33
-37
lines changed

dev-packages/browser-integration-tests/suites/integrations/supabase/db-operations/test.ts

Lines changed: 31 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -4,50 +4,46 @@ import type { Event } from '@sentry/core';
44
import { sentryTest } from '../../../../utils/fixtures';
55
import { getFirstSentryEnvelopeRequest, getMultipleSentryEnvelopeRequests } from '../../../../utils/helpers';
66

7-
describe('Supabase Integration', () => {
8-
sentryTest('should capture Supabase database operation breadcrumbs', async ({ getLocalTestUrl, page }) => {
9-
const url = await getLocalTestUrl({ testDir: __dirname });
10-
11-
const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
12-
13-
expect(eventData.breadcrumbs).toBeDefined();
14-
expect(eventData.breadcrumbs).toContainEqual({
15-
timestamp: expect.any(Number),
16-
type: 'supabase',
17-
category: 'db.insert',
18-
message: 'from(todos)',
19-
data: expect.any(Object),
20-
});
7+
sentryTest('should capture Supabase database operation breadcrumbs', async ({ getLocalTestUrl, page }) => {
8+
const url = await getLocalTestUrl({ testDir: __dirname });
9+
10+
const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
11+
12+
expect(eventData.breadcrumbs).toBeDefined();
13+
expect(eventData.breadcrumbs).toContainEqual({
14+
timestamp: expect.any(Number),
15+
type: 'supabase',
16+
category: 'db.insert',
17+
message: 'from(todos)',
18+
data: expect.any(Object),
2119
});
20+
});
2221

23-
sentryTest('should capture multiple Supabase operations in sequence', async ({ getLocalTestUrl, page }) => {
24-
const url = await getLocalTestUrl({ testDir: __dirname });
22+
sentryTest('should capture multiple Supabase operations in sequence', async ({ getLocalTestUrl, page }) => {
23+
const url = await getLocalTestUrl({ testDir: __dirname });
2524

26-
const events = await getMultipleSentryEnvelopeRequests<Event>(page, 2, { url });
25+
const events = await getMultipleSentryEnvelopeRequests<Event>(page, 2, { url });
2726

28-
expect(events).toHaveLength(2);
27+
expect(events).toHaveLength(2);
2928

30-
events.forEach(event => {
31-
expect(
32-
event.breadcrumbs?.some(
33-
breadcrumb => breadcrumb.type === 'supabase' && breadcrumb?.category?.startsWith('db.'),
34-
),
35-
).toBe(true);
36-
});
29+
events.forEach(event => {
30+
expect(
31+
event.breadcrumbs?.some(breadcrumb => breadcrumb.type === 'supabase' && breadcrumb?.category?.startsWith('db.')),
32+
).toBe(true);
3733
});
34+
});
3835

39-
sentryTest('should include correct data payload in Supabase breadcrumbs', async ({ getLocalTestUrl, page }) => {
40-
const url = await getLocalTestUrl({ testDir: __dirname });
36+
sentryTest('should include correct data payload in Supabase breadcrumbs', async ({ getLocalTestUrl, page }) => {
37+
const url = await getLocalTestUrl({ testDir: __dirname });
4138

42-
const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
39+
const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
4340

44-
const supaBreadcrumb = eventData.breadcrumbs?.find(b => b.type === 'supabase');
41+
const supaBreadcrumb = eventData.breadcrumbs?.find(b => b.type === 'supabase');
4542

46-
expect(supaBreadcrumb).toBeDefined();
47-
expect(supaBreadcrumb?.data).toMatchObject({
48-
table: expect.any(String),
49-
operation: expect.any(String),
50-
timestamp: expect.any(Number),
51-
});
43+
expect(supaBreadcrumb).toBeDefined();
44+
expect(supaBreadcrumb?.data).toMatchObject({
45+
table: expect.any(String),
46+
operation: expect.any(String),
47+
timestamp: expect.any(Number),
5248
});
5349
});

dev-packages/e2e-tests/test-applications/supabase-nextjs/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"test:assert": "pnpm test:prod"
1414
},
1515
"dependencies": {
16-
"@next/font": "14.2.25",
16+
"@next/font": "14.2.15",
1717
"@sentry/nextjs": "latest || *",
1818
"@supabase/auth-helpers-react": "0.5.0",
1919
"@supabase/auth-ui-react": "0.4.7",

packages/core/src/integrations/supabase.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// eslint-disable max-lines
1+
/* eslint-disable max-lines */
22
import { logger, isPlainObject } from '../utils-hoist';
33

44
import type { Span, IntegrationFn } from '../types-hoist';

0 commit comments

Comments
 (0)