-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: Improve test data store logic #85
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빠른 작업 감사합니다!!
수고하셨습니당 :) (양이 엄청 많네요 ㅎㅎ;)
|
||
@SpringBootApplication | ||
@EnableCaching |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@EnableCaching 찾아보니 캐싱을 사용할 클래스나 메서드에 사용한다고 하는데 사용하는 이유가 혹시 있을까요?!
BmController 캐싱처리를 하신 의도가 궁금합니다!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
현재에는 필요 없지만 이왕 redis 도입한 김에 에러로그 캐싱해놓을려고 선언하였습니다. 지금은 필요없습니다 :)
📝 Test code-coverage reports
|
LAZY LOADING
---> testResult 필수! #83Implementation
rewrite
Ingress path