Skip to content

fix: 다일 일정 인근 지역 확장 허용 - #20

Merged
Heyaaz merged 1 commit into
mainfrom
fix/multi-day-nearby-region
May 20, 2026
Merged

fix: 다일 일정 인근 지역 확장 허용#20
Heyaaz merged 1 commit into
mainfrom
fix/multi-day-nearby-region

Conversation

@Heyaaz

@Heyaaz Heyaaz commented May 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • 다일 일정 생성 시 인근 지역 확장 허용

Changes

  • 후보 부족 시 단일 지역 고정 대신 45km 내 인근 지역 후보 포함
  • 일자 변경 시 전날 마지막 장소와의 거리 패널티 초기화
  • 다일 일정이 예산·아산·천안처럼 인근 지역으로 확장되는 회귀 테스트 추가

Verification

  • Not run
  • Build
  • Test: ./gradlew test --tests 'com.tripsync.application.consensus.ConsensusServiceTest' --no-daemon
  • Test: ./gradlew cleanTest test --no-daemon
  • Manual

Notes

@Heyaaz Heyaaz left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Nogada PR Review

Verdict: ✅ No blocking issues found.

Checked scope:

  • 다일 일정에서 같은 지역 후보가 부족할 때 인근 지역 후보로 확장하는 로직
  • 일자 경계에서 이전 장소 거리 패널티가 다음 날짜 첫 슬롯에 과하게 적용되지 않도록 하는 변경
  • 신규 회귀 테스트(multi day schedule can expand to nearby localities...)

Verification:

  • ./gradlew clean test --no-daemon --no-build-cache 성공
  • Test XML summary: 60 tests, 0 failures, 0 errors, 0 skipped
  • git diff --check origin/main...pr-20 통과

Notes:

  • Testcontainers 종료 시점의 Hikari/PostgreSQL shutdown warning은 있었지만 Gradle 빌드와 XML 결과는 성공입니다.

[Heyaaz의 Nogada]

@Heyaaz
Heyaaz merged commit 1a7d30a into main May 20, 2026
1 check passed
@Heyaaz
Heyaaz deleted the fix/multi-day-nearby-region branch May 20, 2026 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant