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

pullrequest: 피드 서비스 테스트 코드 추가 #76

Merged
merged 4 commits into from
Oct 4, 2024

Conversation

juwon-code
Copy link
Collaborator

#️⃣연관된 이슈

Related to: #71

☑️ PR 유형

  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정

📝 작업 내용

잘못된 줄바꿈 수정

BittaApplication에 잘못 들어간 줄바꿈을 수정합니다.

Commit: 255cfa6

파일 관련 컴포넌트 수정

  • 피드 ID로 파일의 URL을 조회하는 쿼리 메소드를 추가합니다.
  • 피드 삭제 메소드 작동 순서를 설정합니다. (파일 제거 -> 레코드 제거)
  • 엔티티 -> DTO 변환 메소드 잘못된 FK 필드 참조 수정합니다.
  • 불필요한 파일 서비스 메소드 제거합니다.

Commit: 6fb87a0

피드 서비스 테스트 코드 추가

FeedService의 전체 메소드의 성공 실패 테스트 코드 작성합니다.

Commit: f6a4616

파일 업로드 크기 제한

500MB, 1GB로 설정되어있는 파일 제한을 25MB, 50MB로 대폭 낮춥니다.
배포 환경 구축 중에 재설정이 필요할 수 있습니다.

Commit: 185dcff

✅ 피드백 반영사항

파일 업로드 크기를 임의로 제한했습니다.

💬 리뷰 요구사항

변경된 파일이 많기 때문에, 각 브랜치의 최신화가 필요합니다.

BittaApplication에 잘못된 줄바꿈이 있어 수정합니다.

Related to: #71
- 파일 업로드 제한 임시 제거
- 파일 삭제 서비스 및 조회 쿼리 추가
- 피드 삭제 서비스 수정

Related to: #71
피드 서비스 테스트 코드를 추가합니다.

Related to: #71
파일 업로드 크기 제한을 낮춥니다.

Related to: #71
@juwon-code juwon-code self-assigned this Oct 4, 2024
@juwon-code juwon-code linked an issue Oct 4, 2024 that may be closed by this pull request
4 tasks
@juwon-code juwon-code changed the title Dev/test/service pullrequest: 피드 서비스 테스트 코드 추가 Oct 4, 2024
Copy link
Collaborator

@YooJHyun YooJHyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다 고생하셨습니다

Copy link
Collaborator

@ghtndl ghtndl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다 고생하셨습니다

@juwon-code juwon-code merged commit 5d13aa8 into main Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test 서비스 테스트 코드 작성
3 participants