🔀 :: [#827] - 볼륨 파일 삭제 API 추가

[dolong2]

개요

• 볼륨에 존재하는 파일을 삭제하는 API를 추가합니다.

작업내용

• 볼륨 파일 삭제 관련 예외 추가
• 볼륨 파일 삭제 유스케이스 추가
• 볼륨 파일 삭제 엔드포인트 추가

체크리스트

탬플릿외에 필요한 항목이 있으면 추가해주세요.

• 로컬에서 빌드가 성공하나요?
• 추가(수정)한 코드가 정상적으로 동작하나요?
• pr 타켓 브랜치가 맞게 설정되어 있나요?
• pr에서 작업할 내용만 작업됐나요?
• 기존 API와 호환되지 않는 사항이 있나요?

Summary by CodeRabbit

• 새 기능

  • 볼륨에 저장된 파일을 삭제할 수 있는 API를 추가했습니다.
  • 요청한 파일 경로와 볼륨 소유권을 확인한 후 안전하게 삭제합니다.
  • 존재하지 않는 파일이나 삭제 실패 상황에 대해 구분된 오류 응답을 제공합니다.

• 테스트

  • 볼륨 파일 삭제 요청의 정상 처리와 성공 응답을 검증하는 테스트를 추가했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e58d1014-465c-48ad-9470-1856bee1577e

📥 Commits

Reviewing files that changed from the base of the PR and between f7b4974 and f905560.

📒 Files selected for processing (6)

• src/main/kotlin/com/dcd/server/core/common/error/ErrorCode.kt
• src/main/kotlin/com/dcd/server/core/domain/volume/exception/VolumeFileDeleteFailureException.kt
• src/main/kotlin/com/dcd/server/core/domain/volume/exception/VolumeFileNotFoundException.kt
• src/main/kotlin/com/dcd/server/core/domain/volume/usecase/DeleteVolumeFileUseCase.kt
• src/main/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapter.kt
• src/test/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapterTest.kt

---

Walkthrough

볼륨 파일 미존재 및 삭제 실패 오류를 추가했습니다. 경로와 소유권을 검증하는 삭제 유스케이스를 구현했습니다. DELETE /{volumeId}/files 엔드포인트와 어댑터 테스트를 추가했습니다.

Changes

볼륨 파일 삭제

Layer / File(s)	Summary
삭제 오류 계약 src/main/kotlin/com/dcd/server/core/common/error/ErrorCode.kt, src/main/kotlin/com/dcd/server/core/domain/volume/exception/*	볼륨 파일 미존재와 삭제 실패를 나타내는 오류 코드 및 예외 클래스를 추가했습니다.
삭제 유스케이스 src/main/kotlin/com/dcd/server/core/domain/volume/usecase/DeleteVolumeFileUseCase.kt	워크스페이스와 볼륨 소속을 검증합니다. 파일 경로를 검사하고 파일을 삭제합니다. 파일 연산 예외를 도메인 예외로 변환합니다.
HTTP 엔드포인트와 테스트 src/main/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapter.kt, src/test/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapterTest.kt	DELETE /{volumeId}/files 엔드포인트를 추가했습니다. 유스케이스 호출과 OK 응답을 검증합니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• dolong2/dcd#710: 볼륨 삭제 기능과 동일한 오류 코드, 예외, 어댑터 및 테스트를 수정합니다.
• dolong2/dcd#714: 볼륨 삭제 유스케이스의 워크스페이스 소유권 검증과 관련됩니다.
• dolong2/dcd#718: VolumeWebAdapter의 볼륨 파일 작업과 관련됩니다.

Poem

토끼가 파일 길을 살펴요
안전한 경로만 쏙 지워요
없으면 오류를 알려요
실패하면 예외를 남겨요
이제 볼륨 정리가 깔끔해요 🐇

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 볼륨 파일 삭제 API 추가라는 풀 리퀘스트의 주요 변경 사항을 명확하고 간결하게 설명합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/delete-file-api

---

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.}