Java icon indicating copy to clipboard operation
Java copied to clipboard

test: refactor `PalindromeTest`

Open alxkm opened this issue 1 year ago • 1 comments

Refactoring PalindromeTest to a ParameterizedTest approach, as it provides more flexibility.

Also replace wrong or condition via and because each isPalindrome method should be correct.

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

alxkm avatar Aug 22 '24 19:08 alxkm

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 46.21%. Comparing base (a6fcbf5) to head (cbe98e6).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5365      +/-   ##
============================================
- Coverage     46.24%   46.21%   -0.04%     
+ Complexity     2865     2863       -2     
============================================
  Files           525      525              
  Lines         15412    15412              
  Branches       2916     2916              
============================================
- Hits           7128     7122       -6     
- Misses         7988     7991       +3     
- Partials        296      299       +3     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Aug 22 '24 19:08 codecov-commenter