Merge pull request #1063 from rvsia/commonSelectTests

Add tests for common select

Author: Hyperkid123

packages/common/src/select/reducer.js

@@ -7,11 +7,6 @@ const reducer = (state, { type, payload, options = [] }) => {
         isLoading: false,
         promises: {}
       };
-    case 'loaded':
-      return {
-        ...state,
-        isLoading: false
-      };
     case 'startLoading':
       return {
         ...state,

packages/common/src/select/select.js

@@ -128,7 +128,8 @@ const Select = ({
         })
         .catch((error) => {
           dispatch({ type: 'setPromises', payload: { [inputValue]: false } });
-          throw error;
+          // eslint-disable-next-line no-console
+          console.error(error);
         });
     }
   };

packages/common/src/test/select/parse-internal-value.test.js

@@ -0,0 +1,36 @@
+import { parseInternalValue } from '../../select';
+
+describe('parseInternalValue', () => {
+  let value;
+  let isMulti;
+
+  it('single', () => {
+    value = 'cats';
+    isMulti = undefined;
+    expect(parseInternalValue(value, isMulti)).toEqual('cats');
+  });
+
+  it('multi and array', () => {
+    value = ['cats', { value: 'dogs' }];
+    isMulti = true;
+    expect(parseInternalValue(value, isMulti)).toEqual(['cats', 'dogs']);
+  });
+
+  it('single and array', () => {
+    value = ['cats', { value: 'dogs' }];
+    isMulti = false;
+    expect(parseInternalValue(value, isMulti)).toEqual('cats');
+  });
+
+  it('single and array - object', () => {
+    value = [{ value: 'dogs' }];
+    isMulti = false;
+    expect(parseInternalValue(value, isMulti)).toEqual('dogs');
+  });
+
+  it('single and undefined', () => {
+    value = [];
+    isMulti = false;
+    expect(parseInternalValue(value, isMulti)).toEqual(undefined);
+  });
+});