-
Notifications
You must be signed in to change notification settings - Fork 0
[Task] DART·KRX 실데이터 수집 integration 테스트 추가 #56
Copy link
Copy link
Open
Labels
area/backendFastAPI backend, repositories, deterministic REST contractsFastAPI backend, repositories, deterministic REST contractskind/testTest, verification, QA readiness workTest, verification, QA readiness workpriority/p2Important follow-up after P1 execution pathImportant follow-up after P1 execution path
Description
Metadata
Metadata
Assignees
Labels
area/backendFastAPI backend, repositories, deterministic REST contractsFastAPI backend, repositories, deterministic REST contractskind/testTest, verification, QA readiness workTest, verification, QA readiness workpriority/p2Important follow-up after P1 execution pathImportant follow-up after P1 execution path
작업 요약
주가(KRX)·공시(DART) 실수집 경로가 테스트로 전혀 보장되지 않는다. 실제 외부 API 호출/응답 파싱을 검증하는 integration 테스트를 추가한다.
배경 (audit 결과)
사용자 질문 "주가·공시 정보를 제대로 가져오는지 실제로 테스트한 게 있나?" 에 대한 조사 결과:
backend/tests/의 수집 관련 테스트는 전부 monkeypatch mock 또는 파라미터 validation 뿐test_external_api_registry.py:59-71—get_trade_info를 mock으로 교체 (실호출 검증 X)test_external_api_registry.py:74-103—fetch_disclosures/fetch_financials를 mock으로 교체backend/collector/krx/trade_info.py의fetch_trade_info_from_kskill/get_trade_info,backend/collector/dart.py의fetch_disclosures/fetch_financials를 실제 HTTP 호출로 검증하는 테스트 없음완료 기준
fetch_disclosures/fetch_financials실호출 integration 테스트 (실제 corp_code로 OpenDART 응답 파싱 검증)get_trade_info/종목검색 실호출 integration 테스트 (k-skill-proxy 응답 파싱 검증)@pytest.mark.integration등 — CI 안정성)비고
.env에 설정됨 / NAVER 키는 빈 값k-skill-proxy.nomadamas.org) 제3자 의존 — rate-limit·다운 가능성 고려한 skip 전략 필요관련 기능 ID
N/A (수집 경로 — A collector:
dart.py,krx/)담당자
A (수집·정규화)
예상 브랜치명
feature/A-collector-integration-tests