Skip to content

fix(expect): use iterableEquality in value matchers - #11304

Closed
sahiljadhav7 wants to merge 2 commits into
vitest-dev:mainfrom
sahiljadhav7:fix/expect-iterable-equality
Closed

sahiljadhav7 wants to merge 2 commits into
vitest-dev:mainfrom
sahiljadhav7:fix/expect-iterable-equality

Conversation

@sahiljadhav7

Copy link
Copy Markdown

Closes #11296

toContainEqual, toHaveProperty, toBeOneOf and the six return/resolve matchers compared Maps and Sets without iterableEquality, so any two of them matched. The six return/resolve matchers also dropped customTesters, so expect.addEqualityTesters did not reach them.

@netlify

netlify Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for vitest-dev ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 6b248ae
🔍 Latest deploy log https://app.netlify.com/projects/vitest-dev/deploys/6abfb1ceb2199f00083fcadb
😎 Deploy Preview https://deploy-preview-11304--vitest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sahiljadhav7
sahiljadhav7 force-pushed the fix/expect-iterable-equality branch 2 times, most recently from fa5734b to d681710 Compare September 19, 2026 13:26
…-equality

# Conflicts:
#	packages/expect/src/custom-matchers.ts
@hi-ogawa

hi-ogawa commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator

closed by #11402

@hi-ogawa hi-ogawa closed this Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

toContainEqual, toHaveProperty, toBeOneOf and return matchers treat any two Maps or Sets as equal

2 participants