-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
test: update wpt test for streams #54129
Conversation
Review requested:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #54129 +/- ##
=======================================
Coverage 87.07% 87.08%
=======================================
Files 643 643
Lines 181581 181576 -5
Branches 34893 34896 +3
=======================================
+ Hits 158118 158125 +7
+ Misses 16742 16730 -12
Partials 6721 6721 |
@daeyeon |
"readable-streams/from.any.js": { | ||
"fail": { | ||
"expected": [ | ||
"ReadableStream.from ignores a null @@asyncIterator" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to fix this in a follow-up PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MattiasBuelens I think we might need to address this issue. It seems like some basic error handling might be necessary. I'm also open to contributing to a follow-up PR to help resolve this.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Landed in 1bcdba2 |
PR-URL: #54129 Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Updating the web platform tests for the streams.