From 6e4efbcf1e4c17c25dc0b66ece237d3e02e7dd7b Mon Sep 17 00:00:00 2001
From: samson910022 <samson910022@users.noreply.github.com>
Date: Wed, 9 Sep 2026 19:04:20 +0800
Subject: [PATCH] test: assert fail-open fetch via mock.calls (fix counting
 bug)

---
 tests/proxy-fallback-no-undici.test.js | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/tests/proxy-fallback-no-undici.test.js b/tests/proxy-fallback-no-undici.test.js
index e5fa90a..a88791c 100644
--- a/tests/proxy-fallback-no-undici.test.js
+++ b/tests/proxy-fallback-no-undici.test.js
@@ -10,14 +10,11 @@ const { createProxyPool } = await import('../src/upstream-proxy/pool.js');
 
 describe('proxy pool without undici runtime', () => {
     let savedFetch;
-    let fetchCalls;
+    let fetchMock;
     beforeEach(() => {
-        fetchCalls = [];
         savedFetch = globalThis.fetch;
-        globalThis.fetch = (async (...args) => {
-            fetchCalls.push(args);
-            return new Response('direct-ok');
-        }) as typeof fetch;
+        fetchMock = jest.fn(async () => new Response('direct-ok'));
+        globalThis.fetch = fetchMock;
     });
     afterEach(() => {
         globalThis.fetch = savedFetch;
@@ -28,10 +25,10 @@ describe('proxy pool without undici runtime', () => {
         expect(pool.engage('free')).toContain('socks5://');
         expect(pool.isEngaged()).toBe(true);
         const res = await pool.proxiedFetch('https://example.com/api', {});
-        expect(fetchCalls.length).toBe(1);
+        expect(fetchMock.mock.calls.length).toBe(1);
         expect(await res.text()).toBe('direct-ok');
         // Second call remembers the failure (no repeated import storms).
         await pool.proxiedFetch('https://example.com/api', {});
-        expect(fetchCalls.length).toBe(2);
+        expect(fetchMock.mock.calls.length).toBe(2);
     });
 });