fix: 스케줄러에 의한 예약 만료 및 재할당 코드

[jhj9109]

동작 확인 해보지 않은 코드입니다. 테스트 후, 수정이 필요하다면 수정후 반영해주셔야합니다. (제가 개발진이 아니라, 할 수 있는게 없습니다^^.....)

개요

fixes #810 스케줄러에서 동작하는 예약 만료 처리 및 재할당하는 코드에서 논리적 문제가 존재하여 올바르게 동작하지 않습니다.

작업 사항

scheduler.ts

• notifyReservationOverdue(), notifyReservation()는 하나로 통합되었습니다. notifyReservationOverdueAndNotifyReservation()
• 이와 함꼐 midnightScheduler에서 예약만료처리 코드를 삭제했습니다.
• 슬랙메시지 전송은 낮이 바람직하고, 일련의 처리를 굳이 새벽과 아침에 나누어 작동해야할 필요성을 크게 느끼지 못했습니다.
• 단, 추후 보완한다면 (기존 로직과 동일하게) 새벽시간대 쿼리 관련 로직을 처리하고, 아침시간대 슬랙메시지를 전송하도록 업데이트 할 수 있을것입니다.

notification.service.ts

• notifyReservationOverdue, notifyReservation 코드는 하나로 통합하였습니다.
• 쿼리 구현부 코드는 모두 reservations.service.ts 에 새롭게 작성된 함수로 대체하였습니다. 이에따라 succeedReservation() 코드는 삭제되었습니다.
• 쿼리 실행 과정에서 오류 발생시 에러를 던집니다. 슬랙 메시지 전송 과정에서 에러가 발생하면 로그만 남깁니다.

reservations.service.ts

• 총 3개의 함수가 추가되었습니다.
• handleReservationOverdueAndAssignReservationToNextWaitingUser는 트랜잭션을 설정하고, 슬랙 메시지 전송을 위해 수행 결과를 반환합니다.
• handleReservationOverdue는 예약 만료 처리를 한 뒤, 해당 예약 정보를 반환합니다.
• assignReservationToNextWaitingUser는 만료된 예약정보를 받아 다음 예약자에게 할당합니다.

handleReservationOverdueAndAssignReservationToNextWaitingUser()

• 예약 만료 처리와 다음 예약자에게 할당하는 것이 하나의 트랜잭션안에서 수행되도록 설정합니다.
• 사이드 이펙트 방지를 위해 로직을 처리하는 동안 lending에 ROCK을 설정합니다.

handleReservationOverdue()

• 기존 코드의 논리적 오류를 수정했습니다.
• 예약 만료 처리해야할 데이터 조회 로직을 수정했습니다. status가 3이 아니라, 0이면서 && 책이 할당되었으면서 && 만료일이 지난 예약을 찾습니다.
• 이제 이곳에서 정상적으로 예약만료 처리과정에서 status = 3을 부여합니다.

assignReservationToNextWaitingUser()

• 기존 코드의 논리적 오류를 수정했습니다.
• 다음 예약자를 찾기위한 데이터 조회 로직을 수정했습니다. bookId와 endAt이 null인지 조건이 추가되었습니다.
• 해당 책에 대해서 다음 예약자가 존재하면 할당합니다.
• handleReservationOverdue()를 통해 예약 만료된 책 정보를 1개씩 입력받습니다.

변경점

backend/src/v1 내

• notifications/notifications.service.ts
• reservations/reservations.service.ts
• utils/scheduler.ts

목적

올바르게 작동하도록 수정을 진행했습니다.

스크린샷 (optional)