[BE] refactor: 트랜잭션을 팀 컨벤션에 맞게 리팩터링

[70825]

Issue

• close [BE] 로그인 서비스 로직 트랜잭션 컨벤션에 맞게 리팩터링 #283
• 관련 이슈: [BE] 회원 등록에서 회원 저장 트랜잭션이 새로 생성되도록 수정 #200

✨ 구현한 기능

• 클래스단에 readOnly = true 추가
• findOrCreateMember에 @Transactional 추가
• save에 @Transactional(propagtion=REQUIRES_NEW) 제거

같은 클래스에서 findOrCreateMember 호출하고, 내부에서 save를 호출하면 save 메서드 위에 적혀있는 @Transactional이 적용되지 않습니다.
왜냐하면 스프링 트랜잭션 공식 문서에서 아래와 같은 내용이 나와있습니다.
그래서 프록시로 호출하기 때문에 같은 클래스 내부에 부모 메서드 호출 -> 자식 메서드를 호출하면 자식 메서드에 적혀있는 트랜잭션이 무시됩니다.

In proxy mode (which is the default), only external method calls coming in through the proxy are intercepted.
This means that self-invocation (in effect, a method within the target object calling another method of the target object) does not lead to an actual transaction at runtime even if the invoked method is marked with @transactional.
Also, the proxy must be fully initialized to provide the expected behavior, so you should not rely on this feature in your initialization code — for example, in a @PostConstruct method.

만약 이게 적용이 안된다면 findOrCreateMember에 @Transactional이 아닌 @Transactional(propagation=REQUIRES_NEW)로 설정해야합니다.

• REQUIRES_NEW : 부모 트랜잭션이 존재하면, 부모 트랜잭션을 중지하고, 새로운 트랜잭션을 생성하고 실행함. 새로운 트랜잭션이 종료되었으면 부모 트랜잭션을 다시 실행함

📢 논의하고 싶은 내용

• x

🎸 기타

• x

⏰ 일정

• 추정 시간 : 0.1
• 걸린 시간 : 0.1

[github-actions]

Unit Test Results

67 tests   67 ✔️  11s ⏱️
34 suites    0 💤
34 files      0 ❌

Results for commit 36a3f72.

[hanueleee]

신기하네요 자식 트랜잭션 무시..
새로운 사실 알아갑니다~~

[Go-Jaecheol]

역시 트랜잭션 권위자..👍

[wugawuga]

LGTM~👍