Skip to content

fix: 일반 멤버 방 탈퇴 처리 - #18

Merged
Heyaaz merged 1 commit into
mainfrom
fix/room-member-leave
May 20, 2026
Merged

fix: 일반 멤버 방 탈퇴 처리#18
Heyaaz merged 1 commit into
mainfrom
fix/room-member-leave

Conversation

@Heyaaz

@Heyaaz Heyaaz commented May 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • 일반 멤버 삭제 요청 시 방 탈퇴 처리
  • 방장 삭제 시 기존 방 soft delete 유지

Changes

  • 방장/일반 멤버 DELETE 분기 구현
  • 일반 멤버 탈퇴 시 멤버십/프로필/만족도 점수 soft delete
  • 탈퇴 멤버 업로드 사진 유지 검증 추가

Verification

  • Not run
  • Build
  • Test
  • Manual

Commands:

  • ./gradlew compileKotlin compileTestKotlin
  • ./gradlew test --tests com.tripsync.AuthContractTests

Notes

  • Testcontainers Docker 미실행으로 통합 테스트 실패: /var/run/docker.sock 없음

@Heyaaz Heyaaz left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Nogada PR Review

Verdict: Approved ✅

확인한 내용

  • 일반 멤버 DELETE /rooms/{roomId} 요청 시 방 삭제가 아니라 본인 멤버십/프로필/만족도 점수만 soft delete 처리되는지 확인
  • 방장 삭제 시 방/일정/슬롯/만족도/갈등지도/멤버/프로필/사진 아카이브 soft delete 흐름 확인
  • 탈퇴한 멤버가 기존 방 상세 접근 권한을 잃고, 방장 입장에서는 방과 공유코드가 유지되는지 확인
  • 재입장 시 soft-deleted RoomMember를 재활성화해서 unique constraint 충돌을 피하는 흐름 확인

Blocking Issues

  • 없음

Non-blocking Notes

  • clean test 종료 시 Testcontainers/PostgreSQL shutdown 로그가 noisy하게 찍히지만, Gradle 결과와 XML 요약 기준 테스트 실패는 없음

Verification

  • git diff --check origin/main...pr-18
  • ./gradlew clean test --no-daemon --no-build-cache
  • ✅ Test XML: 49 tests, 0 failures, 0 errors, 0 skipped

[Heyaaz의 Nogada]

@Heyaaz
Heyaaz merged commit 482e280 into main May 20, 2026
1 check passed
@Heyaaz
Heyaaz deleted the fix/room-member-leave branch May 20, 2026 03:18
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.

1 participant