-
Notifications
You must be signed in to change notification settings - Fork 1
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
Integrated develop #32
Conversation
✅ Test : 테스트 코드 추가 1. SynchronizedCounter 와 CompletableFutureCounter를 생성 2. 두 카운터의 테스트를 작성 3. 테스트는 Integer.MAX_VALUE만큼 실행시키는 것이였으나 OutOfMemory 문제가 발생해서 500만번 수행
1. CompletableFuture 자료형은 스레드에 안전하지 않다. 2. 따라서 counter 값을 읽고 업데이트하는 과정을 하나의 단일 연산으로 만들어줘야한다. 3. CompleableFutureCounter에 대한 테스트도 작성 완료
✅ Test : 테스트 코드 추가 1. CompletableFutureCounter의 테스트를 다른 카운터와 동일하게 작성
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #32 +/- ##
==============================================
+ Coverage 62.50% 82.75% +20.25%
- Complexity 4 14 +10
==============================================
Files 2 5 +3
Lines 8 29 +21
==============================================
+ Hits 5 24 +19
- Misses 3 5 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Describe the bug #32 에 AtomicCounterTest에서 아래 그림처럼 카운트 횟수를 지정하는
To Reproduce
Expected behavior |
문제는 별도 이슈로 분리했어요 |
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.
우선 승인하겠습니다.
개요
변경 사항
✨ Feat : 새로운 기
✅ Test : 테스트 코드 추가
추가 정보
관련 이슈