fix: support onXxx event name for TextInput event checks (#1404)

Author: mdjastrzebski

src/__tests__/fireEvent-textInput.test.tsx

@@ -62,13 +62,17 @@ test('should fire only non-touch-related events on non-editable TextInput with n
 
   const subject = view.getByText('Nested Text');
   fireEvent(subject, 'focus');
+  fireEvent(subject, 'onFocus');
   fireEvent.changeText(subject, 'Text');
   fireEvent(subject, 'submitEditing', { nativeEvent: { text: 'Text' } });
+  fireEvent(subject, 'onSubmitEditing', { nativeEvent: { text: 'Text' } });
   fireEvent(subject, 'layout', layoutEvent);
+  fireEvent(subject, 'onLayout', layoutEvent);
 
   expect(onFocus).not.toHaveBeenCalled();
   expect(onChangeText).not.toHaveBeenCalled();
   expect(onSubmitEditing).not.toHaveBeenCalled();
+  expect(onLayout).toHaveBeenCalledTimes(2);
   expect(onLayout).toHaveBeenCalledWith(layoutEvent);
 });
 

src/fireEvent.ts

@@ -40,20 +40,28 @@ function isPointerEventEnabled(
   return isPointerEventEnabled(parent, true);
 }
 
-// Due to accepting both `press` and `onPress` for event names, we need to
-// cover both forms.
-const touchEventNames = ['press', 'onPress'];
-
-function isTouchEvent(eventName: string) {
-  return touchEventNames.includes(eventName);
-}
-
-// Experimentally checked which events are called on non-editable TextInput
-const textInputEventsIgnoringEditableProp = [
+/**
+ * List of events affected by `pointerEvents` prop.
+ *
+ * Note: `fireEvent` is accepting both `press` and `onPress` for event names,
+ * so we need cover both forms.
+ */
+const eventsAffectedByPointerEventsProp = new Set(['press', 'onPress']);
+
+/**
+ * List of `TextInput` events not affected by `editable` prop.
+ *
+ * Note: `fireEvent` is accepting both `press` and `onPress` for event names,
+ * so we need cover both forms.
+ */
+const textInputEventsIgnoringEditableProp = new Set([
   'contentSizeChange',
+  'onContentSizeChange',
   'layout',
+  'onLayout',
   'scroll',
-];
+  'onScroll',
+]);
 
 function isEventEnabled(
   element: ReactTestInstance,
@@ -63,11 +71,14 @@ function isEventEnabled(
   if (isHostTextInput(nearestTouchResponder)) {
     return (
       nearestTouchResponder?.props.editable !== false ||
-      textInputEventsIgnoringEditableProp.includes(eventName)
+      textInputEventsIgnoringEditableProp.has(eventName)
     );
   }
 
-  if (isTouchEvent(eventName) && !isPointerEventEnabled(element)) {
+  if (
+    eventsAffectedByPointerEventsProp.has(eventName) &&
+    !isPointerEventEnabled(element)
+  ) {
     return false;
   }