Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: 스케줄러 구성을 개선합니다. #465

Merged
merged 3 commits into from
Mar 14, 2024
Merged

Conversation

hseong3243
Copy link
Collaborator

⛏ 작업 사항

  • 스케줄러 configuration을 추가하였습니다.
    • @EnableScheduling 어노테이션을 해당 클래스로 이동시켰습니다.
    • 기존 스케줄러 빈들을 수동 등록으로 변경했습니다.
    • scheduler.enabled라는 설정을 추가하여 테스트 환경에서 스케줄러가 작동하지 않도록 하였습니다.
  • Recruitment 엔티티의 필드 RecruitmentApplicantCount@Embedded가 빠진 것을 발견하여 해당 pr에 포함시켰습니다.

📝 작업 요약

  • 스케줄러 구성 개선 및 테스트 환경에서 비활성화
  • Recruitment 엔티티 @Embedded 누란된 부분 추가

💡 관련 이슈

기존 스케줄러에서 `@Component` 제거 및 수동 빈 등록으로 변경, 테스트 환경 비활성화를 위한 설정을 추가함.
@hseong3243 hseong3243 self-assigned this Mar 14, 2024
@hseong3243 hseong3243 linked an issue Mar 14, 2024 that may be closed by this pull request
@github-actions github-actions bot added the 🔨 Refactor 코드 리팩토링 및 구조 개선 label Mar 14, 2024
Copy link

sonarcloud bot commented Mar 14, 2024

Copy link

Test Results

285 files   - 4  285 suites   - 4   23s ⏱️ -4s
654 tests  - 5  654 ✅  - 5  0 💤 ±0  0 ❌ ±0 
663 runs   - 5  663 ✅  - 5  0 💤 ±0  0 ❌ ±0 

Results for commit e9114d3. ± Comparison against base commit 80591cd.

This pull request removes 5 tests.
com.clova.anifriends.global.scheduler.NotifySchedulerTest ‑ notifyADayBeforeVolunteer 메서드 실행 시
com.clova.anifriends.global.scheduler.NotifySchedulerTest ‑ notifyEncourageCheckAttendance 메서드 실행 시
com.clova.anifriends.global.scheduler.NotifySchedulerTest ‑ notifyEncourageWriteReview 메서드 실행 시
com.clova.anifriends.global.scheduler.NotifySchedulerTest ‑ notifyThreeDayBeforeVolunteer 메서드 실행 시
com.clova.anifriends.global.scheduler.ServiceSchedulerTest ‑ autoCloseRecruitment 메서드 실행 시

Copy link
Member

@bjo6300 bjo6300 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

@hseong3243 hseong3243 merged commit ac77a50 into dev Mar 14, 2024
5 checks passed
@hseong3243 hseong3243 deleted the refactor/#464 branch March 14, 2024 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 리팩토링 및 구조 개선
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

refactor: 스케줄러 구성을 개선합니다.
2 participants