test: 외부 의존성으로 인한 테스트 제거

[JayongLee]

🗞️ 연관된 이슈

🔥 이슈번호

• Resolved : hotfix

✅ 작업 내용

• 외부 의존성으로 인한 테스트로 인한 실패 해결을 위한 테스트 코드 제거

📸 스크린샷 (선택)

체크리스트 ✅

• 코드가 정상적으로 컴파일되나요?
• merge할 브랜치의 위치를 확인했나요?
• 테스트 코드를 작성하셨나요?

기타

[gemini-code-assist]

Summary of Changes

Hello @JayongLee, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 외부 의존성으로 인해 불안정했던 동시성 관련 통합 테스트를 제거하여 테스트 실패 문제를 해결합니다. 동시에, 계좌 잠금 메커니즘을 위한 새로운 설정 항목을 application.yml에 추가하여 시스템의 안정적인 동작을 위한 기반을 마련했습니다.

Highlights

• 테스트 코드 제거: 외부 의존성으로 인해 실패하던 AccountConcurrencyIntegrationTest.java 파일이 제거되었습니다. 이 테스트는 동시 출금 시 계좌 잔액의 정합성을 검증하는 통합 테스트였습니다.
• 계좌 잠금 설정 추가: application.yml 파일에 core.account-lock 관련 설정(접두사, 대기 시간, 임대 시간)이 추가되었습니다. 이는 분산 잠금 메커니즘의 구성을 위한 것으로 보입니다.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

이 PR은 외부 의존성으로 인한 테스트 실패를 해결하기 위해 동시성 통합 테스트(AccountConcurrencyIntegrationTest.java)를 제거합니다. 동시성 테스트는 시스템의 안정성과 데이터 정합성을 보장하는 데 매우 중요하므로, 테스트를 제거하는 대신 수정하는 것을 강력히 권장합니다. 예를 들어, Testcontainers를 사용하여 실제 Redis 환경을 테스트 중에 구동하면, 외부 의존성 문제를 해결하면서도 더 신뢰할 수 있는 테스트를 유지할 수 있습니다. 테스트를 제거하면 동시성 관련 버그가 발생할 위험이 커집니다.

I am having trouble creating individual review comments. Click here to see my feedback.

src/test/java/org/creditto/core_banking/domain/account/AccountConcurrencyIntegrationTest.java (1-222)

동시성 테스트를 제거하는 것은 위험할 수 있습니다. 이 테스트는 여러 스레드가 동시에 계좌에 접근할 때 발생할 수 있는 경쟁 조건(race condition)과 같은 심각한 버그를 방지하는 중요한 역할을 합니다.

테스트가 외부 의존성 때문에 실패한다면, 이를 제거하기보다는 Testcontainers와 같은 도구를 사용하여 격리된 테스트 환경에서 실제 Redis 인스턴스를 실행하는 것을 고려해 보세요. 이렇게 하면 RedissonClient를 모의(mock)할 필요 없이 실제와 유사한 환경에서 동시성 로직을 검증할 수 있어 테스트의 신뢰도를 높일 수 있습니다.

동시성 제어 로직의 정확성을 보장하기 위해 이 테스트를 복구하거나 대체 테스트를 작성하는 것이 좋습니다.