Added tests for pf3 select option

Author: Hyperkid123

packages/pf3-component-mapper/src/form-fields/select/option.js

@@ -3,23 +3,20 @@ import { components } from 'react-select';
 import PropTypes from 'prop-types';
 import clsx from 'clsx';
 
-const Option = props => {
-  return (
-    <div className={ clsx('ddorg__pf3-component-mapper__select__option', {
-      'ddorg__pf3-component-mapper__select__option--is-focused': props.isFocused,
-      'ddorg__pf3-component-mapper__select__option--is-selected': props.isSelected,
-    }) }>
-      <components.Option { ...props } />
-      { props.isSelected && (props.selectProps && !props.selectProps.isCheckbox) && <i className="selected-indicator fa fa-check"></i> }
-    </div>
-  );
-};
+const Option = props => (
+  <div className={ clsx('ddorg__pf3-component-mapper__select__option', {
+    'ddorg__pf3-component-mapper__select__option--is-focused': props.isFocused,
+    'ddorg__pf3-component-mapper__select__option--is-selected': props.isSelected,
+  }) }>
+    <components.Option { ...props } />
+    { props.isSelected && (props.selectProps && !props.selectProps.isCheckbox) && <i className="selected-indicator fa fa-check"></i> }
+  </div>
+);
 
 Option.propTypes = {
   isFocused: PropTypes.bool,
   isSelected: PropTypes.bool,
   getStyles: PropTypes.func.isRequired,
-  selectOption: PropTypes.func,
   cx: PropTypes.func.isRequired,
   data: PropTypes.shape({
     selected: PropTypes.bool,
@@ -35,7 +32,6 @@ Option.propTypes = {
 Option.defaultProps = {
   isFocused: false,
   isSelected: false,
-  selectOption: () => undefined,
   selectProps: {
     isCheckbox: false,
   },

packages/pf3-component-mapper/src/tests/form-fields/select/option.test.js

@@ -0,0 +1,18 @@
+import React from 'react';
+import { mount } from 'enzyme';
+import Option from '../../../form-fields/select/option';
+
+describe('<Option />', () => {
+  const optionProps = {
+    getStyles: jest.fn(),
+    cx: jest.fn(),
+    selectProps: {},
+  };
+  it('should render in selected variant', () => {
+    const wrapper = mount(<Option { ...optionProps }/>);
+    expect(wrapper.find('i.selected-indicator.fa.fa-check')).toHaveLength(0);
+    wrapper.setProps({ isSelected: true });
+    wrapper.update();
+    expect(wrapper.find('i.selected-indicator.fa.fa-check')).toHaveLength(1);
+  });
+});