Skip to content

Commit fb132a5

Browse files
committed
Switch to using docs.sentry.io instead
1 parent 6b45cb0 commit fb132a5

File tree

2 files changed

+14
-37
lines changed

2 files changed

+14
-37
lines changed

packages/remix/test/integration/app/root.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@ export const loader: LoaderFunction = async ({ request }) => {
4848
case 'returnRedirect':
4949
return redirect('/?type=plain');
5050
case 'throwRedirectToExternal':
51-
throw redirect('https://example.com');
51+
throw redirect(`https://docs.sentry.io`);
5252
case 'returnRedirectToExternal':
53-
return redirect('https://example.com');
53+
return redirect('https://docs.sentry.io');
5454
default: {
5555
return {};
5656
}

packages/remix/test/integration/test/client/root-loader.test.ts

Lines changed: 12 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { type Page, expect, test } from '@playwright/test';
1+
import { type Page, expect, test, chromium } from '@playwright/test';
22

33
async function getRouteData(page: Page): Promise<any> {
44
return page.evaluate('window.__remixContext.state.loaderData').catch(err => {
@@ -20,20 +20,8 @@ async function extractTraceAndBaggageFromMeta(
2020
return { sentryTrace: sentryTraceContent, sentryBaggage: sentryBaggageContent };
2121
}
2222

23-
async function mockExampleRoute(page: Page): Promise<void> {
24-
await page.route('https://example.com/**/*', route => {
25-
return route.fulfill({
26-
status: 200,
27-
body: JSON.stringify({
28-
foo: 'bar',
29-
}),
30-
});
31-
});
32-
}
33-
3423
test('should inject `sentry-trace` and `baggage` into root loader returning an empty object.', async ({ page }) => {
3524
await page.goto('/?type=empty');
36-
3725
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
3826

3927
expect(sentryTrace).toMatch(/.+/);
@@ -49,7 +37,6 @@ test('should inject `sentry-trace` and `baggage` into root loader returning an e
4937

5038
test('should inject `sentry-trace` and `baggage` into root loader returning a plain object.', async ({ page }) => {
5139
await page.goto('/?type=plain');
52-
5340
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
5441

5542
expect(sentryTrace).toMatch(/.+/);
@@ -67,7 +54,6 @@ test('should inject `sentry-trace` and `baggage` into root loader returning a pl
6754

6855
test('should inject `sentry-trace` and `baggage` into root loader returning a `JSON response`.', async ({ page }) => {
6956
await page.goto('/?type=json');
70-
7157
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
7258

7359
expect(sentryTrace).toMatch(/.+/);
@@ -85,7 +71,6 @@ test('should inject `sentry-trace` and `baggage` into root loader returning a `J
8571

8672
test('should inject `sentry-trace` and `baggage` into root loader returning a deferred response', async ({ page }) => {
8773
await page.goto('/?type=defer');
88-
8974
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
9075

9176
expect(sentryTrace).toMatch(/.+/);
@@ -101,7 +86,6 @@ test('should inject `sentry-trace` and `baggage` into root loader returning a de
10186

10287
test('should inject `sentry-trace` and `baggage` into root loader returning `null`.', async ({ page }) => {
10388
await page.goto('/?type=null');
104-
10589
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
10690

10791
expect(sentryTrace).toMatch(/.+/);
@@ -117,7 +101,6 @@ test('should inject `sentry-trace` and `baggage` into root loader returning `nul
117101

118102
test('should inject `sentry-trace` and `baggage` into root loader returning `undefined`.', async ({ page }) => {
119103
await page.goto('/?type=undefined');
120-
121104
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
122105

123106
expect(sentryTrace).toMatch(/.+/);
@@ -134,9 +117,7 @@ test('should inject `sentry-trace` and `baggage` into root loader returning `und
134117
test('should inject `sentry-trace` and `baggage` into root loader throwing a redirection to a plain object.', async ({
135118
page,
136119
}) => {
137-
await mockExampleRoute(page);
138120
await page.goto('/?type=throwRedirect');
139-
140121
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
141122

142123
// We should be successfully redirected to the path.
@@ -155,10 +136,9 @@ test('should inject `sentry-trace` and `baggage` into root loader throwing a red
155136

156137
test('should inject `sentry-trace` and `baggage` into root loader returning a redirection to valid path.', async ({
157138
page,
139+
baseURL,
158140
}) => {
159-
await mockExampleRoute(page);
160-
await page.goto('/?type=returnRedirect');
161-
141+
await page.goto(`${baseURL}/?type=returnRedirect`);
162142
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
163143

164144
// We should be successfully redirected to the path.
@@ -175,27 +155,24 @@ test('should inject `sentry-trace` and `baggage` into root loader returning a re
175155
});
176156
});
177157

178-
test('should return redirect to an external path with no baggage and trace injected.', async ({ page }) => {
179-
await mockExampleRoute(page);
180-
await page.goto('/?type=returnRedirectToExternal');
158+
test('should return redirect to an external path with no baggage and trace injected.', async ({ page,baseURL }) => {
159+
await page.goto(`${baseURL}/?type=returnRedirectToExternal`);
181160

182-
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
161+
expect(page.url()).toEqual(expect.stringContaining('docs.sentry.io'));
183162

184-
// We should be successfully redirected to the external path.
185-
expect(page.url()).toEqual(expect.stringContaining('https://example.com'));
163+
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
186164

187165
expect(sentryTrace).toBeUndefined();
188166
expect(sentryBaggage).toBeUndefined();
189167
});
190168

191-
test('should throw redirect to an external path with no baggage and trace injected.', async ({ page }) => {
192-
await mockExampleRoute(page);
193-
await page.goto('/?type=throwRedirectToExternal');
194-
195-
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
169+
test('should throw redirect to an external path with no baggage and trace injected.', async ({ page, baseURL }) => {
170+
await page.goto(`${baseURL}/?type=throwRedirectToExternal`);
196171

197172
// We should be successfully redirected to the external path.
198-
expect(page.url()).toEqual(expect.stringContaining('https://example.com'));
173+
expect(page.url()).toEqual(expect.stringContaining('docs.sentry.io'));
174+
175+
const { sentryTrace, sentryBaggage } = await extractTraceAndBaggageFromMeta(page);
199176

200177
expect(sentryTrace).toBeUndefined();
201178
expect(sentryBaggage).toBeUndefined();

0 commit comments

Comments
 (0)