From e0b74d4838d9f592f56f14fb41a61345cfaf5c8b Mon Sep 17 00:00:00 2001 From: Nic Polumeyv <162764842+Nic-Polumeyv@users.noreply.github.com> Date: Sun, 20 Sep 2026 23:59:20 -0400 Subject: [PATCH] test: check the edge bundle for treeshaking in the build test --- packages/adapter-netlify/test/apps/edge/build.test.js | 5 +++++ packages/adapter-netlify/test/apps/edge/test/test.js | 10 ++-------- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/packages/adapter-netlify/test/apps/edge/build.test.js b/packages/adapter-netlify/test/apps/edge/build.test.js index 93a1b901ab41..0803ca86a838 100644 --- a/packages/adapter-netlify/test/apps/edge/build.test.js +++ b/packages/adapter-netlify/test/apps/edge/build.test.js @@ -30,3 +30,8 @@ test('_redirects are copied to publish directory', () => { ); expect(redirects).toContain('/redirect-me /greeting/redirected 301'); }); + +test('treeshakes component from the server bundle if SSR is turned off', () => { + const edge_function = fs.readFileSync(edge_function_path, 'utf-8'); + expect(edge_function).not.toContain('this should never appear in the server bundle'); +}); diff --git a/packages/adapter-netlify/test/apps/edge/test/test.js b/packages/adapter-netlify/test/apps/edge/test/test.js index 6a6377d19567..35c80a26e652 100644 --- a/packages/adapter-netlify/test/apps/edge/test/test.js +++ b/packages/adapter-netlify/test/apps/edge/test/test.js @@ -29,13 +29,7 @@ test('read from $app/server works', async ({ request }) => { expect(await response.text()).toBe(content); }); -test('treeshakes component from the server bundle if SSR is turned off', async ({ page }) => { +test('page renders on the client if SSR is turned off', async ({ page }) => { await page.goto('/treeshake-server'); - const component_text = 'this should never appear in the server bundle'; - const server_bundle = fs.readFileSync( - path.resolve(import.meta.dirname, '../.netlify/v1/edge-functions/sveltekit-render.js'), - 'utf-8' - ); - expect(server_bundle).not.toContain(component_text); - await expect(page.locator('p')).toHaveText(component_text); + await expect(page.locator('p')).toHaveText('this should never appear in the server bundle'); });