Feat Review fixes

Author: Esemesek

README.md

@@ -148,6 +148,12 @@ test('Component has a structure', () => {
 });
 ```
 
+## `fireEvent`
+
+### press
+
+Invokes `press` event on the element or parent element
+
 ## `debug`
 
 Log prettified shallowly rendered component or test instance (just like snapshot) to stdout.

src/__mocks__/reactNativeMock.js

@@ -0,0 +1,6 @@
+// @flow
+export const View = (props: *) => props.children;
+export const ScrollView = (props: *) => props.children;
+export const Text = (props: *) => props.children;
+export const TextInput = () => null;
+export const TouchableOpacity = (props: *) => props.children;

src/__tests__/fireEvent.test.js

@@ -5,7 +5,7 @@ import {
   Text,
   ScrollView,
   TextInput,
-} from 'react-native'; // eslint-disable-line import/no-unresolved
+} from '../__mocks__/reactNativeMock';
 import fireEvent from '../fireEvent';
 import { render } from '..';
 
@@ -29,34 +29,12 @@ const CustomEventComponent = ({ onCustomEvent }) => (
   </TouchableOpacity>
 );
 
-jest.mock(
-  'react-native',
-  () => ({
-    View(props) {
-      return props.children;
-    },
-    ScrollView(props) {
-      return props.children;
-    },
-    Text(props) {
-      return props.children;
-    },
-    TextInput() {
-      return null;
-    },
-    TouchableOpacity(props) {
-      return props.children;
-    },
-  }),
-  { virtual: true }
-);
-
-describe('fireEvent.invokeEvent', () => {
+describe('fireEvent', () => {
   test('should invoke specified event', () => {
     const onPressMock = jest.fn();
     const { getByTestId } = render(<OnPressComponent onPress={onPressMock} />);
 
-    fireEvent.invokeEvent(getByTestId('button'), 'press');
+    fireEvent(getByTestId('button'), 'press');
 
     expect(onPressMock).toHaveBeenCalled();
   });
@@ -65,17 +43,17 @@ describe('fireEvent.invokeEvent', () => {
     const onPressMock = jest.fn();
     const { getByTestId } = render(<OnPressComponent onPress={onPressMock} />);
 
-    fireEvent.invokeEvent(getByTestId('text-button'), 'press');
+    fireEvent(getByTestId('text-button'), 'press');
 
     expect(onPressMock).toHaveBeenCalled();
   });
 
   test('should throw an Error when event handler was not found', () => {
     const { getByTestId } = render(<WithoutEventComponent />);
 
-    expect(() =>
-      fireEvent.invokeEvent(getByTestId('text'), 'press')
-    ).toThrowError('No handler function found for event: press');
+    expect(() => fireEvent(getByTestId('text'), 'press')).toThrowError(
+      'No handler function found for event: press'
+    );
   });
 
   test('should invoke event with custom name', () => {
@@ -88,7 +66,7 @@ describe('fireEvent.invokeEvent', () => {
       </View>
     );
 
-    fireEvent.invokeEvent(getByTestId('custom'), 'customEvent', EVENT_DATA);
+    fireEvent(getByTestId('custom'), 'customEvent', EVENT_DATA);
 
     expect(handlerMock).toHaveBeenCalledWith(EVENT_DATA);
   });

src/__tests__/render.test.js

@@ -1,24 +1,8 @@
 /* eslint-disable react/no-multi-comp */
 import React from 'react';
-import { View, Text, TouchableOpacity } from 'react-native'; // eslint-disable-line import/no-unresolved
+import { View, Text, TouchableOpacity } from '../__mocks__/reactNativeMock';
 import { render } from '..';
 
-jest.mock(
-  'react-native',
-  () => ({
-    View(props) {
-      return props.children;
-    },
-    Text(props) {
-      return props.children;
-    },
-    TouchableOpacity(props) {
-      return props.children;
-    },
-  }),
-  { virtual: true }
-);
-
 class Button extends React.Component {
   render() {
     return (

src/fireEvent.js

@@ -26,18 +26,38 @@ const invokeEvent = (
 const toEventHandlerName = (eventName: string) =>
   `on${eventName.charAt(0).toUpperCase()}${eventName.slice(1)}`;
 
-const press = (element: ReactTestInstance) => invokeEvent(element, 'press');
-const doublePress = (element: ReactTestInstance) =>
+const pressHandler = (element: ReactTestInstance) =>
+  invokeEvent(element, 'press');
+const doublePressHandler = (element: ReactTestInstance) =>
   invokeEvent(element, 'doublePress');
-const changeText = (element: ReactTestInstance, data: any) =>
+const changeTextHandler = (element: ReactTestInstance, data: any) =>
   invokeEvent(element, 'changeText', data);
-const scroll = (element: ReactTestInstance, data: any) =>
+const scrollHandler = (element: ReactTestInstance, data: any) =>
   invokeEvent(element, 'scroll', data);
 
-export default {
-  press,
-  doublePress,
-  changeText,
-  scroll,
-  invokeEvent,
-};
+const EVENTS = [
+  {
+    name: 'press',
+    handler: pressHandler,
+  },
+  {
+    name: 'doublePress',
+    handler: doublePressHandler,
+  },
+  {
+    name: 'changeText',
+    handler: changeTextHandler,
+  },
+  {
+    name: 'scroll',
+    handler: scrollHandler,
+  },
+];
+
+const fireEvent = invokeEvent;
+
+EVENTS.forEach(event => {
+  fireEvent[event.name] = event.handler;
+});
+
+export default fireEvent;

src/index.js

@@ -3,8 +3,10 @@ import render from './render';
 import shallow from './shallow';
 import flushMicrotasksQueue from './flushMicrotasksQueue';
 import debug from './debug';
+import fireEvent from './fireEvent';
 
 export { render };
 export { shallow };
 export { flushMicrotasksQueue };
 export { debug };
+export { fireEvent };