Refactor/좌석테스트 로직분리 (#236)

* Fix: EnableJpaAuditing이 테스트 코드에서 실행되지 않도록 분리

* Refactor: 좌석 조회 성공 컨트롤러 단위 테스트 작성

* Refactor: 좌석 조회 실패 컨트롤러 단위 테스트 작성

* Refactor: 좌석 선택 성공 컨트롤러 단위 테스트 작성

* Refactor: 미지정석 선택 성공 컨트롤러 단위 테스트 작성

* Refactor: 좌석 5개 이상 선택 시 실패 컨트롤러 단위 테스트 작성

* Refactor: 좌석 선택 실패 케이스 모두 작성

* Refactor: 좌석 취소 성공 테스트 케이스 작성

* Refactor: 좌석 취소 실패 테스트 케이스 작성

* Refactor: 예외 처리로 무조건 404 반환하는 버그 해결

* Refactor: 좌석 조회 성공 및 실패 서비스 단위 테스트 작성

* Refactor: 지정석 성공 및 실패 서비스 단위 테스트 작성

* Refactor: 미지정석 성공 및 실패 서비스 단위 테스트 작성

* Refactor: 좌석 취소 성공 및 실패 서비스 단위 테스트 작성

* Refactor: 주석 처리

Author: ohnoesganj

service/main-server/src/main/java/org/codeNbug/mainserver/domain/seat/dto/SeatLayoutResponse.java

@@ -10,14 +10,16 @@
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
 
+import lombok.AllArgsConstructor;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 
 @Getter
+@NoArgsConstructor
+@AllArgsConstructor
 public class SeatLayoutResponse {
-
-	private final List<SeatDto> seats;
-	private final List<List<String>> layout;
+	private List<SeatDto> seats;
+	private List<List<String>> layout;
 
 	@Getter
 	@NoArgsConstructor

service/main-server/src/main/java/org/codeNbug/mainserver/domain/seat/service/SeatService.java

@@ -14,6 +14,8 @@
 import org.codeNbug.mainserver.domain.seat.entity.SeatLayout;
 import org.codeNbug.mainserver.domain.seat.repository.SeatLayoutRepository;
 import org.codeNbug.mainserver.domain.seat.repository.SeatRepository;
+import org.codeNbug.mainserver.global.exception.globalException.BadRequestException;
+import org.codeNbug.mainserver.global.exception.globalException.ConflictException;
 import org.springframework.stereotype.Service;
 
 import jakarta.transaction.Transactional;
@@ -48,7 +50,7 @@ public SeatLayoutResponse getSeatLayout(Long eventId, Long userId) {
 			throw new IllegalArgumentException("로그인된 사용자가 없습니다.");
 		}
 		SeatLayout seatLayout = seatLayoutRepository.findByEvent_EventId(eventId)
-			.orElseThrow(() -> new IllegalArgumentException("행사가 존재하지 않습니다."));
+			.orElseThrow(() -> new IllegalArgumentException("해당 이벤트에 좌석 레이아웃이 존재하지 않습니다."));
 
 		log.info("SeatLayout ID: {}", seatLayout.getId());
 
@@ -75,15 +77,18 @@ public SeatSelectResponse selectSeat(Long eventId, SeatSelectRequest seatSelectR
 			.orElseThrow(() -> new IllegalArgumentException("행사가 존재하지 않습니다."));
 
 		List<Long> selectedSeats = seatSelectRequest.getSeatList();
-		if (selectedSeats.size() > 4) {
-			throw new IllegalArgumentException("최대 4개의 좌석만 선택할 수 있습니다.");
-		}
 
 		if (event.getSeatSelectable()) {
 			// 지정석 예매 처리
+			if (selectedSeats != null && selectedSeats.size() > 4) {
+				throw new BadRequestException("최대 4개의 좌석만 선택할 수 있습니다.");
+			}
 			selectSeats(selectedSeats, userId, eventId, true, seatSelectRequest.getTicketCount());
 		} else {
 			// 미지정석 예매 처리
+			if (selectedSeats != null && !selectedSeats.isEmpty()) {
+				throw new BadRequestException("[selectSeats] 미지정석 예매 시 좌석 목록은 제공되지 않아야 합니다.");
+			}
 			selectSeats(null, userId, eventId, false, seatSelectRequest.getTicketCount());
 		}
 
@@ -110,24 +115,20 @@ private void selectSeats(List<Long> selectedSeats, Long userId, Long eventId, bo
 					.orElseThrow(() -> new IllegalArgumentException("[selectSeats] 좌석이 존재하지 않습니다."));
 
 				if (!seat.isAvailable()) {
-					throw new IllegalStateException("[selectSeats] 이미 예매된 좌석입니다. seatId = " + seatId);
+					throw new ConflictException("[selectSeats] 이미 예매된 좌석입니다. seatId = " + seatId);
 				}
 
 				reserveSeat(seat, userId, eventId, seatId);
 			}
 		} else {
 			// 미지정석 예매 처리
-			if (selectedSeats != null && !selectedSeats.isEmpty()) {
-				throw new IllegalArgumentException("[selectSeats] 미지정석 예매 시 좌석 목록은 제공되지 않아야 합니다.");
-			}
-
 			List<Seat> availableSeats = seatRepository.findAvailableSeatsByEventId(eventId)
 				.stream()
 				.limit(ticketCount)
 				.toList();
 
 			if (availableSeats.size() < ticketCount) {
-				throw new IllegalStateException("[selectSeats] 예매 가능한 좌석 수가 부족합니다.");
+				throw new ConflictException("[selectSeats] 예매 가능한 좌석 수가 부족합니다.");
 			}
 
 			for (Seat seat : availableSeats) {
@@ -150,7 +151,7 @@ private void reserveSeat(Seat seat, Long userId, Long eventId, Long seatId) {
 
 		boolean lockSuccess = redisLockService.tryLock(lockKey, lockValue, Duration.ofMinutes(5));
 		if (!lockSuccess) {
-			throw new IllegalStateException("[reserveSeat] 이미 선택된 좌석이 있습니다.");
+			throw new BadRequestException("[reserveSeat] 이미 선택된 좌석이 있습니다.");
 		}
 
 		seat.reserve();
@@ -177,7 +178,7 @@ public void cancelSeat(Long eventId, SeatCancelRequest seatCancelRequest, Long u
 			String lockValue = redisLockService.getLockValue(lockKey);
 
 			if (!redisLockService.unlock(lockKey, lockValue)) {
-				throw new IllegalStateException("[cancelSeat] 좌석 락을 해제할 수 없습니다.");
+				throw new BadRequestException("[cancelSeat] 좌석 락을 해제할 수 없습니다.");
 			}
 
 			Seat seat = seatRepository.findById(seatId)

service/main-server/src/main/java/org/codeNbug/mainserver/global/exception/GlobalExceptionHandler.java

@@ -2,6 +2,7 @@
 
 import org.codeNbug.mainserver.global.dto.RsData;
 import org.codeNbug.mainserver.global.exception.globalException.BadRequestException;
+import org.codeNbug.mainserver.global.exception.globalException.ConflictException;
 import org.codeNbug.mainserver.global.exception.globalException.DuplicateEmailException;
 import org.codenbug.user.security.exception.AuthenticationFailedException;
 import org.springframework.http.HttpStatus;
@@ -146,6 +147,19 @@ public ResponseEntity<RsData<Object>> handleBadRequestException(BadRequestExcept
 			.body(new RsData<>("400-BAD_REQUEST", e.getMessage()));
 	}
 
+	/**
+	 * ConflictException 처리
+	 * 리소스 상태가 요청과 충돌할 때 발생하는 예외를 처리합니다.
+	 *
+	 * @param e 예외 객체
+	 * @return API 응답
+	 */
+	@ExceptionHandler(ConflictException.class)
+	public ResponseEntity<RsData<Object>> handleConflictException(ConflictException e) {
+		return ResponseEntity.status(HttpStatus.CONFLICT)
+			.body(new RsData<>("409-CONFLICT", e.getMessage()));
+	}
+
 	/**
 	 * 일반 예외 처리
 	 * 위에서 처리되지 않은 모든 예외를 처리합니다.

service/main-server/src/main/java/org/codeNbug/mainserver/global/exception/globalException/ConflictException.java

@@ -0,0 +1,7 @@
+package org.codeNbug.mainserver.global.exception.globalException;
+
+public class ConflictException extends RuntimeException {
+	public ConflictException(String message) {
+		super(message);
+	}
+}