test: session_handover_check non-ci strict #TBD 회귀 케이스 추가 #358

New Issue

agentson · 2026-03-01T20:58:36+09:00

agentson commented

2026-03-01 20:58:36 +09:00

Summary

session_handover_check.py의 #TBD 차단 규칙은 현재 strict=True, ci=True 케이스만 테스트됨.

리뷰 지적대로 strict=True, ci=False에서도 - next_ticket: #TBD가 반드시 차단되는 회귀 테스트를 추가해,
향후 조건 블록이 if strict and not ci_mode:로 되돌아가는 실수를 방지해야 함.

Why

현재 테스트 3개는 핵심 동작을 검증하지만, 아래 조합이 누락됨:

strict=True
ci_mode=False
next_ticket=#TBD
기대: BLOCK

이 케이스가 없으면 non-ci strict 경로에서 placeholder 차단 회귀를 놓칠 수 있음.

Acceptance Criteria

tests/test_session_handover_check.py에 strict=True, ci=False, #TBD -> BLOCK 테스트 추가
실패 메시지 latest handover entry must not use placeholder next_ticket (#TBD) 검증
기존 테스트/게이트(pytest, ruff, script check`) 통과

Scope Mapping

REQ: REQ-OPS-003
TASK: TASK-OPS-003
TEST: TEST-ACC-009

## Summary `session_handover_check.py`의 `#TBD` 차단 규칙은 현재 `strict=True, ci=True` 케이스만 테스트됨. 리뷰 지적대로 `strict=True, ci=False`에서도 `- next_ticket: #TBD`가 반드시 차단되는 회귀 테스트를 추가해, 향후 조건 블록이 `if strict and not ci_mode:`로 되돌아가는 실수를 방지해야 함. ## Why 현재 테스트 3개는 핵심 동작을 검증하지만, 아래 조합이 누락됨: - strict=True - ci_mode=False - next_ticket=#TBD - 기대: BLOCK 이 케이스가 없으면 non-ci strict 경로에서 placeholder 차단 회귀를 놓칠 수 있음. ## Acceptance Criteria - [ ] `tests/test_session_handover_check.py`에 `strict=True, ci=False, #TBD -> BLOCK` 테스트 추가 - [ ] 실패 메시지 `latest handover entry must not use placeholder next_ticket (#TBD)` 검증 - [ ] 기존 테스트/게이트(`pytest`, `ruff`, script check`) 통과 ## Scope Mapping - REQ: `REQ-OPS-003` - TASK: `TASK-OPS-003` - TEST: `TEST-ACC-009`

agentson referenced this issue

2026-03-01 20:58:43 +09:00

workflow: session handover gate 실행환경 모드 분리 (#353) #354

agentson referenced this issue from a commit

2026-03-01 21:02:53 +09:00

test: cover non-ci strict #TBD guard in handover check (#358)

agentson referenced a pull request that will close this issue

2026-03-01 21:03:06 +09:00

test: add non-ci strict #TBD regression coverage for handover gate (#358) #359

jihoson closed this issue

2026-03-01 21:10:05 +09:00

jihoson referenced this issue from a commit