Skip to content

Conversation

@Dockerel
Copy link
Contributor

#️⃣ 연관된 이슈

📚 배경

Redisson 테스트 시 프로덕션 코드의 Redisson 빈도 같이 생성되어 충돌이 발생한다고 이슈에 적어놨었는데, 그건 아니였고

테스트 프로퍼티가 설정되고(REDIS_HOST, REDIS_PORT, REDIS_PASSWORD), 테스트용 레디스가 실행되고 그 설정 정보가 저장되는데 이때 이게 spring.data.redis.host, spring.data.redis.port, spring.data.redis.password에 저장되어서 프로덕션단의 RedissonClient의 연결 오류가 났던 상황이었습니다.

결론적으로는 프로덕션단의 레디스 관련 코드들이 application-dev.properties들의 변수들을 참조하는데, 테스트 변수들이 application.properties의 값들을 변경해서 생긴 문제였습니다.

그런데 Redisson만 오류가 났던 이유는 LettuceConnectionFactory는 Lazy 연결로 실제로 필요할 때만 연결이 성립되서 오류가 나지 않고 나중에 테스트에서 사용될 때는 테스트용 LettuceConnectionFactory가 먼저 주입되서 문제가 없던 것이었습니다.

📝 작업 내용

  • RedissonClient 생성 시 application.properties의 환경변수들을 참조하도록 변경하였습니다
  • RedisConfig도 수정하였습니다

📸 스크린샷

x

💬 리뷰 요구사항

x

✏ Git Close

close #25

@Dockerel Dockerel requested a review from GitJIHO August 22, 2025 04:35
@Dockerel Dockerel self-assigned this Aug 22, 2025
@Dockerel Dockerel added the 😡 Bug Fix 발생한 버그를 수정하는 경우 label Aug 22, 2025
@github-actions
Copy link

📊 Code Coverage Report

Overall Project 92.67% 🍏
File Coverage
RedissonConfig.java 100% 🍏
RedisConfig.java 100% 🍏

@github-actions
Copy link

Test Results

182 tests   182 ✅  6s ⏱️
 29 suites    0 💤
 29 files      0 ❌

Results for commit 9022270.

@Dockerel Dockerel merged commit d12f206 into main Aug 23, 2025
2 checks passed
@Dockerel Dockerel deleted the Fix/issue-#25 branch August 23, 2025 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

😡 Bug Fix 발생한 버그를 수정하는 경우

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix: Redisson 테스트 config 수정

2 participants