refactor: inline host parent getter

Author: mdjastrzebski

src/helpers/__tests__/component-tree.test.tsx

@@ -3,7 +3,6 @@ import { Text, TextInput, View } from 'react-native';
 import { render, screen } from '../..';
 import {
   getHostChildren,
-  getHostParent,
   getHostSelves,
   getHostSiblings,
   getUnsafeRootElement,
@@ -19,43 +18,6 @@ function MultipleHostChildren() {
   );
 }
 
-describe('getHostParent()', () => {
-  it('returns host parent for host component', () => {
-    render(
-      <View testID="grandparent">
-        <View testID="parent">
-          <View testID="subject" />
-          <View testID="sibling" />
-        </View>
-      </View>,
-    );
-
-    const hostParent = getHostParent(screen.getByTestId('subject'));
-    expect(hostParent).toBe(screen.getByTestId('parent'));
-
-    const hostGrandparent = getHostParent(hostParent);
-    expect(hostGrandparent).toBe(screen.getByTestId('grandparent'));
-
-    expect(getHostParent(hostGrandparent)).toBe(screen.UNSAFE_root);
-  });
-
-  it('returns host parent for null', () => {
-    expect(getHostParent(null)).toBe(null);
-  });
-
-  it('returns host parent for composite component', () => {
-    render(
-      <View testID="parent">
-        <MultipleHostChildren />
-        <View testID="subject" />
-      </View>,
-    );
-
-    // eslint-disable-next-line jest/no-conditional-expect
-    expect(true).toBeTruthy();
-  });
-});
-
 describe('getHostChildren()', () => {
   it('returns host children for host component', () => {
     render(

src/helpers/component-tree.ts

@@ -8,18 +8,6 @@ export function isHostElement(element?: HostElement | null): element is HostElem
   return typeof element?.type === 'string' && element.type !== 'CONTAINER';
 }
 
-/**
- * Returns first host ancestor for given element.
- * @param element The element start traversing from.
- */
-export function getHostParent(element: HostElement | null): HostElement | null {
-  if (element == null) {
-    return null;
-  }
-
-  return element.parent;
-}
-
 /**
  * Returns host children for given element.
  * @param element The element start traversing from.
@@ -62,7 +50,7 @@ export function getHostSelves(element: HostElement | null): HostElement[] {
  * @param element The element start traversing from.
  */
 export function getHostSiblings(element: HostElement | null): HostElement[] {
-  const hostParent = getHostParent(element);
+  const hostParent = element?.parent ?? null;
   const hostSelves = getHostSelves(element);
   return getHostChildren(hostParent).filter((sibling) => !hostSelves.includes(sibling));
 }

src/helpers/pointer-events.ts

@@ -1,5 +1,4 @@
 import { HostElement } from '../renderer/host-element';
-import { getHostParent } from './component-tree';
 
 /**
  * pointerEvents controls whether the View can be the target of touch events.
@@ -17,7 +16,7 @@ export const isPointerEventEnabled = (element: HostElement, isParent?: boolean):
     return false;
   }
 
-  const hostParent = getHostParent(element);
+  const hostParent = element.parent;
   if (!hostParent) return true;
 
   return isPointerEventEnabled(hostParent, true);

src/matchers/to-be-disabled.tsx

@@ -1,6 +1,5 @@
 import { matcherHint } from 'jest-matcher-utils';
 import { computeAriaDisabled } from '../helpers/accessibility';
-import { getHostParent } from '../helpers/component-tree';
 import { HostElement } from '../renderer/host-element';
 import { checkHostElement, formatElement } from './utils';
 
@@ -43,7 +42,7 @@ export function toBeEnabled(this: jest.MatcherContext, element: HostElement) {
 }
 
 function isAncestorDisabled(element: HostElement): boolean {
-  const parent = getHostParent(element);
+  const parent = element.parent;
   if (parent == null) {
     return false;
   }

src/matchers/to-be-visible.tsx

@@ -1,7 +1,6 @@
 import { matcherHint } from 'jest-matcher-utils';
 import { StyleSheet } from 'react-native';
 import { isHiddenFromAccessibility } from '../helpers/accessibility';
-import { getHostParent } from '../helpers/component-tree';
 import { isHostModal } from '../helpers/host-component-names';
 import { HostElement } from '../renderer/host-element';
 import { checkHostElement, formatElement } from './utils';
@@ -45,7 +44,7 @@ function isElementVisible(
     return false;
   }
 
-  const hostParent = getHostParent(element);
+  const hostParent = element.parent;
   if (hostParent === null) {
     return true;
   }

src/user-event/press/press.ts

@@ -1,5 +1,4 @@
 import act from '../../act';
-import { getHostParent } from '../../helpers/component-tree';
 import { isTextInputEditable } from '../../helpers/text-input';
 import { isPointerEventEnabled } from '../../helpers/pointer-events';
 import { isHostText, isHostTextInput } from '../../helpers/host-component-names';
@@ -56,7 +55,7 @@ const basePress = async (
     return;
   }
 
-  const hostParentElement = getHostParent(element);
+  const hostParentElement = element.parent;
   if (!hostParentElement) {
     return;
   }