fix: WAITING_WORKER 데모 업무카드에 승인 이력 추가 - #147
Merged
Merged
Conversation
여권 사본 제출 대기 등 WAITING_WORKER 상태로 시딩된 업무카드가 실제로는 APPROVED 단계를 거치지 않아 대응하는 ApprovalRequest가 없었고, 그 결과 근로자 보안 링크 발급·재발급이 항상 422(TASK_NOT_APPROVED)로 실패했다. transitionPath에 READY_FOR_REVIEW/APPROVED 단계를 추가하고 해당 업무들의 APPROVED 승인 이력을 시딩해 실제 애플리케이션 흐름과 일치시켰다. Fixes #146
WAITING_WORKER 업무 승인 이력 추가로 늘어난 approval_request/ task_transition_history/audit_event 카운트를 반영.
2 tasks
hywznn
previously approved these changes
Aug 12, 2026
Contributor
|
감사합니다 |
WorkerLinkViewController·WorkerLinkDocumentController·WorkerResponseController가 다른 모든 컨트롤러와 달리 /api/v1 접두사 없이 매핑되어 있어, 클라이언트가 실제로 호출하는 /api/v1/public/worker-links/** 경로에서는 SecurityConfig의 permitAll 규칙이 매칭되지 않고 인증이 필요한 규칙으로 떨어졌다. 근로자가 링크로 안내를 조회하거나 서류를 제출하는 흐름 전체가 실제 배포 환경에서 항상 실패하는 상태였다. 세 컨트롤러 매핑과 SecurityConfig의 permitAll 매처에 /api/v1 접두사를 추가하고, 같은 버그를 잡아내지 못했던 통합 테스트의 경로도 실제 API와 일치시켰다. Fixes #148
hywznn
previously approved these changes
Aug 12, 2026
hywznn
approved these changes
Aug 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
문제
라이브 시나리오 테스트 중 "여권 사본 제출 대기"(94000000-...003) 등 WAITING_WORKER 상태 업무카드에서 "근로자 보안 링크 발급·재발급"이 항상
422 TASK_NOT_APPROVED로 실패하는 걸 발견했다 (#146).원인
데모 시드가 업무 3, 8(retired), 10, 11번을
NEEDS_INFO → DRAFT → WAITING_WORKER경로로 직접 WAITING_WORKER 상태로 만들어,READY_FOR_REVIEW/APPROVED단계를 건너뛰었다. 그래서 대응하는ApprovalRequest가 없다. 실제 앱 로직(WorkerLinkService.issue())에서는 APPROVED 상태 + 유효한 ApprovalRequest 없이는 절대 WAITING_WORKER로 갈 수 없으므로, 이 데모 데이터는 실제 애플리케이션이 만들 수 없는 상태 조합이었다.수정
DemoTaskWorkflowSeedCatalog.transitionPath(): 3/8/10/11번 업무에READY_FOR_REVIEW,APPROVED단계 추가DemoTaskWorkflowSeedCatalog.demoApprovals(): 3/10/11번 업무(실제 노출되는 3개, 8번은 golden-flow 데모에서 이미 retired 처리되어 제외)에 APPROVED 승인 이력 추가DemoOperationalSeedCatalog: 늘어난 approval_request/task_transition_history/audit_event 카운트 및 actor_type·status 분포 assertion 갱신, retired golden-flow audit id 목록 갱신DemoOperationalSeedCatalogTest,DemoAuthSeedIntegrationTest: 동일하게 카운트/분포 기댓값 갱신검증
./gradlew test전체 통과 확인 (기존 574개 + 관련 테스트 모두 green)Test plan
./gradlew test전체 통과Closes #146