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

[NB-24] 앨범 도메인 통합 테스트 구현 #50

Merged
merged 2 commits into from
Sep 9, 2024
Merged

Conversation

Aram-su
Copy link
Contributor

@Aram-su Aram-su commented Sep 6, 2024

개요

  • 앨범 도메인 통합 테스트 구현
  • AlbumIntegrationTest 구현

PR 유형

어떤 변경 사항이 있나요?

  • 테스트 추가, 테스트 리팩토링

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고 (Ctrl + 클릭하세요.)
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

- AlbumIntegrationTest 구현
Copy link
Contributor

@liljoon liljoon left a comment

Choose a reason for hiding this comment

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

앨범 관련 통합 테스트 구현된 것 확인했습니다.
jira 티켓과 연결을 깜빡하신 것 같습니다. 그것만 추가하시면 좋을 것 같습니다.

@Aram-su Aram-su changed the title 앨범 도메인 통합 테스트 구현 [NB-24] 앨범 도메인 통합 테스트 구현 Sep 9, 2024
- AlbumIntegrationTest 수정
 - 반환되는 photoUrls가 cdnUrl로 시작하는지 테스트
 - 사진 업로드 URL 생성 시 s3BucketUrlPrefix로 시작하는지 테스트
@Aram-su Aram-su merged commit aa96d22 into dev Sep 9, 2024
1 check passed
@Aram-su Aram-su deleted the test/album branch September 9, 2024 06:32
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.

2 participants