From 184ec250fd17f914bff1dbc729165ebfcebc9246 Mon Sep 17 00:00:00 2001
From: "Md.Nahin Alam" <82433419+alamnahin@users.noreply.github.com>
Date: Fri, 3 Apr 2026 04:18:20 +0600
Subject: [PATCH] test(router): scope FAKE_KEY via pytest monkeypatch fixture

---
 test_smart_router.py | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/test_smart_router.py b/test_smart_router.py
index fac31c85..651b94f1 100644
--- a/test_smart_router.py
+++ b/test_smart_router.py
@@ -7,13 +7,17 @@ Run: pytest test_smart_router.py -v
 
 import pytest
 import asyncio
-import os
 from unittest.mock import AsyncMock, MagicMock, patch
 from smart_router import SmartRouter, Provider
 
 
 # ── Fixtures ──────────────────────────────────────────────────────────────────
 
+
+@pytest.fixture(autouse=True)
+def fake_api_key(monkeypatch):
+    monkeypatch.setenv("FAKE_KEY", "test-key")
+
 def make_provider(name, healthy=True, configured=True,
                   latency=100.0, cost=0.002, errors=0, requests=0):
     p = Provider(
@@ -28,9 +32,7 @@ def make_provider(name, healthy=True, configured=True,
     p.avg_latency_ms = latency
     p.error_count = errors
     p.request_count = requests
-    if configured:
-        os.environ.setdefault("FAKE_KEY", "test-key")
-    else:
+    if not configured:
         p.api_key_env = ""  # makes is_configured False for non-ollama
     return p