Skip to content

[debug/#547] 프로필 이름 변경 시 내 채팅방 목록이 전부 내 이름으로 변경되는 버그#548

Merged
Dimo-2562 merged 4 commits intodevelopfrom
debug/#547
Mar 22, 2026
Merged

[debug/#547] 프로필 이름 변경 시 내 채팅방 목록이 전부 내 이름으로 변경되는 버그#548
Dimo-2562 merged 4 commits intodevelopfrom
debug/#547

Conversation

@Dimo-2562
Copy link
Copy Markdown
Contributor

@Dimo-2562 Dimo-2562 commented Mar 22, 2026 •
edited
Loading

❤️ 기능 설명

프로필 수정 시 DIRECT 채팅방 상대방의 displayName이 정상적으로 업데이트되도록 수정했습니다.

버그 수정 (d084f42)

  • findAllByMemberId()로 본인의 모든 ChatRoomMember를 업데이트하던 로직을 findDirectChatCounterParts()로 DIRECT 채팅방의 상대방만 조회 후 updateDisplayName() 호출하도록 변경
  • ChatRoomMemberRepository에 findDirectChatCounterParts 쿼리 메서드 추가

단위 테스트 (21efbe6)

  • 프로필 수정 시 findDirectChatCounterParts 호출 및 displayName 변경 검증 테스트 추가
  • ChatFixture에 createChatRoomMemberWithDisplayName, createJoinedMember(displayName) 팩토리 메서드 추가

통합 테스트 환경 개선 (12d5856)

  • FileService를 @MockitoBean으로 mock 처리하여 실제 스토리지 의존 제거
  • profileImgUrl을 검증하는 기존 테스트에 stub 추가

통합 테스트 작성 (88e6f96)

  • PATCH /api/my/profile 통합 테스트 6개 추가
    • 모든 필드(memberName, birth, level, keywords, imgKey) 정상 업데이트 검증
    • imgKey 없이 프로필 업데이트
    • 기존 이미지가 있고 imgKey가 다르면 이미지 변경
    • DIRECT 채팅방 상대방 displayName 업데이트 검증
    • PARTY 채팅방 displayName 미변경 검증
    • 존재하지 않는 회원 시 404 에러
  • MemberKeywordRepository에 findAllByMemberId 추가

테스트 성공 결과 스크린샷 첨부
image image

연결된 issue

연결된 issue를 자동으로 닫기 위해 아래 {이슈넘버}를 입력해주세요.

close #547



🩷 Approve 하기 전 확인해주세요!

  • @kanghana1 멤버 통합테스트에서 파일서비스 로직을 호출하던 것을 mock + stub으로 처리하였습니다. 테스트의 데이터가 스토리지에 저장될 이유가 없으므로 개선하였습니다!

✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 테스트 결과 사진을 넣었는가?
  • 이슈넘버를 적었는가?

@Dimo-2562 Dimo-2562 requested review from dbalsk and kanghana1 March 22, 2026 07:22
@Dimo-2562 Dimo-2562 self-assigned this Mar 22, 2026
@Dimo-2562 Dimo-2562 added the 🕷️ BUG Something isn't working label Mar 22, 2026
kanghana1
kanghana1 approved these changes Mar 22, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@kanghana1 kanghana1 left a comment

Choose a reason for hiding this comment

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

확인했습니다! 고생했어용

@Dimo-2562 Dimo-2562 merged commit fb3729f into develop Mar 22, 2026
1 check passed
@Dimo-2562 Dimo-2562 deleted the debug/#547 branch March 22, 2026 09:10
@kanghana1 kanghana1 mentioned this pull request Mar 23, 2026
Merged
kanghana1 added a commit that referenced this pull request Mar 23, 2026
@kanghana1 @Dimo-2562 @dbalsk
Release 1.1.3V
afe4d4e 
* [chore/#528] 서브도메인 명시 (#529)

* [chore/#531] Nginx 설정 파일 수정 (#532)

* chore: Nginx 프로토콜 설정을 Cloudflare에서 전달받아 설정하도록 변경

* chore: Nginx 웹소켓 타임아웃의 기본값이 1분이므로 60분으로 변경

* chore: Nginx gzip 설정을 통해 json 응답은 압축해서 보내도록 변경

* [Fix/#521] 채팅 이미지/파일 API 통합 및 테스트 코드 작성 (#526)

* refact: image -> file로 통합

* refact: S3 -> GCS로 예외 코드 명칭 변경

* refact: 채팅 이미지 업로드/다운로드 코드 제거

* refact: 채팅 이미지 관련 코드 수정

* feat: 채팅 파일 통합테스트

* feat: 채팅 파일 단위테스트

* feat: 파일 도메인 통합테스트

* feat: 파일 도메인 단위테스트

* fix: dto의 필드명은 바뀌지 않도록 롤뱁

* chore: 사용하지 않는 ChatImageRepository 제거

* test: 채팅 도메인 통합 테스트 및 단위 테스트에 이미지 필드 테스트 추가

* fix: 기존 images dto 유지를 위해 필드명 롤백

* fix: 기존 필드명이 imageOrder가 아닌 imgOrder였으므로 롤백

---------

Co-authored-by: dmori <chosehoon2@naver.com>

* [docs/#534] 이슈 템플릿 추가 (#535)

* [chore/#536] FCM 구현을 위한 firebase 의존성 추가 (#537)

* chore: firebase account 설정파일 주입

* chore: ci, cd에 firebase 주입 경로 추가

* chore: build.gradle에 의존성 추가

* fix: 테스트 이슈 라벨링 파싱 문제 해결

* [test/#516] Member 테스트코드 작성 (#533)

* feat: Member 통합테스트 구현

* feat: Member 통합테스트 구현

* feat: 모든 필드를 커버하는 통합테스트 추가

* feat: 모든 필드를 커버하는 통합테스트 추가

* test: 중복되는 프로필 테스트 제거

* [test/#538] 운동 신청/취소, 게스트 초대/취소 API 테스트 작성 (#539)

* test: 기존에 ExerciseParticipationService에서 Builder 대신 Fixture를 사용하도록 하여 가독성 향상

* test: 나이 조건 테스트를 위해 관련 MemberFixture 추가

* test: 운동 신청 API 단위 테스트 및 통합 테스트 코드 작성

* test: ExerciseCommandService에서 Builder를 쓰던 걸 Fixture를 사용하도록 변경

* test: 운동 신청 API에서 파사드 패턴의 CommandService의 단위 테스트 구현

* test: 운동 신청 취소 API의 통합 테스트 및 단위 테스트 코드 작성

* refactor: 운동 신청, 운동 신청 취소, 게스트 초대, 게스트 초대 취소 API를 ResponseEntity로 응답을 감싸 status도 정상적으로 반환할 수 있도록 변경

* chore: 불필요한 의존성 제거 및 줄바꿈 변경

* test: 게스트 초대 API 통합 테스트 및 CommandService 단위 테스트 코드, GusetService 단위 테스트 코드 작성

* test: 게스트 초대 취소 API 통합 테스트, CommandService 단위 테스트, GuestService 단위 테스트 코드 작성

* [feat/#540] FCM 의존성 추가 및 알림 기능 구현 (#541)

* [feat/#540] Firebase 주입방식 변경 (#542)

* chore: firebase 주입방식 변경

* chore: 설정파일 추가

* chore: firebase 키 재발급

* feat: firebaseConfig파일 생성

* feat: 설정파일 추가

* feat: FCM 알림 기능 구현

* test: 단위테스트 작성

* chore: valid추가

* test: 통합테스트 config에 mock 추가

* chore: 환경변수 주입 방식으로 변경

* [feat/#540] firebase 환경변수 주입방식으로 변경, docker-compose파일 수정 (#543)

* chore: firebase 주입방식 변경

* chore: 설정파일 추가

* chore: firebase 키 재발급

* feat: firebaseConfig파일 생성

* feat: 설정파일 추가

* feat: FCM 알림 기능 구현

* test: 단위테스트 작성

* chore: valid추가

* test: 통합테스트 config에 mock 추가

* chore: 환경변수 주입 방식으로 변경

* chore: docker-compose 파일 수정

* [debug/#544] 모임 추천 조회 (콕플 추천 모드) 검색 기능 누락 해결

* [debug/#547] 프로필 이름 변경 시 내 채팅방 목록이 전부 내 이름으로 변경되는 버그 (#548)

* debug: 프로필 이름 변경 시 상대방의 채팅방 목록에서 내 이름이 변경되도록 버그 수정

* test: 프로필 수정 메서드에서 채팅방 이름 변경 로직 단위 테스트 작성

* improve: 통합 테스트에서 실제 스토리지에 데이터를 넣는 건 적절하지 않으므로 mock과 stub으로 대체

* test: 프로필 수정 API 통합 테스트 작성

* [feat/#546] Flyway 도입을 위한 설정 추가 (#549)

* chore: flyway설정파일 추가

* chore: flyway 설정 추가

* chore: test에서도 flyway설정이 되도록 수정

---------

Co-authored-by: Dmori <83327857+Dimo-2562@users.noreply.github.com>
Co-authored-by: Yumin Kwon <dbalsrltk@naver.com>
Co-authored-by: dmori <chosehoon2@naver.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kanghana1 kanghana1 kanghana1 approved these changes

@dbalsk dbalsk Awaiting requested review from dbalsk

Assignees

@Dimo-2562 Dimo-2562

Labels

🕷️ BUG Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[DEBUG] 프로필 이름 변경 시 내 채팅방 목록이 전부 내 이름으로 변경되는 버그

2 participants

@Dimo-2562 @kanghana1