From 02345e4c8cd0ac4cb8f0c4e5ff15dcfacb8130eb Mon Sep 17 00:00:00 2001
From: MizukiMachine <negichanfunakun@gmail.com>
Date: Fri, 20 Feb 2026 23:57:42 +0900
Subject: [PATCH] test: address review feedback for WSL2 detection tests

- Move afterEach to top-level describe block for proper isolation
- Set default return value false for isWSL2Sync mock

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/telegram/network-config.test.ts | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/telegram/network-config.test.ts b/src/telegram/network-config.test.ts
index 79c635a3673..89d91a7b55b 100644
--- a/src/telegram/network-config.test.ts
+++ b/src/telegram/network-config.test.ts
@@ -3,12 +3,16 @@ import { resolveTelegramAutoSelectFamilyDecision } from "./network-config.js";
 
 // Mock isWSL2Sync at the top level
 vi.mock("../infra/wsl.js", () => ({
-  isWSL2Sync: vi.fn(),
+  isWSL2Sync: vi.fn(() => false),
 }));
 
 import { isWSL2Sync } from "../infra/wsl.js";
 
 describe("resolveTelegramAutoSelectFamilyDecision", () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
   it("prefers env enable over env disable", () => {
     const decision = resolveTelegramAutoSelectFamilyDecision({
       env: {
@@ -78,10 +82,6 @@ describe("resolveTelegramAutoSelectFamilyDecision", () => {
   });
 
   describe("WSL2 detection", () => {
-    afterEach(() => {
-      vi.restoreAllMocks();
-    });
-
     it("disables autoSelectFamily on WSL2", () => {
       vi.mocked(isWSL2Sync).mockReturnValue(true);
       const decision = resolveTelegramAutoSelectFamilyDecision({ env: {}, nodeMajor: 22 });