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

Member - Video 간 릴레이션 설정 #25

Merged
merged 19 commits into from
Mar 1, 2024
Merged

Conversation

One-armed-boy
Copy link
Owner

  • Member - Video 간 연관관계 설정(1:N)
  • 연관관계 추가로 인한 사이드 이펙트 수정(ex. 테스트 코드)

@One-armed-boy One-armed-boy self-assigned this Mar 1, 2024
Copy link

github-actions bot commented Mar 1, 2024

Overall Project 66.84% -7.85% 🟢
Files changed 66.59% 🟢

File Coverage
UploadFacade.java 100% 🟢
VideoService.java 100% 🟢
VideoController.java 100% 🟢
Member.java 85.29% 🟢
Video.java 83.55% -3.95% 🟢
Role.java 80% 🟢
CreateVideoCommand.java 76.67% -23.33% 🟢
UploadVideoCommand.java 70.93% -29.07% 🟢
ListVideoResponse.java 56.98% -42.11% 🔴
GetVideoResponse.java 55.72% -0.75% 🟢
SecurityContextHelper.java 37.5% -62.5% 🔴
VideoDto.java 32.82% 🟢
UploadController.java 26.47% -64.71% 🔴
LocalStorageStrategy.java 3.26% 🟢
UploadVideoRequest.java 0% -2.9% 🔴

- 아직 배포되지 않았기 때문에 굳이 마이그레이션 파일을 추가하여 서버 실행 컨텍스트의 불확실성을 늘릴 필요가 없음
Copy link

github-actions bot commented Mar 1, 2024

Overall Project 66.84% -7.85% 🟢
Files changed 66.59% 🟢

File Coverage
UploadFacade.java 100% 🟢
VideoService.java 100% 🟢
VideoController.java 100% 🟢
Member.java 85.29% 🟢
Video.java 83.55% -3.95% 🟢
Role.java 80% 🟢
CreateVideoCommand.java 76.67% -23.33% 🟢
UploadVideoCommand.java 70.93% -29.07% 🟢
ListVideoResponse.java 56.98% -42.11% 🔴
GetVideoResponse.java 55.72% -0.75% 🟢
SecurityContextHelper.java 37.5% -62.5% 🔴
VideoDto.java 32.82% 🟢
UploadController.java 26.47% -64.71% 🔴
LocalStorageStrategy.java 3.26% 🟢
UploadVideoRequest.java 0% -2.9% 🔴

@One-armed-boy One-armed-boy merged commit 2bca322 into main Mar 1, 2024
1 check passed
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.

1 participant