Fix: 부분 취소는 거부하고, 환불 금액 및 포인트를 결제 기준으로 파생하도록 수정 (#415)

Author: soyeon1806

roome/src/main/java/com/roome/domain/payment/controller/PaymentController.java

@@ -96,7 +96,8 @@ public ResponseEntity<Void> failPayment(
   @Operation(summary = "결제 취소 (환불)", description = "결제 취소 요청을 처리하고 환불을 진행한다.")
   @ApiResponses(value = {
       @ApiResponse(responseCode = "200", description = "결제 취소 성공"),
-      @ApiResponse(responseCode = "400", description = "잘못된 결제 취소 요청 (INVALID_PAYMENT_CANCEL)"),
+      @ApiResponse(responseCode = "400", description = "잘못된 결제 취소 요청 / 부분 취소 미지원 (PARTIAL_CANCEL_NOT_SUPPORTED)"),
+      @ApiResponse(responseCode = "403", description = "본인의 결제가 아님 (PAYMENT_ACCESS_DENIED)"),
       @ApiResponse(responseCode = "404", description = "해당 결제 정보를 찾을 수 없음 (PAYMENT_NOT_FOUND)"),
       @ApiResponse(responseCode = "500", description = "서버 내부 오류")
   })

roome/src/main/java/com/roome/domain/payment/service/PaymentService.java

@@ -278,11 +278,14 @@ public PaymentResponseDto cancelPayment(Long userId, String paymentKey, String c
       throw new BusinessException(ErrorCode.PAYMENT_ALREADY_USED);
     }
 
-    // 환불 금액이 판매 중인 상품 가격 단위인지 검증하고, 차감 포인트를 카탈로그에서 파생한다.
-    if (cancelAmount == null) {
-      throw new BusinessException(ErrorCode.INVALID_REFUND_AMOUNT);
+    // 부분 취소는 지원하지 않으므로 cancelAmount가 주어졌다면 결제 전액과 일치해야 함
+    if (cancelAmount != null && cancelAmount != payment.getAmount()) {
+      throw new BusinessException(ErrorCode.PARTIAL_CANCEL_NOT_SUPPORTED);
     }
-    PointProduct refundProduct = PointProduct.findByPrice(cancelAmount)
+
+    // 환불 금액과 차감 포인트는 클라이언트 값이 아니라 이 결제에서 파생
+    int refundAmount = payment.getAmount();
+    PointProduct refundProduct = PointProduct.findByPrice(refundAmount)
         .orElseThrow(() -> new BusinessException(ErrorCode.INVALID_REFUND_AMOUNT));
     int refundPoints = refundProduct.getPoints();
 
@@ -295,17 +298,17 @@ public PaymentResponseDto cancelPayment(Long userId, String paymentKey, String c
     // 결제 상태 업데이트
     payment.markCanceled(LocalDateTime.now());
 
-    saveRefundLog(payment, cancelAmount, paymentKey);
+    saveRefundLog(payment, refundAmount, paymentKey);
 
-    // Toss API에 결제 취소 요청
+    // Toss API에 결제 취소 요청 (전액 취소)
     boolean isCanceled = tossPaymentClient.cancelPayment(payment.getPaymentKey(), cancelReason,
-        cancelAmount);
+        refundAmount);
     if (!isCanceled) {
       throw new BusinessException(ErrorCode.PAYMENT_CANCEL_FAILED);
     }
 
     log.info("결제 취소 완료: paymentKey={}, userId={}, refundPoints={}, refundAmount={}",
-            paymentKey, userId, refundPoints, cancelAmount);
+            paymentKey, userId, refundPoints, refundAmount);
 
     return PaymentResponseDto.builder()
         .orderId(payment.getOrderId())

roome/src/main/java/com/roome/global/exception/ErrorCode.java

@@ -74,6 +74,7 @@ public enum ErrorCode {
   PAYMENT_ALREADY_USED(HttpStatus.BAD_REQUEST, "포인트를 이미 사용하여 환불할 수 없습니다."),
   PAYMENT_ALREADY_PROCESSED(HttpStatus.BAD_REQUEST, "이미 처리된 결제입니다."),
   ORDER_ID_ALREADY_EXISTS(HttpStatus.CONFLICT, "이미 사용 중인 주문 ID입니다."),
+  PARTIAL_CANCEL_NOT_SUPPORTED(HttpStatus.BAD_REQUEST, "부분 취소는 지원하지 않습니다. 전액 취소만 가능합니다."),
   POINT_PURCHASE_NOT_FOUND(HttpStatus.NOT_FOUND, "포인트 결제 정보를 찾을 수 없습니다."),
   // 서평 관련 예외
   MY_BOOK_REVIEW_NOT_FOUND(HttpStatus.NOT_FOUND, "서평을 찾을 수 없습니다."),

roome/src/test/java/com/roome/domain/payment/service/PaymentServiceTest.java

@@ -220,6 +220,43 @@ void cancelPayment_NoUsage_Success() {
     inOrder.verify(tossPaymentClient).cancelPayment("pk123", "단순 변심", 5_000);
   }
 
+  @Test
+  @DisplayName("cancelAmount가 결제 전액과 다르면 부분 취소로 간주해 거부되어야 한다.")
+  void cancelPayment_PartialAmount_Rejected() {
+    // given: 5,000원 결제에 1,000원만 취소 요청
+    Payment payment = successPayment("pk123");
+    when(paymentRepository.findByPaymentKey("pk123")).thenReturn(Optional.of(payment));
+    when(pointHistoryRepository.hasUsedPointsAfter(eq(1L), eq(payment.getApprovedAt()),
+        anyList())).thenReturn(false);
+
+    // when & then
+    assertThatThrownBy(() -> paymentService.cancelPayment(1L, "pk123", "단순 변심", 1_000))
+        .isInstanceOf(BusinessException.class)
+        .hasMessageContaining(ErrorCode.PARTIAL_CANCEL_NOT_SUPPORTED.getMessage());
+
+    verify(pointService, never()).usePoints(any(), any());
+    verify(tossPaymentClient, never()).cancelPayment(any(), any(), any());
+  }
+
+  @Test
+  @DisplayName("cancelAmount가 없으면 결제 전액 기준으로 취소되어야 한다.")
+  void cancelPayment_NullAmount_FullCancel() {
+    // given
+    Payment payment = successPayment("pk123");
+    when(paymentRepository.findByPaymentKey("pk123")).thenReturn(Optional.of(payment));
+    when(pointHistoryRepository.hasUsedPointsAfter(eq(1L), eq(payment.getApprovedAt()),
+        anyList())).thenReturn(false);
+    when(tossPaymentClient.cancelPayment("pk123", "단순 변심", 5_000)).thenReturn(true);
+
+    // when: cancelAmount = null
+    PaymentResponseDto response = paymentService.cancelPayment(1L, "pk123", "단순 변심", null);
+
+    // then: 결제 전액(5,000)과 카탈로그 파생 포인트(550)로 취소
+    assertThat(response.getStatus()).isEqualTo(PaymentStatus.CANCELED);
+    verify(pointService).usePoints(testUser, PointReason.POINT_REFUND_550);
+    verify(tossPaymentClient).cancelPayment("pk123", "단순 변심", 5_000);
+  }
+
   @Test
   @DisplayName("환불 기한(7일)이 지난 결제는 최근에 다른 구매가 있어도 환불이 거부되어야 한다.")
   void cancelPayment_ApprovedOverSevenDaysAgo_PeriodExceeded() {