[FIX] JedWatson#1651 moved option prop sync to componentWillReceivePr…

…ops (JedWatson#1765)

* [FIX] JedWatson#1651 moved option prop sync to componentWillReceiveProps

* [ADD] test for option prop sync by componentWillReceiveProps

src/Async.js

@@ -68,15 +68,12 @@ export default class Async extends Component {
 		}
 	}
 
-	componentWillUpdate (nextProps, nextState) {
-		const propertiesToSync = ['options'];
-		propertiesToSync.forEach((prop) => {
-			if (this.props[prop] !== nextProps[prop]) {
-				this.setState({
-					[prop]: nextProps[prop]
-				});
-			}
-		});
+	componentWillReceiveProps(nextProps) {
+		if (nextProps.options !== this.props.options) {
+			this.setState({
+				options: nextProps.options,
+			});
+		}
 	}
 
 	clearOptions() {

test/Async-test.js

@@ -447,4 +447,20 @@ describe('Async', () => {
 			expect(input, 'to equal', document.activeElement);
 		});
 	});
+
+
+	describe('props sync test', () => {
+		it('should update options on componentWillReceiveProps', () => {
+			createControl({
+			});
+			asyncInstance.componentWillReceiveProps({
+				options: [{
+					label: 'bar',
+					value: 'foo',
+				}]
+			});
+			expect(asyncNode.querySelectorAll('[role=option]').length, 'to equal', 1);
+			expect(asyncNode.querySelector('[role=option]').textContent, 'to equal', 'bar');
+		});
+	});
 });