MM-19678 - Restored setSelected in MultiSelectList component (matterm…

…ost#4033)

* Restored setSelected in multiselectList component

* Added unit test for setSelected being called

* Reverted commented method

* Linting

* Updated tests

components/multiselect/multiselect.test.jsx

@@ -5,7 +5,10 @@ import React from 'react';
 
 import {shallow} from 'enzyme';
 
-import MultiSelect from 'components/multiselect/multiselect.jsx';
+import {mountWithIntl} from 'tests/helpers/intl-test-helper.jsx';
+
+import MultiSelect from './multiselect.jsx';
+import MultiSelectList from './multiselect_list.jsx';
 
 describe('components/multiselect/multiselect', () => {
  const totalCount = 8;
@@ -42,4 +45,37 @@ describe('components/multiselect/multiselect', () => {
  expect(wrapper.state('page')).toEqual(1);
  expect(wrapper).toMatchSnapshot();
  });
+
+ test('MultiSelectList should match state on next page', () => {
+ function renderOption(option, isSelected, onAdd) {
+ return (
+ <p
+ key={option.id}
+ ref={isSelected ? 'selected' : option.id}
+ onClick={() => onAdd(option)}
+ >
+ {option.id}
+ </p>
+ );
+ }
+
+ function renderValue(props) {
+ return props.data.value;
+ }
+
+ const wrapper = mountWithIntl(
+ <MultiSelect
+ {...baseProps}
+ optionRenderer={renderOption}
+ valueRenderer={renderValue}
+ />
+ );
+
+ const listRef = wrapper.ref('list');
+ expect(listRef.setSelected).toBeTruthy();
+
+ expect(wrapper.find(MultiSelectList).state('selected')).toEqual(-1);
+ wrapper.find('.filter-control__next').simulate('click');
+ expect(wrapper.find(MultiSelectList).state('selected')).toEqual(0);
+ });
 });

components/multiselect/multiselect_list.jsx

@@ -50,6 +50,12 @@ export default class MultiSelectList extends React.Component {
  }
  }
 
+ // setSelected updates the selected index and is referenced
+ // externally by the MultiSelect component.
+ setSelected = (selected) => {
+ this.setState({selected});
+ }
+
  handleArrowPress = (e) => {
  if (cmdOrCtrlPressed(e) && e.shiftKey) {
  return;