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

[fix] 테스트 환경의 Redis를 test-container 에서 embedded-redis로 변경 후 junit 테스트 실행 시 실행될 수 있도록 수정 #152

Conversation

Hyeon-Uk
Copy link
Contributor

@Hyeon-Uk Hyeon-Uk commented Aug 27, 2024

💡 다음 이슈를 해결했어요.

Issue Link - #151

  • 테스트 환경에 redis 컨테이너가 설정되어있지 않아 다른 서비스의 통합 테스트 시 redis를 연결할 수 없어서 테스트가 실패함
  • 따라서 모든 테스트가 실행되기 전 redis를 테스트환경에 실행시켜야합니다.(특정 테스트에서만 컨테이너를 띄우지 않고, 모든 환경에서 띄워놔야 다른 통합 테스트가 실패하지 않습니다.)
  • 또한, test-container로 redis 환경을 구축하면 매우 느리고, 포트 충돌의 위험이 있습니다.
  • 따라서 embedded redis로 환경을 구축하고, 모든 테스트가 실행되기 이전에 실행시키도록 코드를 수정했습니다.

💡 이슈를 처리하면서 추가된 코드가 있어요.

  • EmbeddedRedisExtension.java : 테스트 실행 전에 embedded-redis를 구동시킬 수 있는 extension입니다.

✅ 셀프 체크리스트

  • 내 코드를 스스로 검토했습니다.
  • 필요한 테스트를 추가했습니다.
  • 모든 테스트를 통과합니다.
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • wiki를 수정했습니다.

- test-container는 처음 띄울 때 매우 무거워서 시간이 오래걸리고, 테스트별 컨테이너의 포트 충돌의 위험이 있다.
- 따라서 test-container에서 embedded redis를 이용한 테스트로 변경하기위해 의존성 변경
- 더이상 테스트 컨테이너는 사용하지 않을것이므로 삭제
- JUnit 테스트를 실행할 때 extension을 detect할 수 있도록 기능 추가
@Hyeon-Uk Hyeon-Uk added 🔨 Refactor 코드 리팩토링 ✅ Test 테스트 관련 labels Aug 27, 2024
@Hyeon-Uk Hyeon-Uk added this to the 1차 고도화 milestone Aug 27, 2024
@Hyeon-Uk Hyeon-Uk self-assigned this Aug 27, 2024
현재 이슈에서는 redisson을 사용한다는 보장이 없기 때문에 수정
Copy link
Member

@june-777 june-777 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
Member

@kimhyun5u kimhyun5u left a comment

Choose a reason for hiding this comment

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

🏃‍♂️

@Hyeon-Uk Hyeon-Uk merged commit 29be908 into main Aug 27, 2024
1 check passed
@Hyeon-Uk Hyeon-Uk deleted the feature/151_Hyeon-Uk_test-container가-빌드되지-않는-문제 branch August 27, 2024 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 리팩토링 ✅ Test 테스트 관련
Projects
Status: Done
4 participants