refactor: improve code coverage, remove dead code

Author: mdjastrzebski

src/helpers/find-all.ts

@@ -42,8 +42,6 @@ function findAllInternal(
 ): HostElement[] {
   const results: HostElement[] = [];
 
-  //console.log(`${indent} 🟢 findAllInternal`, node.type, node.props);
-
   // Match descendants first but do not add them to results yet.
   const matchingDescendants: HostElement[] = [];
   node.children.forEach((child) => {
@@ -67,20 +65,3 @@ function findAllInternal(
 
   return results;
 }
-
-export function findAllByProps(
-  root: HostElement,
-  props: { [propName: string]: any },
-  options?: FindAllOptions,
-): HostElement[] {
-  return findAll(root, (element) => matchProps(element, props), options);
-}
-
-function matchProps(element: HostElement, props: { [propName: string]: any }): boolean {
-  for (const key in props) {
-    if (props[key] !== element.props[key]) {
-      return false;
-    }
-  }
-  return true;
-}

src/matchers/__tests__/to-be-empty-element.test.tsx

@@ -1,5 +1,5 @@
 import React from 'react';
-import { View } from 'react-native';
+import { Text, View } from 'react-native';
 import { render, screen } from '../..';
 
 // eslint-disable-next-line @typescript-eslint/no-unused-vars
@@ -12,6 +12,7 @@ test('toBeEmptyElement() base case', () => {
   render(
     <View testID="not-empty">
       <View testID="empty" />
+      <Text testID="text">Hello</Text>
     </View>,
   );
 
@@ -32,7 +33,21 @@ test('toBeEmptyElement() base case', () => {
     Received:
       <View
         testID="empty"
-      />"
+      />
+      <Text
+        testID="text"
+      >
+        Hello
+      </Text>"
+  `);
+
+  const text = screen.getByTestId('text');
+  expect(text).not.toBeEmptyElement();
+  expect(() => expect(text).toBeEmptyElement()).toThrowErrorMatchingInlineSnapshot(`
+    "expect(element).toBeEmptyElement()
+
+    Received:
+      Hello"
   `);
 });
 

src/matchers/__tests__/utils.test.tsx

@@ -8,6 +8,7 @@ function fakeMatcher() {
 }
 
 test('formatElement', () => {
+  expect(formatElement('Hello')).toMatchInlineSnapshot(`"Hello"`);
   expect(formatElement(null)).toMatchInlineSnapshot(`"  null"`);
 });