Skip to content
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

✅ Test : simplify tests #35

Merged
merged 4 commits into from
Apr 7, 2024
Merged

✅ Test : simplify tests #35

merged 4 commits into from
Apr 7, 2024

Conversation

ooMia
Copy link
Contributor

@ooMia ooMia commented Apr 6, 2024

개요

  • 테스트 및 main 동작 간소화

변경 사항

  • 🔥 Remove : BasicCounter 제거
  • ✅ Test : 테스트 간소화

추가 정보

  • 벤치마킹

ooMia added 4 commits April 7, 2024 08:47
Signed-off-by: Hyeon-hak Kim <[email protected]>
Signed-off-by: Hyeon-hak Kim <[email protected]>
Signed-off-by: Hyeon-hak Kim <[email protected]>
Signed-off-by: Hyeon-hak Kim <[email protected]>
@ooMia ooMia added the bug Something isn't working label Apr 6, 2024
@ooMia ooMia added this to the 스레드 동시성 제어 milestone Apr 6, 2024
@ooMia ooMia self-assigned this Apr 6, 2024
Copy link

codecov bot commented Apr 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.54%. Comparing base (6d1377d) to head (867a7b8).

Additional details and impacted files
@@              Coverage Diff               @@
##             develop      #35       +/-   ##
==============================================
+ Coverage      78.26%   94.54%   +16.28%     
- Complexity        37       43        +6     
==============================================
  Files              9        8        -1     
  Lines            115      110        -5     
  Branches           6        6               
==============================================
+ Hits              90      104       +14     
+ Misses            22        2       -20     
- Partials           3        4        +1     
Flag Coverage Δ
integration ?
unittests 94.54% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ooMia ooMia linked an issue Apr 7, 2024 that may be closed by this pull request
@ooMia ooMia merged commit 5822bed into develop Apr 7, 2024
3 checks passed
@ooMia ooMia deleted the bugfix/counter-test branch April 7, 2024 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

멀티 쓰레드 자원 업데이트 문제
1 participant