fix lint and add tests

Author: OlimpiaZurek

src/index.js

@@ -10,9 +10,9 @@ import { Dimensions } from 'react-native';
 // This height was tested thoroughly on several iPhone Models (from iPhone 8 to 14 Pro)
 const IOS_MODAL_HEIGHT = 262;
 
-const preserveSpaces = (label) =>  {
+const preserveSpaces = (label) => {
     return label.replace(/ /g, '\u00a0');
- }
+};
 
 export default class RNPickerSelect extends PureComponent {
     static propTypes = {
@@ -282,7 +282,7 @@ export default class RNPickerSelect extends PureComponent {
                 }
             }
         );
-    }
+    };
 
     togglePicker(animate = false, postToggleCallback) {
         const { disabled } = this.props;
@@ -295,8 +295,8 @@ export default class RNPickerSelect extends PureComponent {
 
         if (Keyboard.isVisible()) {
             const keyboardListener = Keyboard.addListener('keyboardDidHide', () => {
-               this.updatePickerState(animate, postToggleCallback);
-               keyboardListener.remove();
+                this.updatePickerState(animate, postToggleCallback);
+                keyboardListener.remove();
             });
             Keyboard.dismiss();
         } else {

test/test.js

@@ -42,6 +42,8 @@ describe('RNPickerSelect', () => {
     beforeEach(() => {
         jest.resetAllMocks();
         jest.spyOn(Keyboard, 'dismiss');
+        jest.spyOn(Keyboard, 'addListener');
+        Keyboard.isVisible = jest.fn().mockReturnValue(false);
     });
 
     describe('when provided an itemKey prop', () => {
@@ -144,6 +146,25 @@ describe('RNPickerSelect', () => {
         expect(wrapper.state().showPicker).toEqual(true);
     });
 
+    it('should call Keyboard.addListener when keyboard is visible', () => {
+        Keyboard.isVisible.mockReturnValue(true);
+        const wrapper = shallow(<RNPickerSelect items={selectItems} onValueChange={noop} />);
+
+        const touchable = wrapper.find('TouchableOpacity').at(1);
+        touchable.simulate('press');
+        expect(Keyboard.addListener).toHaveBeenCalledTimes(1);
+        expect(Keyboard.addListener).toHaveBeenCalledWith('keyboardDidHide', expect.any(Function));
+    });
+
+    it('should not call Keyboard.addListener when keyboard is not visible', () => {
+        Keyboard.isVisible.mockReturnValue(false);
+        const wrapper = shallow(<RNPickerSelect items={selectItems} onValueChange={noop} />);
+
+        const touchable = wrapper.find('TouchableOpacity').at(1);
+        touchable.simulate('press');
+        expect(Keyboard.addListener).not.toHaveBeenCalled();
+    });
+
     it('should not show the picker when pressed if disabled', () => {
         const wrapper = shallow(
             <RNPickerSelect
@@ -245,6 +266,7 @@ describe('RNPickerSelect', () => {
     });
 
     it('should call Keyboard.dismiss when opened', () => {
+        Keyboard.isVisible.mockReturnValue(true);
         const wrapper = shallow(<RNPickerSelect items={selectItems} onValueChange={noop} />);
 
         const touchable = wrapper.find('[testID="ios_touchable_wrapper"]');