Skip to content
This repository was archived by the owner on Jul 7, 2025. It is now read-only.

Conversation

@tlarbals824
Copy link
Member

  • userId를 nullable로 변경하여 비회원 이미지 업로드 지원.
  • 비회원 업로드 시 기본 ID로 "anonymous" 설정.
  • ImageCommandServiceTest 및 Controller 테스트에 관련 시나리오 추가.
  • API 문서 변경: 액세스 토큰 필드를 선택사항으로 수정하고 설명 추가.

- `userId`를 nullable로 변경하여 비회원 이미지 업로드 지원.
- 비회원 업로드 시 기본 ID로 "anonymous" 설정.
- `ImageCommandServiceTest` 및 Controller 테스트에 관련 시나리오 추가.
- API 문서 변경: 액세스 토큰 필드를 선택사항으로 수정하고 설명 추가.
@tlarbals824 tlarbals824 self-assigned this May 18, 2025
@github-actions
Copy link

github-actions bot commented May 18, 2025

Code Coverage Report

Overall Project 90.69% -0.07% 🍏
Files changed 87% 🍏

File Coverage
ImageMetadata.kt 100% 🍏
ImageCommandService.kt 100% 🍏
AccessUserArgumentResolver.kt 100% 🍏
WebConfig.kt 100% 🍏
ImageController.kt 100% 🍏
SecurityContextHolder.kt 88.89% 🍏
UploadImageUsecase.kt 75% -5% 🍏
S3ImageManagementAdapter.kt 16.98% -20.75%

- `ImageMetadata.owner`를 nullable로 변경하여 비회원 지원.
- 비회원 이미지 업로드 시 기본 ID("anonymous") 설정 로직을 `S3ImageManagementAdapter`로 이동.
- `upload` 메서드의 비회원 처리 로직 단순화.
- `ImageCommandServiceTest` 테스트 구조 개선: 시나리오 분리 및 검증 추가.
- `ImageMetadata` 데이터 클래스 구조 수정.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarQube Cloud

@tlarbals824 tlarbals824 merged commit c6f9d5e into main May 18, 2025
3 of 4 checks passed
@tlarbals824 tlarbals824 deleted the ASAP-456 branch May 18, 2025 08:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants