Skip to content

Conversation

@Bananeweizen
Copy link
Contributor

  • The conversion shell scripts cannot be executed on Windows.
  • Path related tests trying to create symbolic links fail with an exception on Windows, as documented in
    java.nio.file.Files.createSymbolicLink(Path, Path, FileAttribute<?>...)

Check List:

With this change I can build the complete Maven reactor. Without it, there are around 50 test failures, exactly on the disabled test classes and methods.

* The conversion shell scripts cannot be executed on Windows.
* Path related tests trying to create symbolic links fail with an
exception on Windows, as documented in
java.nio.file.Files.createSymbolicLink(Path, Path, FileAttribute<?>...)
@scordio scordio self-assigned this Sep 18, 2023
@scordio scordio added the status: pending investigation An issue that needs additional investigation label Sep 18, 2023
@scordio scordio force-pushed the 3.x branch 2 times, most recently from 301ca01 to c730d18 Compare June 1, 2024 16:04
@scordio scordio added this to the 4.0.0-M1 milestone Feb 28, 2025
@scordio scordio changed the base branch from 3.x to main February 28, 2025 23:21
@scordio
Copy link
Member

scordio commented Mar 1, 2025

Hi @Bananeweizen, apologies for getting back so late to this PR!

The conversion shell scripts cannot be executed on Windows.

You're totally right and I wonder why our build doesn't fail on the Windows runner – I'll dig a bit and get back to you.

Path related tests trying to create symbolic links fail with an exception on Windows, as documented in
java.nio.file.Files.createSymbolicLink(Path, Path, FileAttribute<?>...)

I couldn't find a specific reference to Windows at Files::createSymbolicLink. Nevertheless, I experienced the same in a Windows environment with an unprivileged user.

As these tests can be successfully executed on the GitHub runner (and I suppose in any other Windows environment with the appropriate permissions), it might be a good idea to introduce a custom JUnit condition to address this constellation. I will take care of that on top of your changes.

@scordio
Copy link
Member

scordio commented Mar 2, 2025

The conversion shell scripts cannot be executed on Windows.

I refactored how we do script testing in #3785: this is now a separate Maven module running on non-Windows platforms only, and using JUnit @TempDir instead of the manual handling we had before.

Although my PR has taken a different direction from this one, I truly appreciated that you looked into the problem, somehow hidden before, therefore I'm adding you as a coauthor 😉

@scordio scordio removed the status: pending investigation An issue that needs additional investigation label Mar 2, 2025
@scordio
Copy link
Member

scordio commented Mar 5, 2025

Superseded by #3785 and #3788.

@scordio scordio closed this Mar 5, 2025
@scordio scordio removed this from the 4.0.0-M1 milestone Mar 5, 2025
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.

Tests failing on Windows

2 participants