adjust order (#412)

Author: zombieJ

src/Select.tsx

@@ -442,7 +442,7 @@ class Select extends React.Component<Partial<ISelectProps>, ISelectState> {
     const props = this.props;
     const selectedValue = getValuePropValue(item);
     const lastValue = value[value.length - 1];
-    this.fireSelect(selectedValue);
+
     if (isMultipleOrTags(props)) {
       if (findIndexInValueBySingleValue(value, selectedValue) !== -1) {
         return;
@@ -463,6 +463,7 @@ class Select extends React.Component<Partial<ISelectProps>, ISelectState> {
     }
 
     this.fireChange(value);
+    this.fireSelect(selectedValue);
     const inputValue = isCombobox(props) ? getPropValue(item, props.optionLabelProp) : '';
 
     if (props.autoClearSearchValue) {

tests/Select.spec.tsx

@@ -894,8 +894,13 @@ describe('Select', () => {
   });
 
   it('backfill', () => {
-    const handleChange = jest.fn();
-    const handleSelect = jest.fn();
+    const triggerQueue: string[] = [];
+    const handleChange = jest.fn(() => {
+      triggerQueue.push('change');
+    });
+    const handleSelect = jest.fn(() => {
+      triggerQueue.push('select');
+    });
     const wrapper = mount<Select>(
       <Select
         backfill={true}
@@ -924,6 +929,8 @@ describe('Select', () => {
     expect(handleChange).toBeCalledWith('2', expect.anything());
     expect(handleSelect).toBeCalledWith('2', expect.anything());
     expect(wrapper.find('.rc-select-selection-selected-value').text()).toEqual('Two');
+
+    expect(triggerQueue).toEqual(['change', 'select']);
   });
 
   describe('number value', () => {