test: deps.get_session 회귀 + /api/watchlist 통합 테스트 (#19) #24

Merged
xhh merged 1 commit from issue-19-deps-regression-test into main 2026-04-22 02:39:16 +09:00
Owner

요약

PR #17 에서 발견된 deps.get_session 의 commit 누락 잠복 버그 재발을 막는 회귀 테스트. 동시에 /api/watchlist 라우터의 주요 동작(CRUD, 필터, idempotent delete, 404)을 커버.

핵심

test_post_then_get_persists — POST 후 별도 요청으로 GET 을 수행. 빨간불이면 즉시 deps.get_session commit 회귀를 의심하도록 본문 주석에 명시.

결과

uv run pytest13 passed (smoke 3 + watchlist 10).

관례

이후 mutation 포함 라우터(Reports #2, Collect #4, Coverage #21 등) 작성 시 "POST 후 별도 요청 GET persist" 케이스를 최소 1건 포함할 것을 커밋 메시지에 명시.

Closes #19

## 요약 PR #17 에서 발견된 deps.get_session 의 commit 누락 잠복 버그 재발을 막는 회귀 테스트. 동시에 /api/watchlist 라우터의 주요 동작(CRUD, 필터, idempotent delete, 404)을 커버. ## 핵심 `test_post_then_get_persists` — POST 후 별도 요청으로 GET 을 수행. 빨간불이면 즉시 deps.get_session commit 회귀를 의심하도록 본문 주석에 명시. ## 결과 `uv run pytest` → **13 passed** (smoke 3 + watchlist 10). ## 관례 이후 mutation 포함 라우터(Reports #2, Collect #4, Coverage #21 등) 작성 시 "POST 후 별도 요청 GET persist" 케이스를 최소 1건 포함할 것을 커밋 메시지에 명시. Closes #19
deps.get_session 트랜잭션 회귀 테스트 + /api/watchlist 통합 테스트 (#19)
All checks were successful
Tests (PR) / pytest (pull_request) Successful in 37s
f87c2d3fbb
PR #17 에서 발견된 "yield 후 commit 누락" 잠복 버그의 회귀 방지가 주 목적. test_post_then_get_persists 가 빨간불이면 즉시 deps.get_session 의 commit 이 빠진 것으로 진단 가능.

## 테스트 케이스 (10건, parametrize 포함)

- test_empty_list_initially: 빈 목록 smoke
- test_post_then_get_persists: **핵심 회귀 테스트** — POST 후 별도 요청으로 GET 해서 동일 레코드 확인
- test_post_duplicate_returns_409
- test_patch_updates_fields + persist 재확인
- test_delete_is_soft_and_idempotent + active_only 필터 양방향
- test_missing_ticker_returns_404 (GET/PATCH/DELETE 파라미터화)
- test_ticker_is_uppercased (라우터 라우팅/저장 모두 대문자)
- test_market_filter

## 결과
로컬 uv run pytest -> 13 passed (4.28s). #18 smoke 3건 + 이 파일 10건.

## 향후 동일 패턴 권고
- Reports(#2), Collect(#4), Meta/Coverage(#21) 등 mutation 포함 라우터는 "POST 후 별도 요청 GET persist" 확인 케이스를 최소 한 개 포함할 것. deps 트랜잭션 관리 회귀를 즉시 감지하는 가장 저렴한 수단.

Closes #19

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
xhh merged commit 373f633dd5 into main 2026-04-22 02:39:16 +09:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
xhh/financial-data-platform!24
No description provided.