✨ [Feature] 방 상태 변경 스케쥴러

[JaBeast]

📌 개요

• 방 상태 변경 스케쥴러

💻 작업사항

• main에 @EnableScheduler 어노테이션 추가
• RoomService에 스케쥴러 코드 추가 (FINISH, FAIL)

💡Issue 번호

• closed ✨ [Feature] 시간이 지난 게시글 status 변경하는 스케줄러 #721

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.77%. Comparing base (e9225ee) to head (50610ef).
Report is 26 commits behind head on dev.

Additional details and impacted files

@@             Coverage Diff              @@
##                dev     #722      +/-   ##
============================================
- Coverage     74.16%   72.77%   -1.39%     
- Complexity      737     1006     +269     
============================================
  Files           166      344     +178     
  Lines          3178     4614    +1436     
  Branches        340      394      +54     
============================================
+ Hits           2357     3358    +1001     
- Misses          677     1079     +402     
- Partials        144      177      +33     

Flag	Coverage Δ
integrationTest	67.14% <ø> (-1.14%)	⬇️
unitTest	33.18% <ø> (+3.98%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[AYoungSn]

• RoomService 클래스가 담당하고 있는 역할이 아주 많아보이는데, 이러면 나중에 코드를 보는게 어려워서 역할별로 분리해주시면 좋을것 같습니다.
• 저희 기존 스케줄러 확인 해보셨나요? 참고해보시면 좋을것 같습니다!
• 현재 update 문을 setter 처럼 쓰고 계신데, 어떤 상황에 엔티티가 변경되는지에 대해 함수를 보고 파악하기 어려울것 같다는 생각입니다. 혹시 이 부분 고민해보셨을까요?
• 현재 update 함수 호출 이후 save 바로 호출하고 계신데, 바로 호출하시는 이유가 있는지 궁금합니다.

[JaBeast]

Main 원상 복귀하고 기존 scheduler 참고해서 따로 분리시켰습니다.

update문에 어떤게 업데이트 되는지 있어서 문제 없다고 생각합니다.

update 이후 save한거는 혹시 저장 안될까봐 추가로 넣었습니다. 필요없으면 지울게요.

[AYoungSn]

벌크 업데이트 쿼리로도 해볼수있을것 같네요
수고하셨습니다!

[JaBeast]

곧 테스트하며 터질걸 대비해서 벌크 업데이트 쿼리 메모하겠습니다. 코드 봐주시고 제가 모르던 기능들과 해결책들 제시해주셔서 감사합니다.

[AreSain]

고생하셨습니다. 저희 서비스 활성화에 따른 게시글 남기기는 고민을 좀 해봐야할 것 같아요!
스케줄러는 기존 5분형식으로 되어있는데, 만약 프론트랑 상의 후 15분 단위로 나눠야할수도 있을 것 같습니다!

[yes-ee]

고생하셨습니다. 저희 서비스 활성화에 따른 게시글 남기기는 고민을 좀 해봐야할 것 같아요! 스케줄러는 기존 5분형식으로 되어있는데, 만약 프론트랑 상의 후 15분 단위로 나눠야할수도 있을 것 같습니다!

아까 안 그래도 몇 분 단위로 할 건지 프론트 쪽에서 물어봤는데, 나중에 다시 얘기해 봐요

[yes-ee]

수고하셨습니다!