[TIL] 스프링 심화 주차 과제 트러블슈팅

YJin·2025년 5월 15일

Spring

[내배캠 Spring 6기_TIL]

목록 보기

35/56

Lv10. QueryDSL

트러블 슈팅 https://velog.io/@yoon17710/TIL-QueryDSL

Lv11. Transaction

문제

매니저 등록 요청 중 실패하더라도 등록 요청 로그를 DB에 저장하여 남겨야 한다.

메소드 동작 흐름

매니저 등록 서비스 -> 서비스 내부에서 요청 로그 DB 저장 -> 매니저 DB 저장 완료

이 과정은 하나의 트랜잭션으로 묶여 있으며, 만약 오류 발생 시 매니저, 로그 둘다 DB에서 저장이 롤백된다.

분석

실패하더라도 등록 요청 로그만은 DB에 저장되도록 하려면, 트랜잭션의 전파(Propagation) 속성을 적절히 활용해야 한다.
그중에서 REQUIRES_NEW 옵션은 기존 트랜잭션에 참여하지 않고 새로운 트랜잭션을 생성하는 설정이다.

따라서 요청 로그를 먼저 REQUIRES_NEW 전파 속성이 적용된 메서드에서 저장하도록 한다.
그러면 이후 매니저 등록 과정에서 예외가 발생해 전체 트랜잭션이 롤백되더라도,
요청 로그는 별도의 트랜잭션으로 이미 커밋되었기 때문에 함께 롤백되지 않는다.

해결

	@Transactional
    public ManagerSaveResponse saveManager(CustomUserDetails userDetails, long todoId, ManagerSaveRequest managerSaveRequest) {
		...
        // 요청 로그 저장
        logService.saveManagerRegisterRequest(user.getId(), managerSaveRequest.getManagerUserId());

        ...
		// 매니저 등록 저장
        Manager newManagerUser = new Manager(managerUser, todo);
        Manager savedManagerUser = managerRepository.save(newManagerUser);
    }

매니저 등록 서비스에는 @Transactional을 걸어주고

  @Transactional(propagation = Propagation.REQUIRES_NEW)
  public void saveManagerRegisterRequest(Long userId, Long managerId) {
    Log log = new Log(userId,
        "Manager register - Request ID: "+userId+", Manager ID: "+managerId);
    logRepository.save(log);
  }

로그 저장 서비스에는 @Transactional(propagation = Propagation.REQUIRES_NEW) 전파 속성을 설정한다.

트러블 슈팅

문제

처음에는 로그 저장을 별도의 서비스로 분리하지 않고 매니저 서비스 내부에서 logRepository.save()를 직접 호출하여 사용하였다.
매니저 등록 서비스에 @Transactional(propagation = Propagation.REQUIRES_NEW)를 적용하고 logRepository.save() 는 try-catch 내부 블록에서 실행하도록 했지만, 예상과 다르게 로그가 저장되지 않았다.
logRepository.save()는 JPA의 save 메서드로, 내부적으로 트랜잭션이 적용되어 있으므로 별도의 트랜잭션이 생성될 거라 생각했지만, 실제로는 REQUIRES_NEW가 적용되지 않았다.

분석

찾아보니 트랜잭션의 전파 속성(propagation)은 외부에서 호출될 때만 적용된다는 것을 알게 되었다.
즉, logRepository.save() 역시 내부적으로 트랜잭션이 적용되어 있지만
REQUIRES_NEW가 아닌 기본 옵션 REQUIRED의 트랜잭션으로 실행되기 때문에,
Transactional(propagation = REQUIRES_NEW)가 선언된 메서드처럼 동작하지 않는다.

따라서 로그 저장을 담당하는 별도의 로그 서비스로 분리하였다.
그 다음 로그 서비스의 로그 저장 메서드에 @Transactional(propagation = REQUIRES_NEW)를 적용한 뒤, 매니저 서비스에서 해당 로그 서비스의 메서드를 호출하는 방식으로 변경하였다.
그 결과 로그 저장이 정상적으로 별도의 트랜잭션으로 분리되어 처리되었다.

테스트 전략

로그 저장이 실제로 DB에 반영되는지 검증하려면 단순한 메서드 호출 확인(mocking)이 아닌 통합 테스트가 필요하다.

➡️ 따라서 @SpringBootTest를 적용한 실제 DB와 연결되는 환경에서 테스트해야 한다.

테스트 코드

@SpringBootTest
@Transactional
class ManagerServiceTest {

  @Autowired
  ManagerService managerService;

  @Autowired
  LogRepository logRepository;

  @Test
  @DisplayName("매니저 등록 로깅 트랜잭션 테스트")
  void saveManager() {
    // given : 매니저 등록을 할 때
    User user = User.builder()
        .email("email@email.com")
        .id(1L)
        .userRole(UserRole.USER)
        .nickname("nick")
        .password("password")
        .build();

    ManagerSaveRequest request = new ManagerSaveRequest(2L);
    CustomUserDetails userDetails = new CustomUserDetails(user);
    Long todoId = 1L;

    // when : 매니저 등록을 할 때
    try {
      managerService.saveManager(userDetails, todoId, request);
    }
    catch (InvalidRequestException e) {}

    // then : 매니저 등록 실패해도 로깅에 성공
    assertEquals(logRepository.count(), 1L);
  }
}