Skip to content

Commit 0952fed

Browse files
committed
fix tests
1 parent c42a9d8 commit 0952fed

File tree

3 files changed

+13
-12
lines changed
  • dev-packages/browser-integration-tests/suites/tracing/metrics

3 files changed

+13
-12
lines changed

dev-packages/browser-integration-tests/suites/tracing/metrics/handlers-lcp/test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,9 @@ sentryTest(
2929
expect(eventData.measurements).toBeDefined();
3030
expect(eventData.measurements?.lcp?.value).toBeDefined();
3131

32-
expect(eventData.contexts.trace?.data?.['lcp.element']).toBe('body > img');
33-
expect(eventData.contexts.trace?.data?.['lcp.size']).toBe(107400);
34-
expect(eventData.contexts.trace?.data?.['lcp.url']).toBe('https://example.com/path/to/image.png');
32+
expect(eventData.contexts?.trace?.data?.['lcp.element']).toBe('body > img');
33+
expect(eventData.contexts?.trace?.data?.['lcp.size']).toBe(107400);
34+
expect(eventData.contexts?.trace?.data?.['lcp.url']).toBe('https://example.com/path/to/image.png');
3535

3636
const lcp = await (await page.waitForFunction('window._LCP')).jsonValue();
3737
const lcp2 = await (await page.waitForFunction('window._LCP2')).jsonValue();

dev-packages/browser-integration-tests/suites/tracing/metrics/web-vitals-cls/test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ sentryTest('should capture a "GOOD" CLS vital with its source(s).', async ({ get
2323
expect(eventData.measurements?.cls?.value).toBeGreaterThan(0.03);
2424
expect(eventData.measurements?.cls?.value).toBeLessThan(0.07);
2525

26-
expect(eventData.contexts.trace?.data?.['cls.source.1']).toBe('body > div#content > p#partial');
26+
expect(eventData.contexts?.trace?.data?.['cls.source.1']).toBe('body > div#content > p#partial');
2727
});
2828

2929
sentryTest('should capture a "MEH" CLS vital with its source(s).', async ({ getLocalTestPath, page }) => {
@@ -37,7 +37,7 @@ sentryTest('should capture a "MEH" CLS vital with its source(s).', async ({ getL
3737
expect(eventData.measurements?.cls?.value).toBeGreaterThan(0.18);
3838
expect(eventData.measurements?.cls?.value).toBeLessThan(0.23);
3939

40-
expect(eventData.contexts.trace?.data?.['cls.source.1']).toBe('body > div#content > p');
40+
expect(eventData.contexts?.trace?.data?.['cls.source.1']).toBe('body > div#content > p');
4141
});
4242

4343
sentryTest('should capture a "POOR" CLS vital with its source(s).', async ({ getLocalTestPath, page }) => {
@@ -50,5 +50,5 @@ sentryTest('should capture a "POOR" CLS vital with its source(s).', async ({ get
5050
// Flakey value dependent on timings -> we check for a range
5151
expect(eventData.measurements?.cls?.value).toBeGreaterThan(0.34);
5252
expect(eventData.measurements?.cls?.value).toBeLessThan(0.36);
53-
expect(eventData.contexts.trace?.data?.['cls.source.1']).toBe('body > div#content > p');
53+
expect(eventData.contexts?.trace?.data?.['cls.source.1']).toBe('body > div#content > p');
5454
});

dev-packages/browser-integration-tests/suites/tracing/metrics/web-vitals-lcp/test.ts

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,10 @@ sentryTest('should capture a LCP vital with element details.', async ({ browserN
1010
sentryTest.skip();
1111
}
1212

13-
await page.route('**/path/to/image.png', (route: Route) =>
14-
route.fulfill({ path: `${__dirname}/assets/sentry-logo-600x179.png` }),
15-
);
13+
page.route('**', route => route.continue());
14+
page.route('**/path/to/image.png', async (route: Route) => {
15+
return route.fulfill({ path: `${__dirname}/assets/sentry-logo-600x179.png` });
16+
});
1617

1718
const url = await getLocalTestPath({ testDir: __dirname });
1819
const [eventData] = await Promise.all([
@@ -24,7 +25,7 @@ sentryTest('should capture a LCP vital with element details.', async ({ browserN
2425
expect(eventData.measurements).toBeDefined();
2526
expect(eventData.measurements?.lcp?.value).toBeDefined();
2627

27-
expect(eventData.contexts.trace?.data?.['lcp.element']).toBe('body > img');
28-
expect(eventData.contexts.trace?.data?.['lcp.size']).toBe(107400);
29-
expect(eventData.contexts.trace?.data?.['lcp.url']).toBe('https://example.com/path/to/image.png');
28+
expect(eventData.contexts?.trace?.data?.['lcp.element']).toBe('body > img');
29+
expect(eventData.contexts?.trace?.data?.['lcp.size']).toBe(107400);
30+
expect(eventData.contexts?.trace?.data?.['lcp.url']).toBe('https://example.com/path/to/image.png');
3031
});

0 commit comments

Comments
 (0)