fix: green CI on main

- drop never-read `capturedUpstreamContentType` to satisfy unused-vars lint
- assert pan-on-open guard via `panBy` (not rAF spy); the Nuxt/happy-dom
  test env schedules a stray rAF during mount, making the count flaky

Author: harlan-zw

test/nuxt-runtime/google-maps-overlay-view.nuxt.test.ts

@@ -354,44 +354,52 @@ describe('scriptGoogleMapsOverlayView', () => {
     // resolved a position. That caused unexpected map panning on initial mount
     // and remount. The fix gates the rAF scheduling on `open !== false` and
     // re-checks `open` + `overlayPosition` inside the callback before panning.
+    //
+    // We mock `requestAnimationFrame` to fire its callback synchronously so the
+    // panMapToFitOverlay → panBy path is exercised within the test, then assert
+    // on `panBy` (the actual user-visible side effect). Spying on rAF directly
+    // is unreliable in the Nuxt/happy-dom environment, which itself schedules
+    // background rAF calls during mount.
 
     let rafSpy: ReturnType<typeof vi.spyOn>
 
     beforeEach(() => {
-      rafSpy = vi.spyOn(globalThis, 'requestAnimationFrame')
+      rafSpy = vi.spyOn(globalThis, 'requestAnimationFrame').mockImplementation(((cb: FrameRequestCallback) => {
+        cb(0)
+        return 0
+      }) as typeof globalThis.requestAnimationFrame)
     })
 
     afterEach(() => {
       rafSpy.mockRestore()
     })
 
-    it('does not schedule pan-on-open when overlay starts closed (defaultOpen=false)', async () => {
+    it('does not pan when overlay starts closed (defaultOpen=false)', async () => {
       const mocks = createOverlayMocks()
       await mountOverlay(
         { position: { lat: 10, lng: 20 }, defaultOpen: false },
         mocks,
       )
 
-      expect(rafSpy).not.toHaveBeenCalled()
       expect(mocks.mockMap.panBy).not.toHaveBeenCalled()
     })
 
-    it('does not schedule pan-on-open when controlled :open is false on mount', async () => {
+    it('does not pan when controlled :open is false on mount', async () => {
       const mocks = createOverlayMocks()
       await mountOverlay(
         { position: { lat: 10, lng: 20 }, open: false },
         mocks,
       )
 
-      expect(rafSpy).not.toHaveBeenCalled()
       expect(mocks.mockMap.panBy).not.toHaveBeenCalled()
     })
 
     it('schedules pan-on-open when overlay starts open with a position', async () => {
       const mocks = createOverlayMocks()
       await mountOverlay({ position: { lat: 10, lng: 20 } }, mocks)
 
-      // The guard allows the rAF to be scheduled when the happy path applies
+      // Happy path: rAF callback fires synchronously via the mock, then
+      // panMapToFitOverlay runs panBy off the (mock) bounding rects.
       expect(rafSpy).toHaveBeenCalled()
     })
 
@@ -402,7 +410,6 @@ describe('scriptGoogleMapsOverlayView', () => {
         mocks,
       )
 
-      expect(rafSpy).not.toHaveBeenCalled()
       expect(mocks.mockMap.panBy).not.toHaveBeenCalled()
     })
   })

test/unit/proxy-handler-binary.test.ts

@@ -1,7 +1,7 @@
 import type { Server } from 'node:http'
 import { createServer } from 'node:http'
 import { gzipSync } from 'node:zlib'
-import { createApp, defineEventHandler, getHeader, getHeaders, readBody, readRawBody, toNodeListener } from 'h3'
+import { createApp, defineEventHandler, getHeaders, readBody, readRawBody, toNodeListener } from 'h3'
 import { afterAll, beforeAll, beforeEach, describe, expect, it } from 'vitest'
 
 /**
@@ -19,15 +19,12 @@ describe('proxy handler - compressed binary payloads (#618)', () => {
   let proxyPort: number
   let capturedUpstreamBody: Buffer | null = null
 
-  let capturedUpstreamContentType: string | undefined
-
   beforeAll(async () => {
     // Mock upstream: captures raw request bytes exactly as received
     const upstreamApp = createApp()
     upstreamApp.use('/', defineEventHandler(async (event) => {
       const raw = await readRawBody(event, false)
       capturedUpstreamBody = raw ? Buffer.from(raw) : Buffer.alloc(0)
-      capturedUpstreamContentType = getHeader(event, 'content-type')
       return { status: 1 }
     }))
 
@@ -86,7 +83,6 @@ describe('proxy handler - compressed binary payloads (#618)', () => {
 
   beforeEach(() => {
     capturedUpstreamBody = null
-    capturedUpstreamContentType = undefined
   })
 
   afterAll(() => {