Move className to props

[eddyerburgh]

I'm making an enzyme-inferno-adapter, but I've hit some problems getting the last few tests to pass.

Enzyme requires the className to be inside props. This is fine normally, because we can convert the inferno VNode tree to an enzyme RST tree. But when you call the Enzmye contains function, enzyme compares the RST node to the newly created VNode. The RST node has className inside props, the inferno VNode has className on the vNode.

Example:

VNode:

{
type: 'div',
  props: {
    className: 'bar'
  }
}

type: 'div',
  props: {
  },
  className: 'bar'
}

This means we can't call a method like this:

const c = <div className="bar" />;
expect(mount(Component).contains(b)).to.equal(true);

Because enzyme is checking a VNode against an RST Node. This could be an issue to raise with Enzyme, but if there isn't a reason for Inferno to have className outside props, it would improve compatibility to add it to props.

Observed Behaviour

Inferno has className on the vNode

Expected Current Behaviour

Inferno should include className in the props

[Havunen]

className was moved from props to vNode because its the most common property of any html element. It has quite big impact on performance also, because when className is at the root and its the only prop. We don't need to loop props in diff.

[Havunen]

@eddyerburgh Can you do this in enzyme compat to wrap the vNodes? you can use inferno.options.createVNode to accomplish this

[eddyerburgh]

I've made a PR in enzyme to fix this —enzymejs/enzyme#1423