Skip to content

Commit 1167436

Browse files
authored
test(remix): Skip flaky firefox integration tests (#8506)
1 parent 87bdcba commit 1167436

File tree

1 file changed

+34
-3
lines changed

1 file changed

+34
-3
lines changed

packages/remix/test/integration/test/client/meta-tags.test.ts

Lines changed: 34 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,13 @@ import { test, expect } from '@playwright/test';
22
import { getFirstSentryEnvelopeRequest } from './utils/helpers';
33
import { Event } from '@sentry/types';
44

5-
test('should inject `sentry-trace` and `baggage` meta tags inside the root page.', async ({ page }) => {
5+
test('should inject `sentry-trace` and `baggage` meta tags inside the root page.', async ({ page, browserName }) => {
6+
// This test is flaky on firefox
7+
// https://github.com/getsentry/sentry-javascript/issues/8398
8+
if (browserName === 'firefox') {
9+
test.skip();
10+
}
11+
612
await page.goto('/');
713

814
const sentryTraceTag = await page.$('meta[name="sentry-trace"]');
@@ -16,7 +22,16 @@ test('should inject `sentry-trace` and `baggage` meta tags inside the root page.
1622
expect(sentryBaggageContent).toEqual(expect.any(String));
1723
});
1824

19-
test('should inject `sentry-trace` and `baggage` meta tags inside a parameterized route.', async ({ page }) => {
25+
test('should inject `sentry-trace` and `baggage` meta tags inside a parameterized route.', async ({
26+
page,
27+
browserName,
28+
}) => {
29+
// This test is flaky on firefox
30+
// https://github.com/getsentry/sentry-javascript/issues/8398
31+
if (browserName === 'firefox') {
32+
test.skip();
33+
}
34+
2035
await page.goto('/loader-json-response/0');
2136

2237
const sentryTraceTag = await page.$('meta[name="sentry-trace"]');
@@ -30,7 +45,16 @@ test('should inject `sentry-trace` and `baggage` meta tags inside a parameterize
3045
expect(sentryBaggageContent).toEqual(expect.any(String));
3146
});
3247

33-
test('should send transactions with corresponding `sentry-trace` and `baggage` inside root page', async ({ page }) => {
48+
test('should send transactions with corresponding `sentry-trace` and `baggage` inside root page', async ({
49+
page,
50+
browserName,
51+
}) => {
52+
// This test is flaky on firefox
53+
// https://github.com/getsentry/sentry-javascript/issues/8398
54+
if (browserName === 'firefox') {
55+
test.skip();
56+
}
57+
3458
const envelope = await getFirstSentryEnvelopeRequest<Event>(page, '/');
3559

3660
const sentryTraceTag = await page.$('meta[name="sentry-trace"]');
@@ -47,7 +71,14 @@ test('should send transactions with corresponding `sentry-trace` and `baggage` i
4771

4872
test('should send transactions with corresponding `sentry-trace` and `baggage` inside a parameterized route', async ({
4973
page,
74+
browserName,
5075
}) => {
76+
// This test is flaky on firefox
77+
// https://github.com/getsentry/sentry-javascript/issues/8398
78+
if (browserName === 'firefox') {
79+
test.skip();
80+
}
81+
5182
const envelope = await getFirstSentryEnvelopeRequest<Event>(page, '/loader-json-response/0');
5283

5384
const sentryTraceTag = await page.$('meta[name="sentry-trace"]');

0 commit comments

Comments
 (0)