19 lines
1.4 KiB
Markdown
19 lines
1.4 KiB
Markdown
# 관리자 채널후원 정산 리뷰 지적사항 반영 작업 계획
|
|
|
|
- [x] 하위 호환성 유지 이슈는 요구사항 재확인 결과, 기존 이름을 신규 목적 경로로 사용하기로 확정되어 작업 범위에서 제외한다.
|
|
- [x] 엑셀 다운로드 API 테스트에서 `Content-Disposition` 헤더를 실질적으로 검증하도록 보강한다.
|
|
- [x] 관련 테스트와 빌드를 실행해 회귀 여부를 확인한다.
|
|
|
|
## 검증 기록
|
|
|
|
### 1차 반영
|
|
- 무엇을: 엑셀 다운로드 컨트롤러 테스트에서 `Content-Disposition` 헤더를 `getFirst(HttpHeaders.CONTENT_DISPOSITION)`로 조회하고, `attachment; filename*=` 포함 여부를 검증하도록 수정했다.
|
|
- 왜: 기존 `response.headers.contentDisposition` null 체크만으로는 헤더 누락/형식 회귀를 충분히 잡지 못해 테스트 신뢰도를 높이기 위해서다.
|
|
- 어떻게:
|
|
- 코드 변경: `src/test/kotlin/kr/co/vividnext/sodalive/admin/calculate/channelDonation/AdminChannelDonationCalculateControllerTest.kt`
|
|
- 범위 조정: 하위 호환성 유지 이슈는 요구사항 재확인 결과 작업 제외로 확정
|
|
- 실행 결과:
|
|
- `lsp_diagnostics (AdminChannelDonationCalculateControllerTest.kt)` → Kotlin LSP 미설정으로 진단 불가
|
|
- `./gradlew test --tests "*AdminChannelDonationCalculateControllerTest"` → 성공
|
|
- `./gradlew build` → 성공
|