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.

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

Already on GitHub? Sign in to your account

Add TestUtils find/scry DOMComponent by prop #1651

Closed
wants to merge 1 commit into from

Conversation

chenglou
Copy link
Contributor

@chenglou chenglou commented Jun 6, 2014

Fixes #1616, which asked for finding the ID. This is more general.

Fixes facebook#1616, which asked for finding the ID. This is more general.
@sophiebits
Copy link
Collaborator

cf. #1473 (comment)

@sebmarkbage
Copy link
Collaborator

This no longer works for refs so it's not a replacement for #1473. Probably need a more general filter mechanism like a forEach or a callback. However, I'm inclined to not add more stuff to this test utility and just prefer shallow testing.

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.

Add find/scry by ID to test utils
4 participants