remove unnecessary hasDescendandHandler

Author: thymikee

src/__tests__/fireEvent.test.js

@@ -62,6 +62,16 @@ describe('fireEvent', () => {
     expect(onPressMock).toHaveBeenCalled();
   });
 
+  test('should not fire if the press handler is not passed to children', () => {
+    const onPressMock = jest.fn();
+    const { getByText } = render(
+      // TODO: this functionality is buggy, i.e. it will fail if we wrap this component with a View.
+      <WithoutEventComponent onPress={onPressMock} />
+    );
+    fireEvent(getByText('Without event'), 'press');
+    expect(onPressMock).not.toHaveBeenCalled();
+  });
+
   test('should invoke event with custom name', () => {
     const handlerMock = jest.fn();
     const EVENT_DATA = 'event data';

src/fireEvent.js

@@ -52,8 +52,7 @@ const findEventHandler = (
   element: ReactTestInstance,
   eventName: string,
   callsite?: any,
-  nearestTouchResponder?: ReactTestInstance,
-  hasDescendandHandler?: boolean
+  nearestTouchResponder?: ReactTestInstance
 ) => {
   const touchResponder = isTouchResponder(element)
     ? element
@@ -62,19 +61,11 @@ const findEventHandler = (
   const handler = getEventHandler(element, eventName);
   if (handler && isEventEnabled(element, touchResponder)) return handler;
 
-  // Do not bubble event to the root element
-  const hasHandler = handler != null || hasDescendandHandler;
   if (element.parent === null || element.parent.parent === null) {
     return null;
   }
 
-  return findEventHandler(
-    element.parent,
-    eventName,
-    callsite,
-    touchResponder,
-    hasHandler
-  );
+  return findEventHandler(element.parent, eventName, callsite, touchResponder);
 };
 
 const getEventHandler = (element: ReactTestInstance, eventName: string) => {