fix: daily_review 테스트 날짜 불일치 수정 (#129) #130

agentson · 2026-02-16T11:29:27+09:00

agentson commented

2026-02-16 11:29:27 +09:00

Summary

daily_review 테스트에서 하드코딩된 날짜('2026-02-14') 대신 datetime.now(UTC) 기준 TODAY 변수를 사용하도록 수정
DecisionLogger/log_trade가 현재 날짜로 저장하므로 조회 날짜도 일치시켜 0건 반환 문제 해결
영향받은 테스트: test_generate_scorecard_market_scoped, test_generate_scorecard_top_winners_and_losers, test_generate_scorecard_empty_day

Test plan

수정된 3개 테스트 통과 확인
daily_review 전체 10개 테스트 통과 확인
전체 538개 테스트 통과, 실패 0개

Closes #129

🤖 Generated with Claude Code

## Summary - daily_review 테스트에서 하드코딩된 날짜('2026-02-14') 대신 `datetime.now(UTC)` 기준 TODAY 변수를 사용하도록 수정 - DecisionLogger/log_trade가 현재 날짜로 저장하므로 조회 날짜도 일치시켜 0건 반환 문제 해결 - 영향받은 테스트: `test_generate_scorecard_market_scoped`, `test_generate_scorecard_top_winners_and_losers`, `test_generate_scorecard_empty_day` ## Test plan - [x] 수정된 3개 테스트 통과 확인 - [x] daily_review 전체 10개 테스트 통과 확인 - [x] 전체 538개 테스트 통과, 실패 0개 Closes #129 🤖 Generated with [Claude Code](https://claude.com/claude-code)

agentson added 1 commit 2026-02-16 11:29:27 +09:00

fix: daily_review 테스트에서 날짜 불일치로 인한 실패 수정 (#129 )

CI / test (pull_request) Has been cancelled

Details

e2275a23b1

DecisionLogger와 log_trade가 datetime.now(UTC)로 현재 날짜를 저장하는데,
테스트에서 하드코딩된 '2026-02-14'로 조회하여 0건이 반환되던 문제 수정.
generate_scorecard 호출 시 TODAY 변수를 사용하도록 변경.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

jihoson merged commit 31b4d0bf1e into main

2026-02-16 11:30:20 +09:00

jihoson referenced this issue from a commit

2026-02-16 11:30:21 +09:00

Merge pull request 'fix: daily_review 테스트 날짜 불일치 수정 (#129)' (#130) from feature/issue-129-fix-daily-review-test-date into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#130