Skip to content

iterator includes tests#5031

Open
michaelficarra wants to merge 14 commits intomainfrom
iterator-includes
Open

iterator includes tests#5031
michaelficarra wants to merge 14 commits intomainfrom
iterator-includes

Conversation

@michaelficarra
Copy link
Copy Markdown
Member

@michaelficarra michaelficarra commented Apr 15, 2026

Adds tests for https://github.com/michaelficarra/proposal-iterator-includes.

Disclaimer: I used AI in the process of creating this PR (using the same process as in #5011, including a full review and minor tweaks).

Copy link
Copy Markdown
Member

@bakkot bakkot left a comment

Choose a reason for hiding this comment

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

some nits but looks reasonably complete

Comment thread test/built-ins/Iterator/prototype/includes/argument-effect-order.js Outdated
Comment thread test/built-ins/Iterator/prototype/includes/closes-on-match.js
Comment thread test/built-ins/Iterator/prototype/includes/exhaustion-does-not-call-return.js Outdated
Comment thread test/built-ins/Iterator/prototype/includes/get-next-method-only-once.js Outdated
Comment thread test/built-ins/Iterator/prototype/includes/next-method-throws.js
Comment thread test/built-ins/Iterator/prototype/includes/object-identity.js
@michaelficarra
Copy link
Copy Markdown
Member Author

@bakkot Feedback addressed.

lando-worker Bot pushed a commit to mozilla-firefox/firefox that referenced this pull request Apr 24, 2026
These are from tc39/test262#5031 which is
currently under review. Once the tests land upstream, these can be
removed. This will be handled as part of shipping the feature when
it reaches Stage 3.

Differential Revision: https://phabricator.services.mozilla.com/D295351
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.

2 participants