Skip to content

misc(expect-utils): removed creation of a new collection#15874

Merged
cpojer merged 1 commit intojestjs:mainfrom
Connormiha:optimize-getting-object-subset
Jan 21, 2026
Merged

misc(expect-utils): removed creation of a new collection#15874
cpojer merged 1 commit intojestjs:mainfrom
Connormiha:optimize-getting-object-subset

Conversation

@Connormiha
Copy link
Copy Markdown
Contributor

Summary

I moved the check inside the loop to avoid creating a new collection that the filter method produces.

Test plan

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 23, 2025

Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 3ad6ea7
🔍 Latest deploy log https://app.netlify.com/projects/jestjs/deploys/68f9f3cf29e15b0008bda695
😎 Deploy Preview https://deploy-preview-15874--jestjs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@Connormiha Connormiha changed the title misc(expect-utils): removed creation of a new collection Removed creation of a new collection Oct 23, 2025
@Connormiha Connormiha changed the title Removed creation of a new collection misc(expect-utils): removed creation of a new collection Oct 23, 2025
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Oct 23, 2025

Open in StackBlitz

babel-jest

npm i https://pkg.pr.new/babel-jest@15874

babel-plugin-jest-hoist

npm i https://pkg.pr.new/babel-plugin-jest-hoist@15874

babel-preset-jest

npm i https://pkg.pr.new/babel-preset-jest@15874

create-jest

npm i https://pkg.pr.new/create-jest@15874

@jest/diff-sequences

npm i https://pkg.pr.new/@jest/diff-sequences@15874

expect

npm i https://pkg.pr.new/expect@15874

@jest/expect-utils

npm i https://pkg.pr.new/@jest/expect-utils@15874

jest

npm i https://pkg.pr.new/jest@15874

jest-changed-files

npm i https://pkg.pr.new/jest-changed-files@15874

jest-circus

npm i https://pkg.pr.new/jest-circus@15874

jest-cli

npm i https://pkg.pr.new/jest-cli@15874

jest-config

npm i https://pkg.pr.new/jest-config@15874

@jest/console

npm i https://pkg.pr.new/@jest/console@15874

@jest/core

npm i https://pkg.pr.new/@jest/core@15874

@jest/create-cache-key-function

npm i https://pkg.pr.new/@jest/create-cache-key-function@15874

jest-diff

npm i https://pkg.pr.new/jest-diff@15874

jest-docblock

npm i https://pkg.pr.new/jest-docblock@15874

jest-each

npm i https://pkg.pr.new/jest-each@15874

@jest/environment

npm i https://pkg.pr.new/@jest/environment@15874

jest-environment-jsdom

npm i https://pkg.pr.new/jest-environment-jsdom@15874

@jest/environment-jsdom-abstract

npm i https://pkg.pr.new/@jest/environment-jsdom-abstract@15874

jest-environment-node

npm i https://pkg.pr.new/jest-environment-node@15874

@jest/expect

npm i https://pkg.pr.new/@jest/expect@15874

@jest/fake-timers

npm i https://pkg.pr.new/@jest/fake-timers@15874

@jest/get-type

npm i https://pkg.pr.new/@jest/get-type@15874

@jest/globals

npm i https://pkg.pr.new/@jest/globals@15874

jest-haste-map

npm i https://pkg.pr.new/jest-haste-map@15874

jest-jasmine2

npm i https://pkg.pr.new/jest-jasmine2@15874

jest-leak-detector

npm i https://pkg.pr.new/jest-leak-detector@15874

jest-matcher-utils

npm i https://pkg.pr.new/jest-matcher-utils@15874

jest-message-util

npm i https://pkg.pr.new/jest-message-util@15874

jest-mock

npm i https://pkg.pr.new/jest-mock@15874

@jest/pattern

npm i https://pkg.pr.new/@jest/pattern@15874

jest-phabricator

npm i https://pkg.pr.new/jest-phabricator@15874

jest-regex-util

npm i https://pkg.pr.new/jest-regex-util@15874

@jest/reporters

npm i https://pkg.pr.new/@jest/reporters@15874

jest-resolve

npm i https://pkg.pr.new/jest-resolve@15874

jest-resolve-dependencies

npm i https://pkg.pr.new/jest-resolve-dependencies@15874

jest-runner

npm i https://pkg.pr.new/jest-runner@15874

jest-runtime

npm i https://pkg.pr.new/jest-runtime@15874

@jest/schemas

npm i https://pkg.pr.new/@jest/schemas@15874

jest-snapshot

npm i https://pkg.pr.new/jest-snapshot@15874

@jest/snapshot-utils

npm i https://pkg.pr.new/@jest/snapshot-utils@15874

@jest/source-map

npm i https://pkg.pr.new/@jest/source-map@15874

@jest/test-result

npm i https://pkg.pr.new/@jest/test-result@15874

@jest/test-sequencer

npm i https://pkg.pr.new/@jest/test-sequencer@15874

@jest/transform

npm i https://pkg.pr.new/@jest/transform@15874

@jest/types

npm i https://pkg.pr.new/@jest/types@15874

jest-util

npm i https://pkg.pr.new/jest-util@15874

jest-validate

npm i https://pkg.pr.new/jest-validate@15874

jest-watcher

npm i https://pkg.pr.new/jest-watcher@15874

jest-worker

npm i https://pkg.pr.new/jest-worker@15874

pretty-format

npm i https://pkg.pr.new/pretty-format@15874

commit: 3ad6ea7

@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions Bot added the Stale label Jan 21, 2026
@cpojer cpojer merged commit 807f497 into jestjs:main Jan 21, 2026
73 of 74 checks passed
@github-actions
Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants