Disable tests failing on Windows by Bananeweizen · Pull Request #3185 · assertj/assertj · GitHub
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

@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

@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