fix: 제약 조건 이름 매칭 오탐 방지

Author: pooreumjung

src/main/java/com/semosan/api/common/exception/ConstraintViolationUtils.java

@@ -2,24 +2,25 @@
 
 import org.hibernate.exception.ConstraintViolationException;
 
+import java.util.regex.Pattern;
+
 public final class ConstraintViolationUtils {
 
     private ConstraintViolationUtils() {}
 
-    /**
-     * 예외 체인 전체를 탐색해 특정 DB 제약 조건 이름이 원인인지 확인한다.
-     * Hibernate ConstraintViolationException의 constraintName과 메시지 문자열을 모두 검사해
-     * 드라이버마다 다른 예외 래핑 방식에 대응한다.
-     */
     public static boolean isViolation(Throwable exception, String constraintName) {
+        Pattern constraintNamePattern = Pattern.compile(
+                "(?<![A-Za-z0-9_])" + Pattern.quote(constraintName) + "(?![A-Za-z0-9_])"
+        );
         Throwable current = exception;
         while (current != null) {
             if (current instanceof ConstraintViolationException constraintViolation
                     && constraintName.equals(constraintViolation.getConstraintName())) {
                 return true;
             }
             String message = current.getMessage();
-            if (message != null && message.contains(constraintName)) {
+            // 드라이버마다 다른 예외 래핑에 대응하되, 다른 제약명 일부와 겹치는 오탐은 막는다.
+            if (message != null && constraintNamePattern.matcher(message).find()) {
                 return true;
             }
             current = current.getCause();

src/main/java/com/semosan/api/domain/community/post/service/FreePostReportService.java

@@ -3,6 +3,7 @@
 import com.semosan.api.common.alert.DiscordAlertClient;
 import com.semosan.api.common.alert.dto.DiscordEmbed;
 import com.semosan.api.common.alert.dto.DiscordMessage;
+import com.semosan.api.common.exception.ConstraintViolationUtils;
 import com.semosan.api.common.exception.GeneralException;
 import com.semosan.api.common.status.ErrorStatus;
 import com.semosan.api.domain.community.post.entity.FreePost;
@@ -13,7 +14,6 @@
 import com.semosan.api.domain.user.entity.User;
 import com.semosan.api.domain.user.repository.UserRepository;
 import lombok.RequiredArgsConstructor;
-import org.hibernate.exception.ConstraintViolationException;
 import org.springframework.dao.DataIntegrityViolationException;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
@@ -61,7 +61,7 @@ public void afterCommit() {
             });
             return saved;
         } catch (DataIntegrityViolationException e) {
-            if (isUniqueViolation(e)) {
+            if (ConstraintViolationUtils.isViolation(e, REPORT_UNIQUE_CONSTRAINT)) {
                 throw new GeneralException(ErrorStatus.FREE_POST_REPORT_ALREADY_EXISTS);
             }
             throw e;
@@ -93,22 +93,6 @@ private DiscordMessage buildDiscordMessage(FreePostReport report) {
         );
     }
 
-    private boolean isUniqueViolation(Throwable exception) {
-        Throwable current = exception;
-        while (current != null) {
-            if (current instanceof ConstraintViolationException constraintViolation
-                    && REPORT_UNIQUE_CONSTRAINT.equals(constraintViolation.getConstraintName())) {
-                return true;
-            }
-            String message = current.getMessage();
-            if (message != null && message.contains(REPORT_UNIQUE_CONSTRAINT)) {
-                return true;
-            }
-            current = current.getCause();
-        }
-        return false;
-    }
-
     private User findReporterOrThrow(Long reporterId) {
         return userRepository.findByIdAndDeletedFalse(reporterId)
                 .orElseThrow(() -> new GeneralException(ErrorStatus.USER_NOT_FOUND));

src/test/java/com/semosan/api/common/exception/ConstraintViolationUtilsTest.java

@@ -0,0 +1,52 @@
+package com.semosan.api.common.exception;
+
+import org.hibernate.exception.ConstraintViolationException;
+import org.junit.jupiter.api.Test;
+import org.springframework.dao.DataIntegrityViolationException;
+
+import java.sql.SQLException;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+class ConstraintViolationUtilsTest {
+
+    private static final String CONSTRAINT_NAME = "uk_user_blocks_blocker_blocked";
+
+    @Test
+    void isViolationReturnsTrueWhenHibernateConstraintNameMatches() {
+        ConstraintViolationException cause = new ConstraintViolationException(
+                "duplicate",
+                new SQLException("unique violation"),
+                CONSTRAINT_NAME
+        );
+
+        boolean result = ConstraintViolationUtils.isViolation(
+                new DataIntegrityViolationException("duplicate", cause),
+                CONSTRAINT_NAME
+        );
+
+        assertThat(result).isTrue();
+    }
+
+    @Test
+    void isViolationReturnsTrueWhenMessageContainsExactConstraintToken() {
+        DataIntegrityViolationException exception = new DataIntegrityViolationException(
+                "ERROR: duplicate key value violates unique constraint \"" + CONSTRAINT_NAME + "\""
+        );
+
+        boolean result = ConstraintViolationUtils.isViolation(exception, CONSTRAINT_NAME);
+
+        assertThat(result).isTrue();
+    }
+
+    @Test
+    void isViolationReturnsFalseWhenMessageContainsOnlyConstraintNamePrefix() {
+        DataIntegrityViolationException exception = new DataIntegrityViolationException(
+                "ERROR: duplicate key value violates unique constraint \"" + CONSTRAINT_NAME + "_idx\""
+        );
+
+        boolean result = ConstraintViolationUtils.isViolation(exception, CONSTRAINT_NAME);
+
+        assertThat(result).isFalse();
+    }
+}