fix: Magic Number 제거

[JanooGwan]

🔍 개요

• close #이슈번호

---

🚀 주요 변경 내용

---

💬 참고 사항

---

✅ Checklist (완료 조건)

• 코드 스타일 가이드 준수
• 테스트 코드 포함됨
• Reviewers / Assignees / Labels 지정 완료
• 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 713fbc3 and f1d6a8b.

📒 Files selected for processing (6)

• src/test/java/gg/agit/konect/domain/club/service/ClubApplicationServiceIntegrationTest.java
• src/test/java/gg/agit/konect/domain/club/service/ClubRecruitmentServiceIntegrationTest.java
• src/test/java/gg/agit/konect/domain/club/service/ClubServiceIntegrationTest.java
• src/test/java/gg/agit/konect/domain/schedule/service/ScheduleServiceIntegrationTest.java
• src/test/java/gg/agit/konect/domain/studytime/service/StudyTimerServiceIntegrationTest.java
• src/test/java/gg/agit/konect/support/fixture/ClubRecruitmentFixture.java

---

📝 Walkthrough

Walkthrough

여러 테스트 파일 및 픽스처에서 하드코딩된 값(페이지 크기, 시간 범위, 날짜, 개수)을 상수로 대체하여 코드 유지보수성과 일관성을 개선합니다. 기능은 변하지 않습니다.

Changes

Cohort / File(s)	Summary
Club Application & Service Tests src/test/java/gg/agit/konect/domain/club/service/ClubApplicationServiceIntegrationTest.java, src/test/java/gg/agit/konect/domain/club/service/ClubServiceIntegrationTest.java	DEFAULT_PAGE_SIZE 상수(10) 도입 및 세 개의 테스트 케이스에서 하드코딩된 페이지 크기를 상수로 대체.
Club Recruitment Tests & Fixture src/test/java/gg/agit/konect/domain/club/service/ClubRecruitmentServiceIntegrationTest.java, src/test/java/gg/agit/konect/support/fixture/ClubRecruitmentFixture.java	RECRUITMENT_PERIOD_DAYS 상수(30) 도입 및 여러 테스트 케이스에서 30일 기간의 하드코딩된 값을 상수로 대체.
Schedule Service Tests src/test/java/gg/agit/konect/domain/schedule/service/ScheduleServiceIntegrationTest.java	TEST_YEAR, MARCH, APRIL, FEBRUARY, DAYS_5 등의 시간 관련 상수와 EXPECTED_SCHEDULE_COUNT 상수 도입. 여러 테스트 케이스에서 하드코딩된 날짜 리터럴 및 기댓값을 상수로 대체.
Study Timer Service Tests src/test/java/gg/agit/konect/domain/studytime/service/StudyTimerServiceIntegrationTest.java	MISMATCHED_CLIENT_SECONDS 상수(100L) 도입 및 타이머 중지 실패 테스트에서 하드코딩된 값을 상수로 대체.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 매직 넘버들이 자취를 감추고,
상수로 변신한 숫자들이 춤을 춘다네!
코드의 정원에 일관성 피어나고,
유지보수는 훨씬 쉬워졌으니,
이것이야말로 리팩토링의 진정한 기술! ✨

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/testcode-magic-number

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}