test(gateway): replace redundant Windows passthrough test with platform-conditional guard test

SARAMALI15792 · SARAMALI15792 · commit 386dd4e56b6c · 2026-04-21T17:09:34.000+05:00
The previous 'skips ::1 on Windows for non-loopback host passthrough' test
used 0.0.0.0 as bindHost, which exits via the pre-existing non-loopback
early-return before the win32 guard is ever reached. It would pass even
if the Windows guard were removed entirely.

Replace with a test that verifies the guard is win32-only: on darwin,
resolveGatewayListenHosts('127.0.0.1') still returns ['127.0.0.1','::1']
and calls canBindToHost, confirming dual-stack is preserved on non-Windows.
diff --git a/src/gateway/net.test.ts b/src/gateway/net.test.ts
@@ -331,12 +331,12 @@ describe("resolveGatewayListenHosts", () => {
     expect(canBindToHost).not.toHaveBeenCalled();
   });
 
-  it("skips ::1 on Windows for non-loopback host passthrough", async () => {
-    vi.spyOn(process, "platform", "get").mockReturnValue("win32");
+  it("still includes ::1 on non-Windows when IPv6 is bindable", async () => {
+    vi.spyOn(process, "platform", "get").mockReturnValue("darwin");
     const canBindToHost = vi.fn().mockResolvedValue(true);
-    const hosts = await resolveGatewayListenHosts("0.0.0.0", { canBindToHost });
-    expect(hosts).toEqual(["0.0.0.0"]);
-    expect(canBindToHost).not.toHaveBeenCalled();
+    const hosts = await resolveGatewayListenHosts("127.0.0.1", { canBindToHost });
+    expect(hosts).toEqual(["127.0.0.1", "::1"]);
+    expect(canBindToHost).toHaveBeenCalledWith("::1");
   });
 });
 
