Skip to content

fixed path separator handling in test for windows compatibility#116

Closed
csoltenborn wants to merge 1 commit intodflib:mainfrom
csoltenborn:test_fix_for_windows
Closed

fixed path separator handling in test for windows compatibility#116
csoltenborn wants to merge 1 commit intodflib:mainfrom
csoltenborn:test_fix_for_windows

Conversation

@csoltenborn
Copy link

This is basically a duplicate of #98, but only fixes the path issue... locally, I can run the tests without further adjustments, maybe because of 31f1e97 (Testcontainers 2.0).

@andrus
Copy link
Contributor

andrus commented Feb 28, 2026

Hi @csoltenborn . Thanks for bringing this to our attention. Kinda dropped the ball on the original PR. Anyways, I rebased #98 and created a new PR #117. It no longer has CI/CD issues so I was able to merge it to main. Now it should build on Windows. At least the unit tests (mvn test), integration tests (mvn verify) are still failing and are disabled in GitHub actions for Windows

@andrus andrus closed this Feb 28, 2026
@csoltenborn csoltenborn deleted the test_fix_for_windows branch February 28, 2026 14:17
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.

2 participants