fix: 매매 파라미터 3중 소스 정합 + 리스크 가드 복원 + 죽은 코드 2,695줄 제거
## 배경 "전략이 혼란스럽고 현재와 맞지 않는다"를 조사한 결과, 전략 설계가 아니라 전략을 검증했다는 도구와 파라미터 소스가 어긋나 있었다. ## 파라미터 소스 단일화 - 우선순위가 adaptive > strategy_cfg(DB) > config.py 인데 문서는 맨 마지막인 config.py 를 기술하고 있었다. 11곳이 불일치(BULL RSI 46 vs DB 30, SL -7% vs -6%). - DEFAULT_PARAMS 를 config.py 에 정합 + alembic 0012 로 기존 DB 행 갱신 (운영자가 UI 로 바꾼 값은 보존, 옛 기본값과 같은 행만 갱신, 테이블 가드·멱등) ## 백테스트를 실전 로직에 정합 - calculate_adx_simple 이 정확히 period 배(14.0×) 발산 → Aggressive 게이트 `ADX>25` 가 전 레짐 100% 통과. 전환 빈도 5.6%→2.0%(-65%), 성과 상방 편향 - 백테스트 RSI 는 SMA, 실전은 Wilder → 진입 판정 10~11% 불일치 - us_backtest_engine RSI 1봉 지연 제거 (룩어헤드 없음을 절단 테스트로 증명) - backend/backtest_results/ 25건은 폐기 대상 (§8.4) ## 리스크 가드 복원 - 4% 절대 상한이 try 블록 안에 있어 예외 시 적응형 6~7% 가 통과 → try 밖으로 - 청산을 매수보다 먼저 평가 (보유 종목이 추천에 들면 손절 대신 물타기가 나갔다) - 리스크 가드 4개를 매수 전용화 → MDD -10%·보유한도 도달 시에도 손절 작동 - MDD 갱신 5회 연속 실패 시 신규 진입 차단 fail-safe (청산은 계속 허용) - 스윙 청산이 DT 보유분까지 매도하던 문제, sell_pct 기준 혼재 정정 ## 죽은 코드 제거 (2,695줄) - short_term_strategy, kosdaq_scanner, app/strategy/ 패키지 - app/trading/ + korea_investment_api_adapter (real_market_api 정본은 korea_investment_api.py:1482, adapter 는 import 0건) ## 지표·사이징 단일화 (수치 무변경 증명) - indicators_core.py: 3,274건 비교 최대 오차 0.000e+00 - sizing_core.py: 42,105건 비교 불일치 0 - 미국 전략 상수 13개 config 편입 (hex 까지 동일, 3,168 케이스 전수 비교) ## 테스트 안전 (실 KIS 호출 차단) - 유닛 테스트가 실제 KIS 에 매도 주문을 2회 전송하고 있었다. 리팩터링이 close_stale_positions 의 대상을 _execute_exit → _execute_exit_stale 로 바꿨는데 목이 전자에 걸려 빗나갔다. test_api_endpoints 도 캡처된 싱글톤 참조 누수로 실제 잔고 조회를 타 141초를 소비했다. - backend/tests/conftest.py 신설: 주문·토큰·시세·잔고를 호출 즉시 예외로 차단. 예외는 BaseException 파생(프로덕션 except Exception 폴백에 삼켜지지 않게), 클래스 메서드에 적용(캡처된 참조까지 덮음), DATABASE_URL `_test` 가드 강제 - 전체 유닛 스위트: 365초/6 failed → 11.8초/0 failed (소켓 차단 하 559 passed) ## 신규 (둘 다 플래그 OFF, 백테스트 미검증) - range_trading_strategy.py — SIDEWAYS 담당 전략이 없던 빈 칸. 미배선 - check_bear_buy_signal_v2 — BEAR 4중 AND(발화율 2.3%) → 점수제(12.4%) ## 문서 - CLAUDE.md 를 실제 코드 기준으로 재작성. §0 파라미터 우선순위, §4.1 테스트 규약, §6.4 문서화되지 않았던 진입 차단 게이트 9종, §8.4 백테스트 폐기 근거, §8.5 미해결 19항목 - 루트 md 6개 → 2개(docs/ 이동), 중복 배치 문서 병합, 스테일 worktree 제거 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -139,6 +139,41 @@ from app.models.models import Portfolio, TradeHistory
|
||||
# ---------------------------------------------------------------------------
|
||||
# Fixtures
|
||||
# ---------------------------------------------------------------------------
|
||||
@pytest.fixture(autouse=True)
|
||||
def bind_mock_kis_to_captured_refs():
|
||||
"""`_mock_kis`를 이미 실제 싱글톤을 캡처한 모듈들에까지 강제 바인딩.
|
||||
|
||||
파일 상단의 `patch("app.services.korea_investment_api.korea_investment_api", _mock_kis)`는
|
||||
**이 모듈이 가장 먼저 import될 때만** 통한다. 전체 스위트 실행에서는 알파벳 순으로 앞선
|
||||
테스트 모듈(`test_adaptive_strategy` 등)이 이미 `app.main`을 import해
|
||||
`app/api/endpoints/portfolio.py:20` 의 `from app.services.korea_investment_api import
|
||||
korea_investment_api` 가 **실제 싱글톤을 자기 모듈 네임스페이스에 캡처**해 둔 상태다.
|
||||
그래서 소스 모듈 속성만 갈아끼운 상단 patch가 새고, `/api/v1/portfolio/summary` 가
|
||||
라이브 `get_balance()` → `_get_access_token()` → `_handle_rate_limit()` 의 70초
|
||||
`asyncio.sleep` 까지 내려갔다.
|
||||
(2026-09-27 측정: 단독 실행 2.9초 / 전체 스위트에서 이 파일 2건이 71초·142초 + 모니터링 5건 실패)
|
||||
|
||||
`tests/conftest.py` 의 autouse 차단 픽스처가 클래스 메서드를 "호출 시 실패"로 덮으므로,
|
||||
복구도 **픽스처 단계**에서 해야 한다(부모 conftest autouse → 모듈 autouse 순서이므로
|
||||
이쪽이 나중에 적용되어 이긴다).
|
||||
"""
|
||||
from app.services.korea_investment_api import KoreaInvestmentAPI
|
||||
|
||||
patches = [
|
||||
patch.object(mod, "korea_investment_api", _mock_kis)
|
||||
for name, mod in list(sys.modules.items())
|
||||
if name.startswith("app.")
|
||||
and isinstance(getattr(mod, "korea_investment_api", None), KoreaInvestmentAPI)
|
||||
]
|
||||
for p in patches:
|
||||
p.start()
|
||||
try:
|
||||
yield
|
||||
finally:
|
||||
for p in reversed(patches):
|
||||
p.stop()
|
||||
|
||||
|
||||
@pytest_asyncio.fixture(autouse=True)
|
||||
async def setup_database():
|
||||
"""각 테스트 전에 테이블 생성, 테스트 후 모든 데이터 정리"""
|
||||
@@ -159,25 +194,39 @@ async def db_session():
|
||||
|
||||
@pytest_asyncio.fixture
|
||||
async def client():
|
||||
"""httpx AsyncClient (ASGITransport 기반)
|
||||
"""httpx AsyncClient (ASGITransport 기반) — 실제 X-API-Key 헤더로 인증.
|
||||
|
||||
require_api_key 의존성을 테스트 전용 noop 함수로 대체하여
|
||||
.env SECRET_KEY 값에 무관하게 인증을 통과시킴.
|
||||
[2026-09-27 수정] 선재 결함: 이 픽스처는 `require_api_key` 의존성만
|
||||
noop("test-key" 반환)으로 오버라이드했다. 그런데 portfolio/trades 라우터에는
|
||||
멀티테넌트 레이어(`app/api/__init__.py` → `Depends(deny_non_owner_session_when_mt)`)가
|
||||
추가돼 있고, 그 체인은 `get_current_user`(app/core/tenant_auth.py)에서
|
||||
require_api_key 의 **반환값**을 API 키 또는 세션 토큰으로 다시 해석한다.
|
||||
"test-key" 는 둘 다 아니므로 사용자 해석 실패 → 401 (20건 실패의 단일 원인).
|
||||
|
||||
→ 인증을 무력화하지 않고, 실제 운영과 동일하게 X-API-Key 헤더에 유효한 키를
|
||||
실어 보낸다(`_get_api_key()` = API_KEY 우선, 없으면 SECRET_KEY 폴백).
|
||||
이렇게 하면 require_api_key → get_current_user(API 키 호출자 = legacy owner)
|
||||
→ deny_non_owner_session_when_mt(MULTI_TENANT_ENABLED=False 이므로 통과)
|
||||
전체 체인이 실제로 실행된다. 의존성 오버라이드는 get_db 뿐.
|
||||
"""
|
||||
from app.core.security import require_api_key
|
||||
from app.core.security import _get_api_key
|
||||
|
||||
async def _bypass_api_key():
|
||||
return "test-key"
|
||||
transport = ASGITransport(app=app)
|
||||
async with httpx.AsyncClient(
|
||||
transport=transport,
|
||||
base_url="http://test",
|
||||
headers={"X-API-Key": _get_api_key()},
|
||||
) as c:
|
||||
yield c
|
||||
|
||||
app.dependency_overrides[require_api_key] = _bypass_api_key
|
||||
|
||||
@pytest_asyncio.fixture
|
||||
async def unauthenticated_client():
|
||||
"""인증 헤더가 없는 클라이언트 — require_api_key 가 실제로 걸려 있는지 검증용."""
|
||||
transport = ASGITransport(app=app)
|
||||
async with httpx.AsyncClient(transport=transport, base_url="http://test") as c:
|
||||
yield c
|
||||
|
||||
# 테스트 완료 후 오버라이드 제거
|
||||
app.dependency_overrides.pop(require_api_key, None)
|
||||
|
||||
|
||||
def _reset_portfolio_cache(**overrides):
|
||||
"""포트폴리오 캐시를 테스트에 맞게 초기화하는 헬퍼"""
|
||||
@@ -706,3 +755,40 @@ class TestTradeHistoryEndpoints:
|
||||
response2 = await client.get("/api/v1/trades", params={"status": "FAILED"})
|
||||
assert response2.status_code == 200
|
||||
assert len(response2.json()) == 0
|
||||
|
||||
|
||||
# ===========================================================================
|
||||
# 4. 인증 강제 테스트 (require_api_key 가 무력화되지 않았음을 보장)
|
||||
# ===========================================================================
|
||||
class TestApiKeyEnforcement:
|
||||
"""[2026-09-27 추가] client 픽스처가 헤더 인증 방식으로 바뀌었으므로,
|
||||
인증 자체가 살아 있는지를 별도로 고정한다. 과거처럼 require_api_key 를
|
||||
dependency_overrides 로 우회하면 이 테스트가 먼저 깨진다.
|
||||
"""
|
||||
|
||||
@pytest.mark.parametrize(
|
||||
"method,path",
|
||||
[
|
||||
("get", "/api/v1/portfolio"),
|
||||
("get", "/api/v1/portfolio/summary"),
|
||||
("get", "/api/v1/trades"),
|
||||
("get", "/api/v1/trades/stats/summary"),
|
||||
("delete", "/api/v1/trades/pending"),
|
||||
("get", "/api/v1/monitoring/health"),
|
||||
],
|
||||
)
|
||||
@pytest.mark.asyncio
|
||||
async def test_requires_api_key(
|
||||
self, unauthenticated_client: httpx.AsyncClient, method: str, path: str
|
||||
):
|
||||
"""X-API-Key 헤더가 없으면 401"""
|
||||
response = await getattr(unauthenticated_client, method)(path)
|
||||
assert response.status_code == 401, f"{method.upper()} {path} 가 인증 없이 통과했다"
|
||||
|
||||
@pytest.mark.asyncio
|
||||
async def test_rejects_invalid_api_key(self, unauthenticated_client: httpx.AsyncClient):
|
||||
"""잘못된 키도 401 (세션 토큰으로도 해석 불가)"""
|
||||
response = await unauthenticated_client.get(
|
||||
"/api/v1/trades", headers={"X-API-Key": "not-a-valid-key"}
|
||||
)
|
||||
assert response.status_code == 401
|
||||
|
||||
Reference in New Issue
Block a user