Anybody knows something better to use here?

Why do you have to pass the ref here? It will be passed automatically with {...props} with the correct type.

It's because TS does not recognized the ref correctly in here. I tried casting it to expected and it failed. This was the only solution that didn't thre any error.

I had to remove this test because jest is not playing nicely with mount and currentTarget enzymejs/enzyme#218.

I had to use mount because checkbox is now in Checkbox component.

I've run into this same issue (I think also with Checkbox). My solution was to change handleChange in Checkbox.tsx from:

private handleChange = (event: React.FormEvent<HTMLInputElement>): void => {
  this.props.onChange(event.currentTarget.checked, event);
}

to:

private handleChange = (event: React.FormEvent<HTMLInputElement>): void => {
  this.props.onChange(event.target.checked, event);
}

If Checkbox doesn't break, feel free to try that.

I think we should open a high-priority issue to add this test back if we aren't willing to address it in the scope of this PR.

Yeah, that came up in head as well, I didn't wanted to change the original Checkbox though because it's not part of this PR. But I'll probably change it to use target instead of currentTarget.

I could also change handleChange of this component to use target instead of currentTarget which is probably a bit safer.