search

glenjamin / skin-deep

Test assertion helpers for use with React's shallowRender test utils
MIT License
200 stars 40 forks source link

Implement findNodes() #38

Closed aronwoost closed 8 years ago

aronwoost commented 8 years ago

findNode() is awesome! But sometimes you want to query multiple nodes.

This PR implements findNodes() which does exact the same as findNode() but returns an array.

glenjamin commented 8 years ago

Hi!

Thanks for the contribution, but we're actually removing findNode in favour of subTree - which has a sister method everySubTree that should do what you need.

See https://github.com/glenjamin/skin-deep/issues/9 for more discussion, https://github.com/glenjamin/skin-deep/issues/9#issuecomment-156819345 specifically for the current plan.