-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
2 packages failing dart2wasm tests #54800
Comments
@kevmoo Could you provide instructions on how to run those tests? |
done! |
The `int.parse()` function would - if radix is provided and not 10 - use functions that `unsafeCast<StringBase>()`. => The string can be a JSString. Issue #54800 CoreLibraryReviewExempt: Adding test that should pass on all backends. Change-Id: Ie6efa5bc545cadce5ea946e2c1d732b2cf94e306 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/350303 Reviewed-by: Slava Egorov <[email protected]> Commit-Queue: Martin Kustermann <[email protected]>
Also landed cl/350322 for perf fix. |
|
…rved Issue #54800 Change-Id: I26b8c35f60955ee46eeae19797aab7e6c84bf354 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/350580 Reviewed-by: Slava Egorov <[email protected]> Commit-Queue: Martin Kustermann <[email protected]>
@mkustermann – still getting ONE failure (due to timeouts) on crypto: https://github.com/dart-lang/crypto/actions/runs/7801915494/job/21380589146#step:7:674 |
@kevmoo I can have a look. For me all |
The error message from this link seems to be
It says it timed out waiting for the test suite to load, not even to run it. Could this be some flaky infra issue? For me all tests pass just fine (also when running the failing test from your link individually) using
|
using It seems that the |
Crypto is working and committed! |
With the current SDK main branch async tests also pass:
|
well done! |
Reproduction: clone,
dart pub get
,dart test -p chrome --compiler dart2wasm
The text was updated successfully, but these errors were encountered: