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

[FINDY-34] feat: 마커 삭제 #24

Merged
merged 4 commits into from
Nov 17, 2024
Merged

[FINDY-34] feat: 마커 삭제 #24

merged 4 commits into from
Nov 17, 2024

Conversation

Hoya324
Copy link
Member

@Hoya324 Hoya324 commented Nov 17, 2024

✅ 작업 내용

  • feat: marker 삭제 API
    • Marker 삭제를 위한 API를 구현했습니다.
    • 사용자 ID와 Marker ID를 기반으로 마커를 삭제하며, 잘못된 접근에 대한 예외 처리 로직을 추가했습니다.
  • feat: marker 삭제 API swagger
    • Swagger 문서에 Marker 삭제 API를 명세화했습니다.
    • DELETE /markers/{markerId} 경로와 필요한 요청/응답 정보를 추가했습니다.
  • test: marker 삭제 test
    • Marker 삭제 서비스에 대한 단위 테스트를 추가했습니다.
    • 성공/실패(다른 사용자의 Marker 삭제 시도)의 두 가지 테스트 케이스를 포함합니다.
  • refactor: marker 조회 response 시에 markerId 반환
    • Marker 조회 시 응답 데이터에 markerId를 포함하도록 수정했습니다.
    • 이를 통해 클라이언트에서 Marker 식별이 용이해졌습니다.

🤔 고민했던 부분

  • NullPointerException 문제 해결

    • 테스트에서 Bookmark 객체의 Usernull로 설정되어 NullPointerException이 발생하는 문제가 있었습니다.
    • 이를 해결하기 위해 UserBookmark 간의 관계를 명시적으로 설정하고, Mock 객체에 올바른 값을 주입하도록 수정했습니다.
    • 더불어, validateMarkerUser 메서드에 null 체크를 추가하는 방안을 고민했으며, 안정성을 위해 추가했습니다.
  • Marker 삭제 로직에서 유효성 검증 방식

    • Marker가 현재 사용자 소유인지 확인하는 과정에서 Optional을 활용해 null-safe 검증 로직을 작성했습니다.
    • 예외 메시지가 API 사용자에게 명확히 전달될 수 있도록 메시지와 코드 구조를 개선했습니다.

@Hoya324 Hoya324 added ✨기능 새로운 기능을 추가합니다. 🧪테스트 test를 추가한 pr입니다. labels Nov 17, 2024
@Hoya324 Hoya324 self-assigned this Nov 17, 2024
Copy link

마커 제거

Copy link

Unit Test Results

83 tests   83 ✔️  13s ⏱️
27 suites    0 💤
27 files      0

Results for commit aa18a4a.

Copy link

Code Coverage

File Coverage [93.39%] 🍏
MarkerController.java 100% 🍏
PlaceRepositoryCustomImpl.java 99.12% 🍏
FindAllPagedMarkersService.java 97.73% 🍏
DeleteMarkerService.java 70.45%
Total Project Coverage 87.57% 🍏

@Hoya324 Hoya324 merged commit 8831532 into develop Nov 17, 2024
2 checks passed
@Hoya324 Hoya324 deleted the feat/마커-삭제 branch November 17, 2024 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨기능 새로운 기능을 추가합니다. 🧪테스트 test를 추가한 pr입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant