Merge pull request JedWatson#834 from RealHQ/master

Update required when Select gets new props

Author: JedWatson

src/Select.js

@@ -138,6 +138,14 @@ const Select = React.createClass({
 		}
 	},
 
+	componentWillReceiveProps(nextProps) {
+		if (this.props.value !== nextProps.value && nextProps.required) {
+			this.setState({
+				required: this.handleRequired(nextProps.value, nextProps.multi),
+			});
+		}
+	},
+
 	componentWillUpdate (nextProps, nextState) {
 		if (nextState.isOpen !== this.state.isOpen) {
 			const handler = nextState.isOpen ? nextProps.onOpen : nextProps.onClose;

test/Select-test.js

@@ -2816,6 +2816,18 @@ describe('Select', () => {
 				expect(instance.state.required, 'to be true');
 			});
 
+			it('input should not have required attribute after updating the component with a value', () => {
+				wrapper = createControlWithWrapper({
+					options: defaultOptions,
+					value: '',
+					required: true
+				});
+
+				expect(instance.state.required, 'to be true');
+				wrapper.setPropsForChild({ value: 'one' });
+				expect(instance.state.required, 'to be false');
+			});
+
 		});
 
 		describe('required with multi=true', () => {