Skip to content

Start running some async tests in CI#1286

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:async-tests
Apr 29, 2025
Merged

Start running some async tests in CI#1286
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:async-tests

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

This commit starts to execute some async tests in CI via artifacts downloaded from the wasip3-prototyping repository. While the status is reported on PRs this won't actually result in failing to merge something if the job fails at this time since it's known that async is a bit of a shifting landscape. In the meantime it's hoped that at least having this status on PRs will be sufficient for seeing failures for now.

The actual set of initial tests here are relatively edge-case-y where I added them some time ago when testing Rust bindings. They're not comprehensive by any means but are hopefully at least a start. I hope to next start adding more tests in follow-up PRs.

This commit starts to execute some async tests in CI via artifacts
downloaded from the wasip3-prototyping repository. While the status is
reported on PRs this won't actually result in failing to merge something
if the job fails at this time since it's known that async is a bit of a
shifting landscape. In the meantime it's hoped that at least having this
status on PRs will be sufficient for seeing failures for now.

The actual set of initial tests here are relatively edge-case-y where I
added them some time ago when testing Rust bindings. They're not
comprehensive by any means but are hopefully at least a start. I hope to
next start adding more tests in follow-up PRs.
@alexcrichton alexcrichton requested a review from dicej April 29, 2025 15:17
@alexcrichton alexcrichton added this pull request to the merge queue Apr 29, 2025
Merged via the queue into bytecodealliance:main with commit 8656e20 Apr 29, 2025
19 checks passed
@alexcrichton alexcrichton deleted the async-tests branch April 29, 2025 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants