Skip to content

Add XCTAsyncTest and XCTAssertThrowsError from gRPC Swift #493

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

Merged
merged 7 commits into from
Nov 26, 2021

Conversation

dnadoba
Copy link
Collaborator

@dnadoba dnadoba commented Nov 25, 2021

Motivation

We want to add async/await tests soon but the Linux implementation of XCTest doesn't have async-await support in Swift 5.5. In addition, our generate_linux_tests.rb script does not like async/await functions either. We therefore need a workaround to still be to test async/await code.

Changes

@fabianfett fabianfett requested review from glbrntt and Lukasa November 25, 2021 15:35
@fabianfett fabianfett added the 🔨 semver/patch No public API change. label Nov 25, 2021
@dnadoba
Copy link
Collaborator Author

dnadoba commented Nov 25, 2021

@swift-server-bot test this please
I think we just found a flaky test HTTPClientTests.testConnectionPoolSizeConfigValueIsRespected
https://ci.swiftserver.group/job/async-http-client-swift52-prb/1008/console

@dnadoba dnadoba merged commit 1119893 into swift-server:main Nov 26, 2021
@dnadoba dnadoba deleted the dn-xctasynctest branch November 26, 2021 09:21
@fabianfett fabianfett added semver/none No version bump required. and removed 🔨 semver/patch No public API change. labels Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants