🌐 US-Proxy
class="logged-out env-production page-responsive" style="word-wrap: break-word;" >
Skip to content

Disable tests failing on Windows - #3185

Closed
Bananeweizen wants to merge 1 commit into
assertj:mainfrom
Bananeweizen:disable_failing_tests_windows
Closed

Disable tests failing on Windows#3185
Bananeweizen wants to merge 1 commit into
assertj:mainfrom
Bananeweizen:disable_failing_tests_windows

Conversation

@Bananeweizen

Copy link
Copy Markdown
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

scordio commented Mar 1, 2025

Copy link
Copy Markdown
Member

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

scordio commented Mar 2, 2025

Copy link
Copy Markdown
Member

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

scordio commented Mar 5, 2025

Copy link
Copy Markdown
Member

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