Skip to content

refactor: add parameterized tests for BitSwap #6347

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Jul 6, 2025

Conversation

alxkm
Copy link
Contributor

@alxkm alxkm commented Jul 6, 2025

Added parameterized tests covering:

  • Different bit cases
  • Same bit cases
  • Edge positions
  • Invalid positions
  • Original test cases
  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • All new code is formatted with clang-format -i --style=file path/to/your/file.java

@codecov-commenter
Copy link

codecov-commenter commented Jul 6, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 74.56%. Comparing base (9b4dec0) to head (f8ed188).

Files with missing lines Patch % Lines
...ava/com/thealgorithms/bitmanipulation/BitSwap.java 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6347      +/-   ##
============================================
- Coverage     74.57%   74.56%   -0.01%     
- Complexity     5450     5453       +3     
============================================
  Files           682      682              
  Lines         19034    19036       +2     
  Branches       3689     3690       +1     
============================================
  Hits          14194    14194              
- Misses         4277     4278       +1     
- Partials        563      564       +1     

☔ 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.

Copy link
Member

@siriak siriak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@siriak siriak merged commit 350f149 into TheAlgorithms:master Jul 6, 2025
6 of 7 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants