Skip to content
This repository was archived by the owner on Dec 10, 2024. It is now read-only.

Add Linux test runner #80

Closed
wants to merge 1 commit into from
Closed

Conversation

jey
Copy link

@jey jey commented May 7, 2017

Made swift test work under Linux

@dduan
Copy link
Owner

dduan commented May 7, 2017

Hi @jey, thank you very much for the contribution! I tried this once and found that a lot of tests actually fail on Linux right now. Can you provide some stats on this before I merge your patch? How many of these tests are currently failing? Or, if you have left out some failing tests, which ones?

@dduan dduan closed this May 7, 2017
@dduan dduan reopened this May 7, 2017
@dduan
Copy link
Owner

dduan commented May 7, 2017

(also, I apologize for accidentally closing this PR, it wasn't intentional 😅)

@jey
Copy link
Author

jey commented May 8, 2017

Hi @dduan. I've disabled the mimetype related tests on Linux because they cause a crash, and aside from that there's 4 tests that fail. I believe two are related to HTTP basic authentication and two are related to handling of redirects.

It looks like the NSURLSession implementation in Foundation is actively being worked on (see swiftlang/swift-corelibs-foundation#970, swiftlang/swift-corelibs-foundation#969, swiftlang/swift-corelibs-foundation#968, swiftlang/swift-corelibs-foundation#979. I'm hoping some of these bugs will get ironed out in the process, but I'll take another look at it after those larger changes have landed.

@gonzo-oin
Copy link

Does the library works on Linux ?

@JustHTTP
Copy link
Collaborator

JustHTTP commented May 27, 2017 via email

@gonzo-oin
Copy link

Ok and thank you for the answer.
I'm looking for a http library like Just but that could work on Linux and didn't found one yet.

@JustHTTP
Copy link
Collaborator

Let's wait until Foundation is more mature. Enough work needs to happen at URLSession for linux for Just to "just work"

@JustHTTP JustHTTP closed this Sep 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants