fix: no need warning when provide optionLabelProp (#717)

Author: zombieJ

src/Select.tsx

@@ -253,7 +253,7 @@ const Select = React.forwardRef(
             rawDisabled = option?.disabled;
 
             // Warning if label not same as provided
-            if (process.env.NODE_ENV !== 'production' && !isRawValue(val)) {
+            if (process.env.NODE_ENV !== 'production' && !optionLabelProp) {
               const optionLabel = option?.[mergedFieldNames.label];
               if (optionLabel !== undefined && optionLabel !== rawLabel) {
                 warning(false, '`label` of `value` is not same as `label` in Select options.');

tests/Multiple.test.tsx

@@ -475,31 +475,59 @@ describe('Select.Multiple', () => {
     expect(wrapper.exists('.rc-select-selection-item-remove')).toBeFalsy();
   });
 
-  it('optionLabelProp', () => {
-    const wrapper = mount(
-      <Select
-        mode="multiple"
-        value={['bamboo', 'little']}
-        open
-        optionLabelProp="selector"
-        options={[
-          {
-            label: 'Bamboo',
-            value: 'bamboo',
-            selector: 'BAMBOO',
-          },
-          {
-            label: 'Little',
-            value: 'little',
-            selector: 'LITTLE',
-          },
-        ]}
-      />,
-    );
+  describe('optionLabelProp', () => {
+    it('basic', () => {
+      const wrapper = mount(
+        <Select
+          mode="multiple"
+          value={['bamboo', 'little']}
+          open
+          optionLabelProp="selector"
+          options={[
+            {
+              label: 'Bamboo',
+              value: 'bamboo',
+              selector: 'BAMBOO',
+            },
+            {
+              label: 'Little',
+              value: 'little',
+              selector: 'LITTLE',
+            },
+          ]}
+        />,
+      );
 
-    expect(findSelection(wrapper, 0).text()).toBe('BAMBOO');
-    expect(findSelection(wrapper, 1).text()).toBe('LITTLE');
-    expect(wrapper.find('div.rc-select-item-option-content').at(0).text()).toBe('Bamboo');
-    expect(wrapper.find('div.rc-select-item-option-content').at(1).text()).toBe('Little');
+      expect(findSelection(wrapper, 0).text()).toBe('BAMBOO');
+      expect(findSelection(wrapper, 1).text()).toBe('LITTLE');
+      expect(wrapper.find('div.rc-select-item-option-content').at(0).text()).toBe('Bamboo');
+      expect(wrapper.find('div.rc-select-item-option-content').at(1).text()).toBe('Little');
+    });
+
+    it('select no warning', () => {
+      const wrapper = mount(
+        <Select
+          mode="multiple"
+          open
+          optionLabelProp="selector"
+          options={[
+            {
+              label: 'Bamboo',
+              value: 'bamboo',
+              selector: 'BAMBOO',
+            },
+          ]}
+        />,
+      );
+
+      // Select one
+      const errSpy = jest.spyOn(console, 'error');
+
+      toggleOpen(wrapper);
+      selectItem(wrapper);
+
+      expect(errSpy).not.toHaveBeenCalled();
+      errSpy.mockRestore();
+    });
   });
 });