Skip to content

Feature/concurrent hash set test#16022

Closed
LI123456mo wants to merge 5 commits intoapache:3.3from
LI123456mo:feature/concurrent-hash-set-test
Closed

Feature/concurrent hash set test#16022
LI123456mo wants to merge 5 commits intoapache:3.3from
LI123456mo:feature/concurrent-hash-set-test

Conversation

@LI123456mo
Copy link
Contributor

Created ConcurrentHashSetTest.java to provide unit test coverage for the ConcurrentHashSet utility.

Verified the Set property of ensuring unique elements (duplicate adds return false).

Tested the internal delegation to ConcurrentHashMap for size(), isEmpty(), and contains().

Confirmed that the iterator() correctly scrolls through the set keys.

Validated remove() and clear() functionality to ensure the set is properly cleaned.

Applied spotless formatting to match project standards.

@codecov-commenter
Copy link

codecov-commenter commented Jan 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.78%. Comparing base (a92d5d0) to head (31b3d0d).

Additional details and impacted files
@@             Coverage Diff              @@
##                3.3   #16022      +/-   ##
============================================
- Coverage     60.75%   58.78%   -1.97%     
+ Complexity    11736       15   -11721     
============================================
  Files          1949     1949              
  Lines         88801    88801              
  Branches      13391    13391              
============================================
- Hits          53952    52206    -1746     
- Misses        29314    30935    +1621     
- Partials       5535     5660     +125     
Flag Coverage Δ
integration-tests-java21 ?
integration-tests-java8 ?
samples-tests-java21 ?
samples-tests-java8 ?
unit-tests-java11 59.02% <ø> (-0.02%) ⬇️
unit-tests-java17 58.52% <ø> (+0.01%) ⬆️
unit-tests-java21 58.52% <ø> (+<0.01%) ⬆️
unit-tests-java25 58.46% <ø> (-0.02%) ⬇️
unit-tests-java8 59.03% <ø> (+<0.01%) ⬆️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@LI123456mo
Copy link
Contributor Author

@ZLWR can this be a duplicate pr,
can you help here see the issue

@zrlw
Copy link
Contributor

zrlw commented Jan 20, 2026

Each PR should not include other PR codes, it should base on a fully new branch checked out from latest 3.3 branch.
CharSequenceComparatorTest was included in #16013 and it should not be included at this pr.

@zrlw zrlw closed this Jan 20, 2026
@LI123456mo
Copy link
Contributor Author

Each PR should not include other PR codes, it should base on a fully new branch checked out from latest 3.3 branch. CharSequenceComparatorTest was included in #16013 and it should not be included at this pr.

I got it , thank you
Can I do this newly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants