test: Add tests and CI for NIOAsyncRuntime#3488
Draft
scottmarchant wants to merge 3 commits intoapple:mainfrom
Draft
test: Add tests and CI for NIOAsyncRuntime#3488scottmarchant wants to merge 3 commits intoapple:mainfrom
scottmarchant wants to merge 3 commits intoapple:mainfrom
Conversation
90d91a7 to
ef885a9
Compare
acdb4a9 to
0db40da
Compare
68 tasks
46491d6 to
01f6fe9
Compare
…ncRuntime is a new module that provides alternative implementations of MTELG and NIOThreadPool using Swift Concurrency. It compiles to wasm and runs in the browser.
…me provides the same functionality as equivalent components in NIOPosix.
01f6fe9 to
620b816
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ports relevant tests from NIOPosix for the functionality in NIOAsyncRuntime and adjusts CI to build NIOAsyncRuntime, NIOPosix, and NIOEmbedded
IMPORTANT: Please wait to review this PR until the dependent PR's listed below are merged. This PR will be updated and rebased once #3487 is merged, and kept in draft until then.
Motivation:
NIOAsyncRuntime should have thorough tests and CI integration. It should pass all the same tests that are available in NIOPosix, within reason
Modifications:
Result:
NIOAsyncRuntime functionality achieves same or nearly the same test coverage as NIOPosix.
PR Dependencies
The following PR's must be merged first: