Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows x64 MSVC Clang gnustep-2.0 -- Tests for NSURL* keep failing... #488

Open
gcasa opened this issue Jan 25, 2025 · 7 comments
Open
Assignees

Comments

@gcasa
Copy link
Member

gcasa commented Jan 25, 2025

Can we please correct the tests for this environment so that CI is not always RED? Otherwise it looks like it is ALWAYS failing.

@triplef
Copy link
Member

triplef commented Jan 27, 2025

See #474 (comment) for details.

@triplef
Copy link
Member

triplef commented Jan 27, 2025

@hmelder is it realistic to fix this in the short term, or should we just merge #478 to mark them as hopeful for now?

@hmelder
Copy link
Contributor

hmelder commented Jan 27, 2025

I'll take a look at it. If you want the tests to be green rn, feel free to mark them as hopeful.

@gcasa
Copy link
Member Author

gcasa commented Jan 28, 2025

  175 |                                                       [server port]]];
      |                                                       ^~~~~~~~~~~~~
15 warnings generated.
 Linking test_tool uploadTaskTests ...

base/NSURLSession/simpleTaskTests.m:
Failed file:  simpleTaskTests.m aborted without running all tests!

Is this test even compiling?

@hmelder
Copy link
Contributor

hmelder commented Feb 4, 2025

Should be fixed now. Seems like something changed in the newer MSVC header files, so I needed to include WS2tcpip.h for socklen_t.

@hmelder hmelder closed this as completed Feb 4, 2025
@triplef
Copy link
Member

triplef commented Feb 7, 2025

Unfortunately it looks like this test is still failing at least sometimes on Windows MSVC:

base/NSURLSession/simpleTaskTests.m:
Failed test:     (2025-02-06 19:52:45.095 +0000) simpleTaskTests.m:875 ... All transfers were completed before a timeout occurred

@triplef triplef reopened this Feb 7, 2025
@hmelder
Copy link
Contributor

hmelder commented Feb 7, 2025

Ah yes that seems to be a sporadic failure due to libdispatch swiftlang/swift-corelibs-libdispatch#833

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants