Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

toHaveProperty should not allow a missing property when looking for an undefined one #8786

Closed
jeysal opened this issue Aug 6, 2019 · 3 comments 路 Fixed by #8923
Closed

toHaveProperty should not allow a missing property when looking for an undefined one #8786

jeysal opened this issue Aug 6, 2019 · 3 comments 路 Fixed by #8923
Labels
Bug Report Confirmed
Milestone
Jest 28

Comments

@jeysal
Copy link
Contributor

jeysal commented Aug 6, 2019 •
edited

馃悰 Bug Report

Follow up to #8067, tracking for Jest 25.
cc @pedrottimark

To Reproduce

expect({}).toHaveProperty('a', undefined) // passes

Expected behavior

Throws, because {} does not have a property a
@jeysal jeysal added this to the Jest 25 milestone Aug 6, 2019
@SixTfour SixTfour mentioned this issue Sep 6, 2019
Merged
@sklaus64
Copy link

sklaus64 commented Sep 6, 2019

#8923

@cpojer cpojer modified the milestones: Jest 26, High priority future May 2, 2020
@SimenB
Copy link
Member

SimenB commented Feb 24, 2022

https://github.com/facebook/jest/releases/tag/v28.0.0-alpha.5

@SimenB SimenB modified the milestones: High priority future, Jest 28 Feb 24, 2022
@github-actions
Copy link

This issue 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 Mar 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Bug Report Confirmed
Projects
None yet
Milestone
Jest 28
Development

Successfully merging a pull request may close this issue.

Fixed toHaveProperty returning false positives when looking for undefined property SixTfour/jest
4 participants
@cpojer @SimenB @jeysal @sklaus64