Skip to content

test(transpile): add failing test for cooperative threading intrinsics - #2088

Open
TartanLlama wants to merge 5 commits into
bytecodealliance:mainfrom
TartanLlama:sy/coop-thread-test
Open

test(transpile): add failing test for cooperative threading intrinsics#2088
TartanLlama wants to merge 5 commits into
bytecodealliance:mainfrom
TartanLlama:sy/coop-thread-test

Conversation

@TartanLlama

Copy link
Copy Markdown
Contributor

I've ported over one of the POSIX Open Test suite tests that we use in testing wasi-libc's threading implementation.

My idea is that we can use this as basic validation for the jco implementation, then port over more tests from there to build more confidence in it and find edge cases.

vados-cosmonic
vados-cosmonic previously approved these changes Sep 9, 2026

@vados-cosmonic vados-cosmonic left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

Thanks for updating the SDK version!

@vados-cosmonic

Copy link
Copy Markdown
Collaborator

BTW commitlint error:

input: tests(transpile): skip test
✖   type must be one of [build, chore, ci, debug, docs, feat, fix, perf, refactor, release, revert, style, test, sec] [type-enum]

✖   found 1 problems, 0 warnings

looks like tests -> test on that list commit title (again sorry this is so strict!)

@TartanLlama TartanLlama changed the title test(threading): add failing test for cooperative threading intrinsics test(transpile): add failing test for cooperative threading intrinsics Sep 9, 2026
@vados-cosmonic

Copy link
Copy Markdown
Collaborator

OK, so it looks like there might actually be an unexpected bug that has nothing to do with this PR but triggers it because of the sdk update? Will look into it

@vados-cosmonic

Copy link
Copy Markdown
Collaborator

OK, so it looks like there was a problem calling destructors in components that have this sort of... outside-of-task execution via core functions, fix is #2089 -- once it merges I think you should be able to rebase to main and this PR should be 🟢

Signed-off-by: Sy Brand <sy.brand@fastly.com>
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