test/nuxt/nuxt-island.test.ts (1)

85-122: Ensure test servers close on failure

If one of the assertions throws, the close() call is skipped and the srvx listener can bleed into later tests. Wrapping the mount/expect block in a try/finally (here and in the base-URL variant just below) keeps the suite robust.

-    const { server, port } = await createServer(handler)
-    const wrapper = await mountSuspended(NuxtIsland, {
-      props: {
-        name: 'Test',
-        source: `http://localhost:${port}`,
-      },
-    })
-    expect(wrapper.html()).toMatchInlineSnapshot('"<div>hello world from another server</div>"')
-    await server.close()
+    const { server, port } = await createServer(handler)
+    try {
+      const wrapper = await mountSuspended(NuxtIsland, {
+        props: {
+          name: 'Test',
+          source: `http://localhost:${port}`,
+        },
+      })
+      expect(wrapper.html()).toMatchInlineSnapshot('"<div>hello world from another server</div>"')
+    } finally {
+      await server.close()
+    }
@@
-    const { server, port } = await createServer(handler)
-    const wrapper = await mountSuspended(NuxtIsland, {
-      props: {
-        name: 'Test',
-        source: `http://localhost:${port}/app`,
-      },
-    })
-    expect(wrapper.html()).toMatchInlineSnapshot('"<div>hello world from another server</div>"')
-    expect(url!.startsWith(`http://localhost:${port}/app/__nuxt_island`)).toBe(true)
-    await server.close()
+    const { server, port } = await createServer(handler)
+    try {
+      const wrapper = await mountSuspended(NuxtIsland, {
+        props: {
+          name: 'Test',
+          source: `http://localhost:${port}/app`,
+        },
+      })
+      expect(wrapper.html()).toMatchInlineSnapshot('"<div>hello world from another server</div>"')
+      expect(url!.startsWith(`http://localhost:${port}/app/__nuxt_island`)).toBe(true)
+    } finally {
+      await server.close()
+    }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: CR
PR: nuxt/nuxt#0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-07-18T16:46:07.446Z
Learning: Applies to **/*.{test,spec}.{ts,tsx,js,jsx} : Write unit tests for core functionality using `vitest`

packages/nuxt/src/app/components/nuxt-island.ts (1)

210-212: Preserve response context and align error shape with catch path

Throwing createError on non‑OK is correct. To aid debugging and keep a consistent NuxtError on all failure paths, include the response as the error cause (and optionally the URL), and mirror this in the catch branch that currently throws a plain Error.

Apply this diff to enrich the thrown error on non‑OK:

-      if (!r.ok) {
-        throw createError({ statusCode: r.status, statusMessage: r.statusText })
-      }
+      if (!r.ok) {
+        throw createError({
+          statusCode: r.status,
+          statusMessage: r.statusText,
+          cause: r,
+          data: { url: (r as Response).url }
+        })
+      }

And update the catch branch below to use createError as well (outside the changed hunk, shown for clarity):

} catch (e: any) {
  if (r.status !== 200) {
    throw createError({
      statusCode: r.status,
      statusMessage: r.statusText,
      cause: r,
      data: { url: (r as Response).url }
    })
  }
  throw e
}

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro