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

2 packages failing dart2wasm tests #54800

Closed
2 of 4 tasks
kevmoo opened this issue Feb 1, 2024 · 11 comments
Closed
2 of 4 tasks

2 packages failing dart2wasm tests #54800

kevmoo opened this issue Feb 1, 2024 · 11 comments
Labels
area-dart2wasm Issues for the dart2wasm compiler.

Comments

@kevmoo
Copy link
Member

kevmoo commented Feb 1, 2024

Reproduction: clone, dart pub get, dart test -p chrome --compiler dart2wasm

test/subscription_stream_test.dart: cancelOnError source: no: - subscriptionStream: no [E]
  Expected: [1, 2, 'To err is divine!', 4]
    Actual: [1, 2, 'To err is divine!']
     Which: at location [3] is [1, 2, 'To err is divine!'] which shorter than expected
@kevmoo kevmoo added the area-dart2wasm Issues for the dart2wasm compiler. label Feb 1, 2024
@mkustermann
Copy link
Member

@kevmoo Could you provide instructions on how to run those tests?

@kevmoo
Copy link
Member Author

kevmoo commented Feb 1, 2024

done!

copybara-service bot pushed a commit that referenced this issue Feb 5, 2024
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]>
@mkustermann
Copy link
Member

Also landed cl/350322 for perf fix.

@kevmoo
Copy link
Member Author

kevmoo commented Feb 6, 2024

pkg:crypto tests all pass now! 🥳

copybara-service bot pushed a commit that referenced this issue Feb 7, 2024
…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]>
@kevmoo
Copy link
Member Author

kevmoo commented Feb 8, 2024

@mkustermann – still getting ONE failure (due to timeouts) on crypto: https://github.com/dart-lang/crypto/actions/runs/7801915494/job/21380589146#step:7:674

@mkustermann
Copy link
Member

@kevmoo I can have a look. For me all package:crypto tests passed locally, may be flaky or a perf issue. For package:async there is now one failure remaining, it's a async* semantics issue that's somewhat very rare, I'll have to see how much work it is to fix and whether it makes sense to prioritize it now or not.

@mkustermann
Copy link
Member

@mkustermann – still getting ONE failure (due to timeouts) on crypto: https://github.com/dart-lang/crypto/actions/runs/7801915494/job/21380589146#step:7:674

The error message from this link seems to be

[Chrome, Dart2Wasm] loading test/hmac_sha2_test.dart (failed)
  Failed to load "test/hmac_sha2_test.dart":
  Timed out waiting for browser to load test suite.

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 origin/main of Dart SDK

<sdk>/third_party/pkg/crypto % xvfb-run -a dart test -p chrome --compiler dart2wasm                           
Building package executable... 
Built test:test.
01:02 +822: All tests passed!                                                                  

<sdk>/third_party/pkg/crypto % xvfb-run -a dart test -p chrome --compiler dart2wasm test/hmac_sha2_test.dart  
Building package executable... (4.7s)
Built test:test.
00:02 +7: All tests passed!                  

@kevmoo
Copy link
Member Author

kevmoo commented Feb 8, 2024

using -j 1 also helps. (which is similar to running just one test directly)

It seems that the large fire produces correct hash test is randomly slow with wasm.

@kevmoo
Copy link
Member Author

kevmoo commented Feb 8, 2024

Crypto is working and committed!

@mkustermann mkustermann removed their assignment Feb 16, 2024
@osa1
Copy link
Member

osa1 commented Mar 6, 2024

With the current SDK main branch async tests also pass:

$ dart test -p chrome --compiler dart2wasm
Building package executable... (3.9s)
Built test:test.
00:06 +290: [Chrome, Dart2Wasm] test/stream_queue_test.dart: startTransaction operation produces a transaction that when rejected can reject a transaction where one copy is fully consumed in a transaction and a second copy is made
  Skip: https://github.com/dart-lang/async/issues/229
00:14 +606 ~1: All tests passed!

@osa1 osa1 closed this as completed Mar 6, 2024
@kevmoo
Copy link
Member Author

kevmoo commented Mar 6, 2024

well done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-dart2wasm Issues for the dart2wasm compiler.
Projects
None yet
Development

No branches or pull requests

3 participants