Skip to content

feat: Review 수정 Mutation 추가#24

Merged
robinjoon merged 2 commits intomainfrom
ticket/#19
Mar 6, 2026
Merged

feat: Review 수정 Mutation 추가#24
robinjoon merged 2 commits intomainfrom
ticket/#19

Conversation

@robinjoon
Copy link
Member

Summary

  • updateReview Mutation 추가 — 기존 KPT 회고의 단계별 내용(steps)을 수정
  • 소유권 검증: 본인 회고만 수정 가능 (AccessDeniedException)
  • 미존재 회고 수정 시 EntityNotFoundException 발생

Changes

  • Domain: ReviewCommand.Update, Review.withUpdatedSteps(), ReviewRepository.update()
  • Application: ReviewService.update() (findById → 소유권 검증 → repository.update)
  • Infrastructure: ExposedReviewRepository.update() (steps, updatedAt 갱신)
  • Presentation: ReviewDataFetcher.updateReview() + GraphQL 스키마 (UpdateReviewInput)
  • Test: Domain(ReviewTest), Application(ReviewServiceTest) — 성공/미존재/권한 위반 케이스

Test plan

  • Domain 단위 테스트 (withUpdatedSteps)
  • Application 단위 테스트 (성공, EntityNotFoundException, AccessDeniedException)
  • 전체 빌드 및 기존 테스트 통과 확인

Closes #19

🤖 Generated with Claude Code

robinjoon and others added 2 commits March 6, 2026 20:45
updateReview Mutation을 추가하여 기존 회고의 KPT 단계별 내용을 수정할 수 있도록 한다.
본인 회고만 수정 가능하며, 소유권 검증을 포함한다.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@robinjoon robinjoon merged commit 58b2763 into main Mar 6, 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.

Review 수정 Mutation 추가

1 participant