Skip to content

Fix/#49 interview termination state - #53

Merged
2-dayeon merged 7 commits into
mainfrom
fix/#49-interview-termination-state
Oct 20, 2026
Merged

2-dayeon merged 7 commits into
mainfrom
fix/#49-interview-termination-state

Conversation

@2-dayeon

Copy link
Copy Markdown
Contributor

49

@kimwuju kimwuju left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰 남깁니다. 머지를 막을 이슈는 없습니다.

  • 진행 중인 세션에 남은 질문이 있을 때만 COMPLETE를 거절하고, 이미 확정된 상태는 outbox 결과를 그대로 반환하는 흐름을 확인했습니다. REST는 전용 핸들러로 409 INTERVIEW_NOT_FINISHED를 반환하고, WebSocket은 ERROR를 보낸 뒤 연결을 유지해서 문서와 동작이 일치합니다.
  • END의 status는 서비스 응답(재전송 시에는 outbox의 interviewStatus)에서 가져오므로, QUIT과 COMPLETE가 엇갈려도 실제 상태가 전달됩니다. 삭제한 end(String)를 호출하는 곳도 남아 있지 않습니다.
  • 복구 워커가 잠금 안에서 종료 상태를 다시 확인하므로, 스캔 이후 같은 sessionId로 다시 준비된 IN_PROGRESS 세션을 지우지 않습니다.
  • 현재 Web은 명시적인 COMPLETE를 보내지 않으므로, 조기 완료를 거절해도 기존 화면 흐름은 깨지지 않습니다.
  • 로컬에서 Java 21로 전체 테스트 267개가 통과했고 CI도 통과했습니다. #52와 충돌 없이 합쳐집니다.

@2-dayeon
2-dayeon merged commit 8a0f42f into main Oct 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants