PDF 다운로드 API 및 관리자 조회 API 추가#23
Conversation
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
There was a problem hiding this comment.
Pull request overview
문서 변환 작업에 대해 PDF 다운로드 API와 관리자용 전체 작업 조회 API를 추가하고, 이를 지원하기 위해 Service/Repository 계층에 “전체 조회” 기능을 확장한 PR입니다.
Changes:
- 변환 성공(Job SUCCEEDED) 시 PDF 아티팩트를 다운로드하는 엔드포인트 추가 (
GET /api/v1/convert/jobs/{jobId}/download) - 관리자용 전체 작업 조회 엔드포인트 및 응답 DTO 추가,
deadLettered필터 지원 (GET /api/v1/admin/convert/jobs) ConversionJobRepository.findAll()/DocumentConversionService.getAllJobs()추가 및 관련 테스트 보강
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/main/java/com/clearfolio/viewer/controller/ConversionController.java | PDF 다운로드 API 추가 및 체크섬 헤더 제공 |
| src/main/java/com/clearfolio/viewer/controller/AdminController.java | 관리자용 전체 작업 조회 API 추가 및 deadLettered 필터 처리 |
| src/main/java/com/clearfolio/viewer/api/AdminJobListResponse.java | 관리자 작업 목록 응답 DTO 추가 |
| src/main/java/com/clearfolio/viewer/service/DocumentConversionService.java | 전체 작업 조회(getAllJobs) 서비스 API 추가 |
| src/main/java/com/clearfolio/viewer/service/DefaultDocumentConversionService.java | getAllJobs 구현(Repository findAll 위임) |
| src/main/java/com/clearfolio/viewer/repository/ConversionJobRepository.java | findAll Repository API 추가 |
| src/main/java/com/clearfolio/viewer/repository/InMemoryConversionJobRepository.java | findAll 구현 추가 |
| src/test/java/com/clearfolio/viewer/controller/ConversionControllerTest.java | 다운로드 엔드포인트 동작 테스트 추가 및 생성자 변경 반영 |
| src/test/java/com/clearfolio/viewer/controller/AdminControllerTest.java | Admin 조회/필터 테스트 신규 추가 |
| src/test/java/com/clearfolio/viewer/service/DefaultDocumentConversionServiceTest.java | getAllJobs 서비스 동작 테스트 추가 및 테스트 더블 보완 |
| src/test/java/com/clearfolio/viewer/service/ServiceInterfaceDefaultMethodsTest.java | 인터페이스 변경(getAllJobs) 대응 |
| src/test/java/com/clearfolio/viewer/repository/InMemoryConversionJobRepositoryTest.java | findAll 동작 테스트 추가 |
| CHANGELOG.md | 신규 기능/테스트 변경 이력 추가 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- GET /api/v1/convert/jobs/{jobId}/download 엔드포인트를 구현하여 PDF 다운로드 (체크섬 포함) 지원
- GET /api/v1/admin/convert/jobs 엔드포인트를 통해 시스템 전체 작업 조회 및 deadLettered 필터 기능 구현
- 모든 신규 기능에 대해 Repository/Service/Controller 계층의 100% 테스트 커버리지 달성
- 변경 사항을 CHANGELOG.md에 반영
seonghobae
force-pushed
the
feature/download-and-admin-api-14062863546942526553
branch
from
1c41076 to
8f2c88a
Compare
|
Rebased onto main (which was history-reset to a single squashed commit) by replaying the PDF-download + admin-query feature on top; resolved conflicts in ConversionController/repository/service and their tests, and |
Acknowledged. |
현재 시스템에 누락되어 있던 문서 다운로드 기능과 관리자가 시스템 내 변환 작업들을 일괄로 조회할 수 있는 Admin 엔드포인트를 자율적으로 설계하고 구현했습니다. 요구사항에 맞게 코드와 커밋 메시지는 한국어로 작성하였고, 전체 테스트가 100% 통과되도록 보강하였습니다.
PR created automatically by Jules for task 14062863546942526553 started by @seonghobae