callemall / material-ui / 7714

66%

push

travis-ci

[TextField] Fix first character composition issue (#6456)

* [TextField]Fix first character composition issue #3394

Tiny change. It seems to me that, when the component is conrolled,
'value' needs to be checked only in componentWillReceiveProps,
and event.target.value in handleInputChange has nothing to do with
hasValue state.

At least, this PR can fix #3394 issue which, still remaining despite #5540,
breaks 'first' character composition of CJK or other input methods
when used with, for example, redux-form.

* Add tests of hasValue

Tests for uncontrolled component's hasValue state regarding event.target.value.
Tests for controlled component's hasValue state reagarding value prop.

1548 of 3218 branches covered (48.1%)

3127 of 5018 relevant lines covered (62.32%)

43.56 hits per line