Behavior of find and findAll

Should find and findAll return all nodes including the root node, or excluding the root node

Including root node:

const compiled = compileToFunctions('<div><div /></div>')
const wrapper = mount(compiled).findAll('div').length // 2

excluding root node:

const compiled = compileToFunctions('<div><div /></div>')
const wrapper = mount(compiled).findAll('div').length // 1

I think it should exclude the root node, but would be interested to hear others thoughts

[callumacrae]

+1 to including the root node: if the root node is returned and the user wasn’t expecting it to, it’s clear what happened and easy to work around, while it’s less obvious the other way round.

[eddyerburgh]

@Austio I don’t think we should add an extra find method. Two is enough.

I thought about it this weekend and I agree with @yyx990803 . We should include the root node, in every find and findAll call.