assertj icon indicating copy to clipboard operation
assertj copied to clipboard

feat: dont throw FileNotFound for Path endsWith assertions, add test

Open ky0n opened this issue 6 months ago • 2 comments

Check List:

  • Fixes #3840
  • Unit tests : YES / NO / NA
  • Javadoc with a code example (on API only) : YES / NO / NA
  • PR meets the contributing guidelines

Following the contributing guidelines will make it easier for us to review and accept your PR.

this is a starting implementation for issue #3840 happy to receive any feedback. thank you! Implementation for startsWith still open


Co-authored-by: Benjamin König <[email protected]>

ky0n avatar Jun 13 '25 11:06 ky0n

@ky0n the two failed jobs are unrelated to your changes, you can ignore them.

scordio avatar Jun 13 '25 20:06 scordio

can you have a look please @scordio, thank you. I tried to be as close as possible to toRealPath-Method with toAbsolutePath().normalize(), while the later will still work without the file existing. I didn't change the toRealPath(Path) method.

ky0n avatar Jun 13 '25 21:06 ky0n

This is finally merged! Thanks for your efforts, @bekoenig and @ky0n!

scordio avatar Nov 08 '25 16:11 scordio