fix more

sveltejs · Dec 13, 2023 · d31afbb · d31afbb
1 parent b24e8eb
commit d31afbb
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/packages/kit/test/apps/basics/src/hooks.client.js b/packages/kit/test/apps/basics/src/hooks.client.js
@@ -3,8 +3,8 @@ import { env } from '$env/dynamic/public';
 window.PUBLIC_DYNAMIC = env.PUBLIC_DYNAMIC;
 
 /** @type{import("@sveltejs/kit").HandleClientError} */
-export function handleError({ error, event }) {
+export function handleError({ error, event, status, message }) {
 	return event.url.pathname.endsWith('404-fallback')
 		? undefined
-		: { message: /** @type {Error} */ (error).message };
+		: { message: `${/** @type {Error} */ (error).message} (${status} ${message})` };
 }
diff --git a/packages/kit/test/apps/basics/test/client.test.js b/packages/kit/test/apps/basics/test/client.test.js
@@ -83,7 +83,7 @@ test.describe('Load', () => {
 	test('accessing url.hash from load errors and suggests using page store', async ({ page }) => {
 		await page.goto('/load/url-hash#please-dont-send-me-to-load');
 		expect(await page.textContent('#message')).toBe(
-			'This is your custom error page saying: "Cannot access event.url.hash. Consider using `$page.url.hash` inside a component instead"'
+			'This is your custom error page saying: "Cannot access event.url.hash. Consider using `$page.url.hash` inside a component instea (500 Internal Error)"'
 		);
 	});
 

diff --git a/packages/kit/test/apps/basics/test/cross-platform/client.test.js b/packages/kit/test/apps/basics/test/cross-platform/client.test.js
@@ -457,7 +457,7 @@ test.describe.serial('Errors', () => {
 
 		expect(await page.textContent('footer')).toBe('Custom layout');
 		expect(await page.textContent('#message')).toBe(
-			'This is your custom error page saying: "Crashing now"'
+			'This is your custom error page saying: "Crashing now (500 Internal Error)"'
 		);
 	});
 

diff --git a/packages/kit/test/apps/basics/test/cross-platform/test.js b/packages/kit/test/apps/basics/test/cross-platform/test.js
@@ -482,7 +482,7 @@ test.describe('Redirects', () => {
 			expect(page.url()).toBe(`${baseURL}/redirect/loopy/a`);
 			expect(await page.textContent('h1')).toBe('500');
 			expect(await page.textContent('#message')).toBe(
-				'This is your custom error page saying: "Redirect loop"'
+				'This is your custom error page saying: "Redirect loop (500 Internal Error)"'
 			);
 		} else {
 			// there's not a lot we can do to handle server-side redirect loops