[Fix] 마이페이지 동아리 수정 오류 수정

.gitignore

@@ -39,3 +39,5 @@ out/
 ### .env file ###
 *.env
 
+### test file ###
+/src/test/*

build.gradle

@@ -65,4 +65,18 @@ dependencies {
 
 tasks.named('test') {
 	useJUnitPlatform()
+	// 테스트 JVM도 확실히 UTF-8
+	jvmArgs '-Dfile.encoding=UTF-8'
+	testLogging {
+		events "FAILED", "SKIPPED", "STANDARD_ERROR"
+		exceptionFormat "FULL"
+		showCauses true; showExceptions true; showStackTraces true
+	}
+}
+tasks.withType(JavaCompile).configureEach {
+	options.encoding = 'UTF-8'
+}
+
+tasks.withType(Javadoc).configureEach {
+	options.encoding = 'UTF-8'
 }

src/main/java/com/WhoIsRoom/WhoIs_Server/domain/user/service/UserService.java

@@ -21,10 +21,7 @@
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.util.LinkedHashSet;
-import java.util.List;
-import java.util.Objects;
-import java.util.Set;
+import java.util.*;
 import java.util.stream.Collectors;
 
 @Slf4j
@@ -117,48 +114,59 @@ private void updateUserNickName(User user, String newNickName) {
 
     private void updateUserClubs(User user, List<Long> newClubIdList) {
 
-        // null이면 빈 리스트로 간주 => 모두 탈퇴 처리
-        Set<Long> requested = newClubIdList == null ? Set.of()
+        Long userId = user.getId();
+
+        // 1) 요청 정규화 (null => 빈 집합, 중복 제거)
+        Set<Long> targetClubIds = newClubIdList == null ? Set.of()
                 : newClubIdList.stream()
                 .filter(Objects::nonNull)
-                .collect(Collectors.toCollection(LinkedHashSet::new)); // 순서 유지 필요시
+                .collect(Collectors.toCollection(LinkedHashSet::new));
 
-        Long userId = user.getId();
+        // === 전체 요청 clubId 기준 존재 여부 검증 ===
+        validateClubExistence(targetClubIds);
 
-        // 현재 가입된 clubId 목록
-        Set<Long> current = new LinkedHashSet<>(memberRepository.findClubIdsByUserId(userId));
+        // 2) 현재 가입된 clubId 목록 조회
+        Set<Long> currentClubIds = new HashSet<>(memberRepository.findClubIdsByUserId(userId));
 
-        // 계산: 추가/삭제 집합
-        Set<Long> toAdd = new LinkedHashSet<>(requested);
-        toAdd.removeAll(current);
+        // 3) 추가할 것과 제거할 것 계산
+        Set<Long> toAdd = new HashSet<>(targetClubIds);
+        toAdd.removeAll(currentClubIds);   // 새로 추가할 것만 남김
 
-        Set<Long> toRemove = new LinkedHashSet<>(current);
-        toRemove.removeAll(requested);
+        Set<Long> toRemove = new HashSet<>(currentClubIds);
+        toRemove.removeAll(targetClubIds); // 요청에 없는 건 제거
 
-        // 삭제 먼저 (없으면 no-op)
+        // 4) 제거 실행
         if (!toRemove.isEmpty()) {
             memberRepository.deleteByUserIdAndClubIdIn(userId, toRemove);
         }
 
-        // 추가할 Club의 존재성 검증
+        // 5) 추가할 Club 존재 여부 검증
         if (!toAdd.isEmpty()) {
             List<Club> clubs = clubRepository.findAllById(toAdd);
-
             if (clubs.size() != toAdd.size()) {
-                // 어떤 ID는 존재X
                 throw new BusinessException(ErrorCode.CLUB_NOT_FOUND);
             }
 
-            // Member 엔티티 생성
+            // 6) 추가 실행
             List<Member> newMembers = clubs.stream()
                     .map(club -> Member.builder()
                             .user(user)
                             .club(club)
                             .build())
                     .toList();
 
-            // 저장 (유니크 제약 (user_id, club_id) 있어도 toAdd는 중복이 아님)
             memberRepository.saveAll(newMembers);
         }
     }
+
+    private void validateClubExistence(Set<Long> clubIds) {
+        if (clubIds == null || clubIds.isEmpty()) {
+            return; // 요청이 없으면 패스
+        }
+
+        List<Club> clubs = clubRepository.findAllById(clubIds);
+        if (clubs.size() != clubIds.size()) {
+            throw new BusinessException(ErrorCode.CLUB_NOT_FOUND);
+        }
+    }
 }