Skip to content
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

Run Wasm tests in workers as well #552

Merged
merged 3 commits into from
Dec 2, 2024

Conversation

daxpedda
Copy link
Contributor

@daxpedda daxpedda commented Nov 29, 2024

This change runs Wasm tests in workers as well.

Fixes #319.

@daxpedda daxpedda force-pushed the wasm-test-workers branch 2 times, most recently from 7ce3c73 to 94f3621 Compare November 29, 2024 19:11
@daxpedda daxpedda marked this pull request as draft November 29, 2024 19:16
@daxpedda
Copy link
Contributor Author

Apparently the Node.js was broken already. I unfortunately don't know enough about Node.js, locally it seems to work for me, so I'm going to do some tests with the CI.

@daxpedda daxpedda force-pushed the wasm-test-workers branch 3 times, most recently from fce0241 to f03425f Compare November 29, 2024 19:23
@@ -240,31 +240,48 @@ jobs:

web:
name: Web
runs-on: ubuntu-22.04
runs-on: ubuntu-24.04
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was required because the old Node.js version is incompatible with the latest Rust versions, which enabled reference types by default. See the blog post about it.

Alternatively I can also use the official GH action to install a newer version, let me know if you want me to do something here.

@daxpedda daxpedda marked this pull request as ready for review November 29, 2024 19:28
tests/mod.rs Outdated Show resolved Hide resolved
@newpavlov newpavlov merged commit c697b5c into rust-random:master Dec 2, 2024
56 checks passed
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.

Test that getrandom works correctly in a (service) worker in web environments.
2 participants