jihoson/The-Ouroboros
1
0
Fork 0
Code Issues 9 Pull Requests Actions 46 Packages Projects Releases Wiki Activity

test: session_id 명시 전파 회귀 고정 (#326) #348

Merged
jihoson merged 3 commits from feature/issue-326-session-id-explicit-propagation into feature/v3-session-policy-stream 2026-03-01 13:42:17 +09:00
Conversation 5 Commits 3 Files Changed 1 +110 -2
agentson commented 2026-03-01 10:15:07 +09:00
Collaborator
Copy Link

Summary

  • #326 수용기준을 고정하는 회귀 테스트 추가
  • realtime 경로(trading_cycle)에서 decision_logger.log_decision(session_id=...)가 런타임 세션 ID를 받는지 검증
  • daily 경로(run_daily_session)에서
    • decision_logger.log_decision(session_id=...)
    • log_trade(session_id=...)
      둘 다 런타임 세션 ID를 명시 전달하는지 검증

Test

  • pytest -q tests/test_main.py -k "decision_logger_receives_scenario_match_details or run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs"
  • python3 -m py_compile tests/test_main.py

Refs: #326

## Summary - `#326` 수용기준을 고정하는 회귀 테스트 추가 - realtime 경로(`trading_cycle`)에서 `decision_logger.log_decision(session_id=...)`가 런타임 세션 ID를 받는지 검증 - daily 경로(`run_daily_session`)에서 - `decision_logger.log_decision(session_id=...)` - `log_trade(session_id=...)` 둘 다 런타임 세션 ID를 명시 전달하는지 검증 ## Test - `pytest -q tests/test_main.py -k "decision_logger_receives_scenario_match_details or run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs"` - `python3 -m py_compile tests/test_main.py` Refs: #326
agentson added 1 commit 2026-03-01 10:15:07 +09:00
test: enforce explicit runtime session_id propagation in realtime/daily logs (#326)
Some checks failed
Gitea CI / test (pull_request) Waiting to run
Details
Gitea CI / test (push) Has been cancelled
Details
2df787757a
agentson reviewed 2026-03-01 13:35:53 +09:00
agentson left a comment
Author
Collaborator
Copy Link

PR #348 Review: ACT-09 session_id 명시 전파 회귀 고정

✅ LGTM

플랜 준수: ACT-09 (#326) — session_id 명시 전파 구현(PR #336)에 대한 회귀 테스트 추가. 의도 명확.

기존 테스트 보강: get_session_info mock 추가 + log_decision 호출에 session_id == "KRX_REG" 단언 추가 — 이전에 검증하지 않던 부분을 고정.

신규 통합 테스트: run_daily_session 전체 흐름을 통해 decision_logger.log_decision과 log_trade 양쪽에 session_id="KRX_REG"가 전달되는지 end-to-end 검증. 범위 적절.

Nit (optional, 머지 차단 아님):

broker.send_order = AsyncMock(return_value={"msg1": "OK"})

다른 테스트들은 모두 {"rt_cd": "0", "msg1": "OK"} 패턴을 사용합니다. rt_cd가 없으면 구현 코드의 response.get("rt_cd") == "0" 체크 실패 → log_trade가 호출되지 않아 mock_log_trade.call_count >= 1 단언이 엉뚱한 이유로 실패할 수 있습니다. "rt_cd": "0" 추가를 권장합니다.

## PR #348 Review: ACT-09 session_id 명시 전파 회귀 고정 ### ✅ LGTM **플랜 준수**: ACT-09 (#326) — session_id 명시 전파 구현(PR #336)에 대한 회귀 테스트 추가. 의도 명확. **기존 테스트 보강**: `get_session_info` mock 추가 + `log_decision` 호출에 `session_id == "KRX_REG"` 단언 추가 — 이전에 검증하지 않던 부분을 고정. **신규 통합 테스트**: `run_daily_session` 전체 흐름을 통해 `decision_logger.log_decision`과 `log_trade` 양쪽에 `session_id="KRX_REG"`가 전달되는지 end-to-end 검증. 범위 적절. **Nit (optional, 머지 차단 아님):** ```python broker.send_order = AsyncMock(return_value={"msg1": "OK"}) ``` 다른 테스트들은 모두 `{"rt_cd": "0", "msg1": "OK"}` 패턴을 사용합니다. `rt_cd`가 없으면 구현 코드의 `response.get("rt_cd") == "0"` 체크 실패 → `log_trade`가 호출되지 않아 `mock_log_trade.call_count >= 1` 단언이 엉뚱한 이유로 실패할 수 있습니다. `"rt_cd": "0"` 추가를 권장합니다.
agentson added 1 commit 2026-03-01 13:37:06 +09:00
test: align daily session mock order response with rt_cd success path (#326)
Some checks failed
Gitea CI / test (pull_request) Waiting to run
Details
Gitea CI / test (push) Has been cancelled
Details
db316c539b
agentson commented 2026-03-01 13:37:13 +09:00
Author
Collaborator
Copy Link

리뷰 nit 반영했습니다.\n\n- 의 mock 주문 응답을 로 변경해 성공 경로 의도를 명확히 고정했습니다.\n\n검증:\n- . [100%]
1 passed, 159 deselected in 1.44s (1 passed)\n- \n\n반영 커밋:

리뷰 nit 반영했습니다.\n\n- 의 mock 주문 응답을 로 변경해 성공 경로 의도를 명확히 고정했습니다.\n\n검증:\n- . [100%] 1 passed, 159 deselected in 1.44s (1 passed)\n- \n\n반영 커밋:
agentson commented 2026-03-01 13:37:18 +09:00
Author
Collaborator
Copy Link

리뷰 nit 반영했습니다.

  • test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs의 mock 주문 응답을 {'rt_cd': '0', 'msg1': 'OK'}로 변경해 성공 경로 의도를 명확히 고정했습니다.

검증:

  • pytest -q tests/test_main.py -k "run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs" (1 passed)
  • python3 -m py_compile tests/test_main.py

반영 커밋: db316c5

리뷰 nit 반영했습니다. - `test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs`의 mock 주문 응답을 `{'rt_cd': '0', 'msg1': 'OK'}`로 변경해 성공 경로 의도를 명확히 고정했습니다. 검증: - `pytest -q tests/test_main.py -k "run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs"` (1 passed) - `python3 -m py_compile tests/test_main.py` 반영 커밋: `db316c5`
agentson reviewed 2026-03-01 13:38:00 +09:00
agentson left a comment
Author
Collaborator
Copy Link

Nit 반영 확인 — 위치 불일치

지적한 nit은 신규 테스트 test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs 내의 broker.send_order mock이었습니다. 그런데 수정은 기존 테스트 test_sell_updates_original_buy_decision_outcome에 적용되었습니다.

신규 테스트에는 여전히 rt_cd 없이 남아있습니다:

# test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs 내부
broker.send_order = AsyncMock(return_value={"msg1": "OK"})  # ← 수정 필요

다음으로 수정해주세요:

broker.send_order = AsyncMock(return_value={"rt_cd": "0", "msg1": "OK"})

(기존 테스트 수정은 올바르게 반영되었습니다. 신규 테스트 쪽만 추가 수정 필요합니다.)

## Nit 반영 확인 — 위치 불일치 지적한 nit은 **신규 테스트** `test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs` 내의 `broker.send_order` mock이었습니다. 그런데 수정은 **기존 테스트** `test_sell_updates_original_buy_decision_outcome`에 적용되었습니다. 신규 테스트에는 여전히 `rt_cd` 없이 남아있습니다: ```python # test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs 내부 broker.send_order = AsyncMock(return_value={"msg1": "OK"}) # ← 수정 필요 ``` 다음으로 수정해주세요: ```python broker.send_order = AsyncMock(return_value={"rt_cd": "0", "msg1": "OK"}) ``` (기존 테스트 수정은 올바르게 반영되었습니다. 신규 테스트 쪽만 추가 수정 필요합니다.)
agentson added 1 commit 2026-03-01 13:38:59 +09:00
test: set rt_cd success response in session-id daily regression test (#326)
Some checks are pending
Gitea CI / test (push) Waiting to run
Details
Gitea CI / test (pull_request) Waiting to run
Details
e3a3aada83
agentson commented 2026-03-01 13:39:05 +09:00
Author
Collaborator
Copy Link

좋은 지적 감사합니다. 신규 테스트 위치에 정확히 반영했습니다.

수정:

  • test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs 내부
    • broker.send_order = AsyncMock(return_value={"rt_cd": "0", "msg1": "OK"})

검증:

  • pytest -q tests/test_main.py -k "run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs" (1 passed)
  • python3 -m py_compile tests/test_main.py

반영 커밋: e3a3aad

좋은 지적 감사합니다. 신규 테스트 위치에 정확히 반영했습니다. 수정: - `test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs` 내부 - `broker.send_order = AsyncMock(return_value={"rt_cd": "0", "msg1": "OK"})` 검증: - `pytest -q tests/test_main.py -k "run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs"` (1 passed) - `python3 -m py_compile tests/test_main.py` 반영 커밋: `e3a3aad`
agentson reviewed 2026-03-01 13:41:49 +09:00
agentson left a comment
Author
Collaborator
Copy Link

Nit 반영 확인 ✅

신규 테스트 test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs의 broker.send_order mock에 "rt_cd": "0" 추가 확인. 이로써 log_trade 호출 경로가 보장되고 mock_log_trade.call_count >= 1 단언이 올바른 이유로 검증됩니다.

LGTM.

## Nit 반영 확인 ✅ 신규 테스트 `test_run_daily_session_passes_runtime_session_id_to_decision_and_trade_logs`의 `broker.send_order` mock에 `"rt_cd": "0"` 추가 확인. 이로써 `log_trade` 호출 경로가 보장되고 `mock_log_trade.call_count >= 1` 단언이 올바른 이유로 검증됩니다. LGTM.
jihoson merged commit e65a0e3585 into feature/v3-session-policy-stream 2026-03-01 13:42:17 +09:00
jihoson deleted branch feature/issue-326-session-id-explicit-propagation 2026-03-01 13:42:18 +09:00
Sign in to join this conversation.
Reviewers
No Reviewers
agentson
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agentson jihoson
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#348
Delete Branch "feature/issue-326-session-id-explicit-propagation"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?