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

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

Merged
jihoson merged 1 commits from feature/issue-358-handover-nonci-tbd into feature/v3-session-policy-stream 2026-03-01 21:10:05 +09:00
Conversation 1 Commits 1 Files Changed 1 +28
agentson commented 2026-03-01 21:03:06 +09:00
Collaborator
Copy Link

Summary

session_handover_check.py의 non-CI strict 경로에서 - next_ticket: #TBD 차단 회귀를 고정하는 테스트를 추가했습니다.

Changes

  • tests/test_session_handover_check.py
    • test_non_ci_strict_still_blocks_tbd_next_ticket 추가
    • 검증 조합: strict=True, ci_mode=False, next_ticket=#TBD -> BLOCK

Validation

  • pytest -q tests/test_session_handover_check.py (4 passed)
  • ruff check tests/test_session_handover_check.py
  • python3 scripts/session_handover_check.py --strict --ci

Traceability

## Summary `session_handover_check.py`의 non-CI strict 경로에서 `- next_ticket: #TBD` 차단 회귀를 고정하는 테스트를 추가했습니다. ## Changes - `tests/test_session_handover_check.py` - `test_non_ci_strict_still_blocks_tbd_next_ticket` 추가 - 검증 조합: `strict=True`, `ci_mode=False`, `next_ticket=#TBD` -> BLOCK ## Validation - `pytest -q tests/test_session_handover_check.py` (`4 passed`) - `ruff check tests/test_session_handover_check.py` - `python3 scripts/session_handover_check.py --strict --ci` ## Traceability - Follow-up for review gap on #354 - Closes #358
agentson added 1 commit 2026-03-01 21:03:06 +09:00
test: cover non-ci strict #TBD guard in handover check (#358)
All checks were successful
Gitea CI / test (push) Successful in 33s
Details
Gitea CI / test (pull_request) Successful in 32s
Details
999091e003
agentson commented 2026-03-01 21:05:29 +09:00
Author
Collaborator
Copy Link

PR #359 코드 리뷰

변경 요약: strict=True, ci=False, #TBD 경로의 회귀 테스트 추가 (이전 리뷰에서 지적한 Missing 케이스)

✅ 잘 된 점

테스트 설계가 명확하다.

  • merged_to_feature_branch=yes로 설정해 merge-gate 에러를 제거한 뒤 #TBD 차단만 고립시켰다. 다른 에러와 섞이지 않아 실패 원인을 명확히 지목할 수 있다.
  • ci_mode=False로 non-CI 경로를 명시해 CI 경로(test_ci_mode_still_blocks_tbd)와 대칭을 이룬다.
  • 4 passed 직접 확인 완료.

최종 테스트 커버리지 매트릭스 — 이번 PR로 완성됨

strict ci_mode #TBD 기대 동작 테스트
True True ✓ BLOCK ✅ test_ci_mode_still_blocks_tbd
True False ✓ BLOCK ✅ test_non_ci_strict_still_blocks_tbd (신규)
True True ✗ / merge=no PASS ✅ test_ci_mode_skips
True False ✗ / merge=no FAIL ✅ test_non_ci_strict_enforces

🔵 Minor — PR 설명 Scope 참조가 다소 느슨함

PR 설명의 Scope가 REQ-OPS-003 / TASK-OPS-003 / TEST-ACC-009를 가리키고 있는데, 이 항목들은 "구현 태스크는 테스트를 동반해야 한다"는 프로세스 규칙이다. 실제 변경 대상은 PR #354에서 추가된 --ci 기능의 회귀 보호이므로, 연결이 간접적이다.

블로커는 아니고 이슈로 분리할 필요도 없다. 다음번엔 직접 연관 항목(예: TEST-HANDOVER-CI 또는 단순히 Closes #358)만 기재하는 편이 깔끔하다.

✅ 최종 판정

변경 범위: 테스트 1개 추가 (31줄), 기능 변경 없음.
지적된 회귀 갭을 정확히 메웠다. Merge 가능 상태.

## PR #359 코드 리뷰 **변경 요약**: `strict=True, ci=False, #TBD` 경로의 회귀 테스트 추가 (이전 리뷰에서 지적한 Missing 케이스) --- ### ✅ 잘 된 점 **테스트 설계가 명확하다.** - `merged_to_feature_branch=yes`로 설정해 merge-gate 에러를 제거한 뒤 `#TBD` 차단만 고립시켰다. 다른 에러와 섞이지 않아 실패 원인을 명확히 지목할 수 있다. - `ci_mode=False`로 non-CI 경로를 명시해 CI 경로(`test_ci_mode_still_blocks_tbd`)와 대칭을 이룬다. - 4 passed 직접 확인 완료. **최종 테스트 커버리지 매트릭스 — 이번 PR로 완성됨** | strict | ci_mode | #TBD | 기대 동작 | 테스트 | |--------|---------|------|----------|--------| | True | True | ✓ | BLOCK | ✅ `test_ci_mode_still_blocks_tbd` | | True | False | ✓ | BLOCK | ✅ `test_non_ci_strict_still_blocks_tbd` (신규) | | True | True | ✗ / merge=no | PASS | ✅ `test_ci_mode_skips` | | True | False | ✗ / merge=no | FAIL | ✅ `test_non_ci_strict_enforces` | --- ### 🔵 Minor — PR 설명 Scope 참조가 다소 느슨함 PR 설명의 Scope가 `REQ-OPS-003` / `TASK-OPS-003` / `TEST-ACC-009`를 가리키고 있는데, 이 항목들은 "구현 태스크는 테스트를 동반해야 한다"는 프로세스 규칙이다. 실제 변경 대상은 PR #354에서 추가된 `--ci` 기능의 회귀 보호이므로, 연결이 간접적이다. 블로커는 아니고 이슈로 분리할 필요도 없다. 다음번엔 직접 연관 항목(예: `TEST-HANDOVER-CI` 또는 단순히 `Closes #358`)만 기재하는 편이 깔끔하다. --- ### ✅ 최종 판정 변경 범위: 테스트 1개 추가 (31줄), 기능 변경 없음. 지적된 회귀 갭을 정확히 메웠다. **Merge 가능** 상태.
jihoson merged commit e430d6f28a into feature/v3-session-policy-stream 2026-03-01 21:10:05 +09:00
jihoson deleted branch feature/issue-358-handover-nonci-tbd 2026-03-01 21:10:05 +09:00
Sign in to join this conversation.
Reviewers
No Reviewers
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#359
Delete Branch "feature/issue-358-handover-nonci-tbd"

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?