Skip to content

[Refactor/#117] 클로버 미션 로직 수정#118

Merged
kae-mi merged 3 commits intodevelopfrom
refactor/#117-clover-mission
Nov 24, 2025
Merged

[Refactor/#117] 클로버 미션 로직 수정#118
kae-mi merged 3 commits intodevelopfrom
refactor/#117-clover-mission

Conversation

@kae-mi
Copy link
Copy Markdown
Contributor

@kae-mi kae-mi commented Nov 24, 2025

✅ PR 타입 (하나 이상 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 리팩토링 / 코드 개선
  • 의존성 / 환경 설정 변경
  • 버그 수정
  • 기타 (하단에 설명)

 

✨ 어떤 내용인가요?

포토 미션이 아닌 다른 타입의 미션에서도 선택적으로 이미지를 저장할 수 있도록 개선했습니다.
미션의 상태를 STARTED에서도 STARTED로 변경하는 api를 호출할 수 있도록 수정했습니다.

 

🔍 작업 상세 내용

  • 클로버 미션의 상태를 STARTED로 변경하는 api의 로직 수정
    • 기존에는 STARTED 상태에서 해당 api를 호출하면 에러 발생했음 -> 수정해서 STARTED에서도 api 호출할 수 있도록 함
  • 클로버 미션 기록 추가/수정 로직 개선
    • 포토 미션: 이미지 필수 (기존 동작 유지)
    • 기타 미션(TIMER, DISTANCE, VISIT): 이미지 선택적 저장 가능
    • 이미지 URL 존재 여부로 분기 처리하도록 변경
  • 테스트 코드 추가
    • 비포토 타입 미션에 이미지 포함 피드백 추가/수정 케이스 추가
    • TIMER, DISTANCE, VISIT 타입별 이미지 포함 테스트 작성

 

 

🔗 관련 이슈

#117

- 최초 기록하는 api의 addMissionRecord() 메서드 수정
- 기록 업데이트하는 api의 updateMissionRecord() 메서드 수정
- 사진 미션이 아니어도 사진 추가되도록 검증하는 테스트 추가
@kae-mi kae-mi self-assigned this Nov 24, 2025
@github-actions
Copy link
Copy Markdown

📊 코드 커버리지 리포트

Overall Project 64.78% -0.04% 🔴
Files changed 92.65% 🍏

File Coverage
CloverMissionRecordService.java 90.32% -2.3% 🍏
CloverMissionRecord.java 82.27% 🍏

@kae-mi kae-mi merged commit a20604d into develop Nov 24, 2025
1 check passed
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