Skip to content

Un-tentative textStream() tests (for Blob and Body) - #60356

Merged
noamr merged 2 commits into
masterfrom
text-stream
Jun 4, 2026
Merged

Un-tentative textStream() tests (for Blob and Body)#60356
noamr merged 2 commits into
masterfrom
text-stream

Conversation

@noamr

@noamr noamr commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

@noamr noamr changed the title Un-tentative textStream() tests Un-tentative textStream() tests (for Blob and Body) Jun 3, 2026

@annevk annevk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good, but for the null case we should:

  1. Assert bodyUsed as well. Always false, right?
  2. Assert that if you invoke the method multiple times, the objects don't share identity. And are all empty.

@noamr
noamr enabled auto-merge (squash) June 4, 2026 15:32
@noamr
noamr merged commit 15694f7 into master Jun 4, 2026
24 checks passed
@noamr
noamr deleted the text-stream branch June 4, 2026 16:49
annevk pushed a commit to whatwg/fetch that referenced this pull request Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants