fix(signal_v2-phase3a): V1 KIS env pattern + test isolation fix
config.py: kis_env_type (virtual/real) + KIS_REAL_*/KIS_VIRTUAL_* env variables (V1 호환). kis_app_key/kis_app_secret/kis_account properties auto-select based on env type. main.py: KIS not-configured warning uses descriptive message including env type + expected var name. test_main.py: monkeypatch load_dotenv to no-op + setenv empty string (instead of delenv) — defeats .env re-read on importlib.reload. Pre-existing test_startup_warns_if_webai_api_key_missing also fixed. 33/33 tests pass (was 31/33). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -23,11 +23,15 @@ def test_health_endpoint_returns_status_online(monkeypatch):
|
||||
|
||||
|
||||
def test_startup_warns_if_webai_api_key_missing(monkeypatch, caplog):
|
||||
monkeypatch.delenv("WEBAI_API_KEY", raising=False)
|
||||
# Use setenv with empty string + no-op load_dotenv to defeat .env re-read on reload
|
||||
monkeypatch.setattr("signal_v2.config.load_dotenv", lambda *a, **k: None)
|
||||
monkeypatch.setenv("WEBAI_API_KEY", "")
|
||||
monkeypatch.setenv("STOCK_API_URL", "https://test.stock.local")
|
||||
import importlib
|
||||
from signal_v2 import config as cfg
|
||||
importlib.reload(cfg)
|
||||
# After reload, load_dotenv reference is fresh — re-patch
|
||||
monkeypatch.setattr("signal_v2.config.load_dotenv", lambda *a, **k: None)
|
||||
from signal_v2 import main as main_mod
|
||||
importlib.reload(main_mod)
|
||||
with caplog.at_level(logging.WARNING, logger="signal_v2.main"):
|
||||
@@ -37,17 +41,22 @@ def test_startup_warns_if_webai_api_key_missing(monkeypatch, caplog):
|
||||
|
||||
|
||||
def test_startup_warns_if_kis_app_key_missing(monkeypatch, caplog):
|
||||
"""KIS_APP_KEY 미설정 시 startup WARNING (KIS 호출 disabled)."""
|
||||
"""KIS app_key 미설정 시 startup WARNING (KIS 호출 disabled) — V1 패턴."""
|
||||
monkeypatch.setattr("signal_v2.config.load_dotenv", lambda *a, **k: None)
|
||||
monkeypatch.setenv("STOCK_API_URL", "https://test.stock.local")
|
||||
monkeypatch.setenv("WEBAI_API_KEY", "test-secret")
|
||||
monkeypatch.delenv("KIS_APP_KEY", raising=False)
|
||||
# V1 pattern: kis_env_type=virtual, both virtual keys empty
|
||||
monkeypatch.setenv("KIS_ENV_TYPE", "virtual")
|
||||
monkeypatch.setenv("KIS_VIRTUAL_APP_KEY", "")
|
||||
monkeypatch.setenv("KIS_REAL_APP_KEY", "")
|
||||
|
||||
import importlib
|
||||
from signal_v2 import config as cfg
|
||||
importlib.reload(cfg)
|
||||
monkeypatch.setattr("signal_v2.config.load_dotenv", lambda *a, **k: None)
|
||||
from signal_v2 import main as main_mod
|
||||
importlib.reload(main_mod)
|
||||
with caplog.at_level(logging.WARNING, logger="signal_v2.main"):
|
||||
with TestClient(main_mod.app) as client:
|
||||
client.get("/health")
|
||||
assert any("KIS_APP_KEY" in rec.message for rec in caplog.records)
|
||||
assert any("KIS" in rec.message and "app_key" in rec.message.lower() for rec in caplog.records)
|
||||
|
||||
Reference in New Issue
Block a user