Conversation
Walkthrough
Estimated code review effort: 3 (Moderate) | ~20 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
필수 과목 문항을 15개 → 9개로 개편하고 답변을 6개 유형에 재매핑. - V18 마이그레이션: 기존 연애고사 데이터/제출 결과 리셋 후 9문항 재삽입, is_dating_exam_submitted 초기화로 전원 재응시 유도 - recalculateDominant 동률 우선순위 변경: 자기주도형 > 자극모험형 > 동반성장형 > 현실안정형 > 이성중심형 > 정서교감형 - 검증 로직은 이미 DB 문항 수 기준 동적이라 코드 변경 불필요
stemmmm
changed the title
Test results831 tests 831 ✅ 16s ⏱️ Results for commit 277e493. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/main/resources/db/migration/V18__reset_required_exam_to_9_questions.sql`:
- Around line 1-3: Flyway가 자동 실행하지 않도록 마이그레이션 파일을
src/main/resources/db/migration/에서 docs/db-migrations/로 이동하고 파일명을 .draft.sql
형식으로 변경하세요. SQL 내용은 유지하고, Flyway 스캔 대상 경로에 해당 파일이 남지 않도록 확인하세요.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 4007c1fa-ce4a-4672-801c-f7e0cb1acd5a
📒 Files selected for processing (3)
src/main/java/deepple/deepple/datingexam/domain/DatingExamSubmitResult.javasrc/main/resources/db/migration/V18__reset_required_exam_to_9_questions.sqlsrc/test/java/deepple/deepple/datingexam/domain/DatingExamSubmitResultTest.java
| -- #452 필수 과목 문항 15개 → 9문항 변경 + 답변 6유형 재매핑 | ||
| -- 문항/유형 매핑이 바뀌므로 기존 제출/집계 결과를 초기화하고 전원 재응시를 유도한다. | ||
|
|
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🔴 Critical | ⚡ Quick win
마이그레이션 파일 경로가 PR 설명과 일치하지 않습니다.
PR 설명에 따르면 Flyway 자동 실행을 방지하기 위해 마이그레이션 파일을 docs/db-migrations/ 경로에 .draft.sql 형식으로 배치해야 하지만, 현재 파일은 Flyway 스캔 대상인 src/main/resources/db/migration/ 경로에 위치해 있습니다.
이 상태로 애플리케이션이 실행되면 마이그레이션이 자동 적용되어 의도치 않게 기존 데이터가 모두 삭제될 위험이 있습니다. PR 설명의 의도대로 Flyway가 인식하지 않도록 파일 경로를 docs/ 하위로 이동해 주시기 바랍니다.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/main/resources/db/migration/V18__reset_required_exam_to_9_questions.sql`
around lines 1 - 3, Flyway가 자동 실행하지 않도록 마이그레이션 파일을
src/main/resources/db/migration/에서 docs/db-migrations/로 이동하고 파일명을 .draft.sql
형식으로 변경하세요. SQL 내용은 유지하고, Flyway 스캔 대상 경로에 해당 파일이 남지 않도록 확인하세요.
1 participant
개요
필수 과목 문항을 15개 → 9개로 개편하고, 답변을 6개 성격 유형에 재매핑합니다. (#453의 6유형 기반)
변경 사항
src/main/resources/db/migration/V18__reset_required_exam_to_9_questions.sql)dating_exam_submit_result) 리셋is_dating_exam_submitted = FALSE로 전원 재응시 유도DatingExamSubmitResult.recalculateDominant)유형 매핑 (한글명 → enum)
문항·유형 매핑이 바뀌어 과거 집계(구 15문항·4유형 기준)가 무의미하므로 기존 결과를 삭제하고 재응시를 유도합니다. 배포 시 모든 회원의 대표 유형이 초기화됩니다.
마이그레이션 방식
환경 독립적인 리셋 + 재삽입 방식입니다(하드코딩 ID 미사용). 문항/답변 ID는 재생성됩니다.
테스트
Closes #452