Skip to content

Refactor: Redis Conn Test에 try-with-resources 적용#20

Merged
Dockerel merged 1 commit intodev-DDIP:mainfrom
Dockerel:Refactor/issue-#19
Aug 3, 2025
Merged

Refactor: Redis Conn Test에 try-with-resources 적용#20
Dockerel merged 1 commit intodev-DDIP:mainfrom
Dockerel:Refactor/issue-#19

Conversation

@Dockerel
Copy link
Contributor

@Dockerel Dockerel commented Aug 3, 2025

#️⃣ 연관된 이슈

📚 배경

MySQL Conn Test에는 try-with-resources가 적용되어 있는데 Redis Conn Test에는 적용되어 있지 않아 해당 부분을 수정하였습니다.

📝 작업 내용

1. Redis Conn Test에 try-with-resources 저장

  • finally에서 예외 발생 시에도 안전하게 자원 반환
  • 에러 스택 트레이스가 누락되지 않음
  • 가독성 향상

2. connection.ping()이 이미 String을 반환하여 String으로 감싼 부분 제거

📸 스크린샷

x

💬 리뷰 요구사항

x

✏ Git Close

close #19

@Dockerel Dockerel self-assigned this Aug 3, 2025
@Dockerel Dockerel added ♻️ Refactoring 코드 리팩토링 & 클린 코드 작업을 진행하는 경우 ✅ Test Code 테스트 관련 작업을 진행하는 경우 labels Aug 3, 2025
@Dockerel Dockerel requested a review from GitJIHO August 3, 2025 12:14
Copy link
Contributor

@GitJIHO GitJIHO left a comment

Choose a reason for hiding this comment

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

감사합니다 👍 꼼꼼한 확인 덕분에 좀 더 완성도 높은 코드가 된 것 같네요!

@Dockerel Dockerel merged commit f1e0d13 into dev-DDIP:main Aug 3, 2025
1 check passed
@Dockerel Dockerel deleted the Refactor/issue-#19 branch August 3, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ Refactoring 코드 리팩토링 & 클린 코드 작업을 진행하는 경우 ✅ Test Code 테스트 관련 작업을 진행하는 경우

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor: try-with-resources 적용

2 participants