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
WIP: THRIFT-4547 Add Swift cross tests #2048
WIP: THRIFT-4547 Add Swift cross tests #2048
Conversation
Linux version 4.2.0 does not have FileHandle.acceptConnectionInBackgroundAndNotify() implemented. We need at least 5.1 for TSocketServer to run in docker. In the meantime, URLRequest was moved to FoundationNetworking. To keep compatibility with previous versions, conditional import is used.
Requires these PRs: - apache#2036 - apache#2038 - apache#2047 Not finished, only few tests in client and server are implemented.
4a1ffc5
to
a659108
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
/// - secondsToSleep: | ||
/// - Throws: | ||
func testOneway(secondsToSleep: Int32) throws { | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
implementation missing
/// - arg: | ||
/// - Throws: Xception | ||
func testException(arg: String) throws { | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
implementattion missing
/// - Returns: Xtruct | ||
/// - Throws: Xception, Xception2 | ||
func testMultiException(arg0: String, arg1: String) throws -> Xtruct { | ||
return Xtruct(string_thing: arg1, byte_thing: 0, i32_thing: 0, i64_thing: 0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
impl not according spec
Please cherty-pick only the first commit, the rest is not finished yet. See #2050 |
I noticed. That's why I deleted my comment :-) |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
This issue has been automatically closed due to inactivity. Thank you for your contributions. |
This is Work-in-progress implementation of Swift cross language test client and server.
Only few tests are implemented so far both in client and in server. The goal now was to write at least some test so that I can see if it at least partially works.
Requires #2038, #2047 to work.
Server does not work yet (in linux command line app; they do work in iOS app), notifications are not delivered from NotificationCenter and I don't know how to setup it properly.
This obviously introduces a lot of failures to the cross tests.
Any comments/improvemets are welcome!
[skip ci]
at the end of your pull request to free up build resources.