feat: 데모 사업장 및 근로자 시드 데이터 확장 - #69
Open
krestar wants to merge 3 commits into
Open
Conversation
- 기존 FOWOCO Demo Company의 계정 구성을 유지하고 근로자 5명을 추가한다. - FOWOCO Test Company를 별도 사업장으로 생성한다. - Test Company에 ADMIN, HR, VIEWER 계정을 각각 1명씩 추가한다. - Test Company에 근로자 5명을 추가한다. - 고정 UUID를 사용해 시드를 재실행해도 데이터가 중복되지 않도록 한다. - 예약된 ID나 이메일이 기존 데이터와 충돌하면 덮어쓰지 않고 실행을 중단한다. - Demo Seed 설정과 개발 문서를 두 사업장 기준으로 확장한다. - 단위 및 통합 테스트에서 회사 2개, 계정 23명, 근로자 10명을 검증한다.
krestar
commented
Jul 31, 2026
| demoWorker.preferredLanguage(), | ||
| today.plusDays(demoWorker.stayExpiryDays()), | ||
| today.minusYears(1), | ||
| today.plusDays(demoWorker.contractEndDays()), |
BcKmini
self-requested a review
July 31, 2026 02:25
BcKmini
previously approved these changes
Jul 31, 2026
| 명시적으로 켭니다. 설정한 관리자 1명과 고정된 가상 계정 19명 | ||
| (`ADMIN` 1명, `HR` 12명, `VIEWER` 6명)을 만들며, 모든 계정은 로컬에서 설정한 | ||
| 같은 비밀번호로 로그인할 수 있습니다. 기본값과 비밀번호 기본값은 없습니다. | ||
| local H2 또는 로컬 PostgreSQL `dev`에서만 사용할 데모 데이터가 필요할 때 |
Member
There was a problem hiding this comment.
재성님 데이터 만들어주신거 감사합니다. 추가로 만들어 주셨으면 하는 데이터가 있어서 리뷰 남깁니다
Member
There was a problem hiding this comment.
클라이언트 화면 검증 관점에서 보면 Task/Document/AuditEvent 시드가 없어서, 이 PR만으로는 대시보드·업무함·문서함·근로자 상세 화면이 전부 빈 상태로만 보이게 되어서 화면 확인이 어려워요. 화면 확인을 위해 아래 데이터도 함께 시드해주실 수 있을까요?
Member
There was a problem hiding this comment.
- Task (업무) 시드 — Demo Company 근로자 기준으로 상태별 최소 1건씩 필요해요.
- DRAFT (AI 준비 완료 카드 확인용)
- READY_FOR_REVIEW (승인 대기 카드/탭 확인용)
- WAITING_WORKER / WAITING_EXTERNAL (근로자 응답 대기 카드 확인용)
- COMPLETED — updated_at을 오늘 날짜로 최소 1건 (업무함 "오늘 완료" 지표 확인용)
- 기한(due_date)도 7일 이내/30일 이내/여유 있는 케이스를 섞어주시면 긴급도 UI까지 확인 가능할거같아요
- Document (서류) 시드 — 근로자당 1~2건, 상태 다양하게 필요해요
- MISSING / SUBMITTED / VERIFIED 각각 최소 1건
- expiry_date를 30일 이내로 잡은 문서 1건 이상 (문서함 "만료 예정" 탭/지표 확인용)
- AuditEvent (활동 이력) 시드 — 위에서 만든 Task 중 최소 1건에 대해 TASK_CREATED/APPROVAL_REQUESTED 같은 이벤트 2~3건 (근로자 상세 패널의 "최근 Timeline" 확인용)
데이터 모델·API 변경 없이 시드 데이터만 추가하는 선에서 가능하면, 이번 PR 범위에 포함하거나 후속 PR로 분리해주셔도 좋을 것 같습니다.
Contributor
There was a problem hiding this comment.
@BcKmini 위를 구분짓게 해주기 위해서 문서에 있는 slot에서 필요한 데이터들을 최대한 다 넣어달라고 했는데 + 라벨값들도 같이 넣어달라고 하겠습니다
Member
There was a problem hiding this comment.
PR 참고해주세요
| 화면 | 참고 PR | 필요한 데이터 |
|---|---|---|
| Today(대시보드) | fowoco/client#23, #202 | Task 상태별(READY_FOR_REVIEW/DRAFT/COMPLETED), 기한 다양 |
| 업무함 | fowoco/client#26, #203 | Task 상태·기한 전체 조합 (Metric Strip 4종) |
| 업무 상세(CASE) | fowoco/client#33 | Task 1건 + AuditEvent 여러 건(상태 전이 이력) |
| 근로자 목록/상세 | fowoco/client#35, #65, #204 | Task(worker_id 연결) + AuditEvent(Timeline) |
| 서류관리 | fowoco/client#66, #205 | Document 상태별(MISSING/SUBMITTED/VERIFIED) + 만료 임박 건 |
| 기한 색상 구분 | fowoco/client#87 | Task/Document due_date·expiry_date 다양한 값 |
- Demo Company에 상태별 데모 Task 5개를 추가한다. - AI 후보와 승인 대기, 근로자 응답 대기, 외부 처리 대기, 당일 완료 상태를 구성한다. - 마감일을 7일 이내, 30일 이내, 여유 구간으로 분산한다. - 근로자별 1~2개의 서류를 추가하고 MISSING, SUBMITTED, VERIFIED 상태를 포함한다. - 30일 이내 만료되는 근로자 서류를 추가한다. - 승인 대기 Task에 최근 AuditEvent 3개를 추가한다. - 고정 UUID와 충돌 검증으로 재실행 시 중복 생성을 방지한다. - 통합 테스트와 Demo Seed 개발 문서를 확장한다.
- 검색어가 없을 때 keyword 파라미터를 문자열로 명시적으로 캐스팅한다. - PostgreSQL에서 발생하던 lower(bytea) 오류를 방지한다. - Task 목록 조회가 500 응답을 반환하던 문제를 수정한다.
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
3 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.
왜 필요한가요?
기존 Demo Seed는
FOWOCO Demo Company와 사용자 계정만 생성해 사업장별 근로자 조회 및 테넌트 분리 시나리오를 검증하기 어려웠습니다.두 개의 독립된 사업장과 각 사업장에 속한 근로자 데이터가 필요해 다음 시나리오를 재현할 수 있도록 확장합니다.
무엇이 바뀌나요?
FOWOCO Test Company를 별도 사업장으로 생성합니다.ADMIN,HR,VIEWER계정을 각각 1명씩 추가합니다.company_id에만 소속됩니다..env.example의 Demo Seed 설명을 확장했습니다.사업장별 생성 데이터
재실행 및 충돌 처리
어떻게 검증했나요?
./gradlew clean test./gradlew build/health와 Swagger UI 확인실행한 대상 테스트: