Remove style changes

Author: Methuselah96

packages/react-select/src/Select.js

@@ -685,13 +685,13 @@ export default class Select extends Component<Props, State> {
     const { isMulti } = this.props;
     const { selectValue } = this.state;
     const candidate = this.getOptionValue(removedValue);
-    const filteredValue = selectValue.filter(
+    const newValues = selectValue.filter(
       i => this.getOptionValue(i) !== candidate
     );
     const newValue = isMulti
-      ? filteredValue
-      : filteredValue.length > 0
-      ? filteredValue[0]
+      ? newValues
+      : newValues.length > 0
+      ? newValues[0]
       : null;
     this.onChange(newValue, {
       action: 'remove-value',
@@ -713,11 +713,11 @@ export default class Select extends Component<Props, State> {
     const { isMulti } = this.props;
     const { selectValue } = this.state;
     const lastSelectedValue = selectValue[selectValue.length - 1];
-    const slicedValue = selectValue.slice(0, selectValue.length - 1);
+    const newValues = selectValue.slice(0, selectValue.length - 1);
     const newValue = isMulti
-      ? slicedValue
-      : slicedValue.length > 0
-      ? slicedValue[0]
+      ? newValues
+      : newValues.length > 0
+      ? newValues[0]
       : null;
     this.announceAriaLiveSelection({
       event: 'pop-value',

packages/react-select/src/__tests__/Select.test.js

@@ -1643,48 +1643,46 @@ test('should not call onChange on hitting backspace even when backspaceRemovesVa
   expect(onChangeSpy).not.toHaveBeenCalled();
 });
 
-cases(
-  'should call onChange with `null` on hitting backspace when backspaceRemovesValue is true',
-  ({ props = { ...BASIC_PROPS }, expectedValue }) => {
-    let onChangeSpy = jest.fn();
-    let { container } = render(
-      <Select
-        {...props}
-        backspaceRemovesValue
-        isClearable
-        onChange={onChangeSpy}
-      />
-    );
-    fireEvent.keyDown(container.querySelector('.react-select__control'), {
-      keyCode: 8,
-      key: 'Backspace',
-    });
-    expect(onChangeSpy).toHaveBeenCalledWith(null, expectedValue);
-  },
-  {
-    'and isMulti is false': {
-      props: {
-        ...BASIC_PROPS,
-        isMulti: false,
-      },
-      expectedValue: {
-        action: 'clear',
-        name: 'test-input-name',
-      },
-    },
-    'and isMulti is true': {
-      props: {
-        ...BASIC_PROPS,
-        isMulti: true,
-      },
-      expectedValue: {
-        action: 'pop-value',
-        name: 'test-input-name',
-        removedValue: undefined,
-      },
-    },
-  }
-);
+test('should call onChange with `null` on hitting backspace when backspaceRemovesValue is true and isMulti is false', () => {
+  let onChangeSpy = jest.fn();
+  let selectWrapper = mount(
+    <Select
+      {...BASIC_PROPS}
+      backspaceRemovesValue
+      isClearable
+      isMulti={false}
+      onChange={onChangeSpy}
+    />
+  );
+  selectWrapper
+    .find(Control)
+    .simulate('keyDown', { keyCode: 8, key: 'Backspace' });
+  expect(onChangeSpy).toHaveBeenCalledWith(null, {
+    action: 'clear',
+    name: 'test-input-name',
+  });
+});
+
+test('should call onChange with an array on hitting backspace when backspaceRemovesValue is true and isMulti is true', () => {
+  let onChangeSpy = jest.fn();
+  let selectWrapper = mount(
+    <Select
+      {...BASIC_PROPS}
+      backspaceRemovesValue
+      isClearable
+      isMulti
+      onChange={onChangeSpy}
+    />
+  );
+  selectWrapper
+    .find(Control)
+    .simulate('keyDown', { keyCode: 8, key: 'Backspace' });
+  expect(onChangeSpy).toHaveBeenCalledWith([], {
+    action: 'pop-value',
+    name: 'test-input-name',
+    removedValue: undefined,
+  });
+});
 
 test('multi select > clicking on X next to option will call onChange with all options other that the clicked option', () => {
   let onChangeSpy = jest.fn();