Skip to content

Conversation

@thkim7
Copy link
Collaborator

@thkim7 thkim7 commented Sep 26, 2025

📝 작업 내용


🔗 관련 이슈

  • Closes #이슈번호
  • Related to #이슈번호

💬 추가 요청사항


✅ 체크리스트

코드 품질

  • 커밋 컨벤션 준수 (feat/fix/docs/refactor 등)
  • 불필요한 코드/주석 제거

테스트

  • 로컬 환경에서 동작 확인 완료
  • 기존 기능에 영향 없음 확인

배포 준비

  • 환경변수 추가/변경사항 문서화
  • DB 마이그레이션 필요 여부 확인
  • 배포 시 주의사항 없음

@thkim7 thkim7 closed this Sep 26, 2025
1. Job/Task 서비스 통합
   - JobService, TaskService 삭제
   - WorkflowService에 Job/Task CRUD 메서드 통합
   - JobController, TaskController에서 WorkflowService 사용하도록 변경

2. Task 데이터 구조 확장 (9개 → 10개)
   - 새로운 Task 추가: "이미지 OCR 태스크" (ID: 8)
   - 기존 Task ID 변경: RAG 생성 (8→9), 블로그 발행 (9→10)
   - H2 테스트 데이터(03-insert-workflow-h2.sql) 업데이트

3. Integration 테스트 리팩토링
   - JobApiIntegrationTest, TaskApiIntegrationTest에서 WorkflowService 사용
   - Task 10개 구조에 맞춘 테스트 케이스 추가
   - 새로운 OCR Task 및 업데이트된 블로그 발행 Task 테스트 추가

4. 코드 일관성 및 응집도 개선
   - 워크플로우 관련 모든 CRUD 기능을 단일 서비스로 집중
   - Job-Task 연결 구조 업데이트: Job1(Task 1-8), Job2(Task 9-10)
@thkim7 thkim7 reopened this Sep 27, 2025
1. h2.sql task 9개 -> 10개
2. JobApiIntegrationTest는 모두 통과
3. TaskApiIntegrationTest는 자꾸 Json 직렬화 때문에 통과를 못함
@thkim7 thkim7 self-assigned this Sep 29, 2025
@thkim7 thkim7 marked this pull request as ready for review September 29, 2025 03:45
@jihukimme jihukimme self-assigned this Sep 29, 2025
- verify 호출 시 ArgumentMatchers.eq() 사용하여 InvalidUseOfMatchersException 해결
- workflowId 매개변수에 Matcher 적용으로 통합 테스트 안정화
…am-icebang into feature/task

# Conflicts:
#	apps/user-service/src/test/java/site/icebang/integration/tests/workflow/WorkflowRunApiIntegrationTest.java
@thkim7 thkim7 merged commit ab59877 into develop Sep 29, 2025
7 checks passed
@thkim7 thkim7 deleted the feature/task branch September 29, 2025 04:58
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.

3 participants