18 to go

Author: mdjastrzebski

eslint.config.mjs

@@ -15,9 +15,15 @@ export default [
   },
   ...callstackConfig,
   ...tseslint.configs.strict,
+  {
+    rules: {
+      'no-console': 'error',
+    },
+  },
   {
     files: ['**/*.test.{ts,tsx}', 'src/test-utils/**'],
     rules: {
+      'react/no-multi-comp': 'off',
       'react-native/no-color-literals': 'off',
       'react-native/no-inline-styles': 'off',
       'react-native/no-raw-text': 'off',

src/__tests__/render.test.tsx

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import * as React from 'react';
 import { Pressable, Text, TextInput, View } from 'react-native';
 import { fireEvent, render, RenderAPI, screen } from '..';

src/__tests__/wait-for.test.tsx

@@ -19,7 +19,7 @@ class Banana extends React.Component<any> {
   }
 }
 
-class BananaContainer extends React.Component<{}, any> {
+class BananaContainer extends React.Component<object, any> {
   state = { fresh: false };
 
   onChangeFresh = async () => {
@@ -196,7 +196,7 @@ test.each([false, true])(
 
 const blockThread = (timeToBlockThread: number, legacyFakeTimers: boolean) => {
   jest.useRealTimers();
-  let end = Date.now() + timeToBlockThread;
+  const end = Date.now() + timeToBlockThread;
 
   while (Date.now() < end) {
     // do nothing

src/helpers/object.ts

@@ -1,4 +1,4 @@
-export function pick<T extends {}>(object: T, keys: (keyof T)[]): Partial<T> {
+export function pick<T extends object>(object: T, keys: (keyof T)[]): Partial<T> {
   const result: Partial<T> = {};
   keys.forEach((key) => {
     if (object[key] !== undefined) {

src/index.ts

@@ -10,7 +10,6 @@ if (!process?.env?.RNTL_SKIP_AUTO_CLEANUP) {
   // if you don't like this then either import the `pure` module
   // or set the RNTL_SKIP_AUTO_CLEANUP env variable to 'true'.
   if (typeof afterEach === 'function') {
-    // eslint-disable-next-line no-undef
     afterEach(async () => {
       await flushMicroTasks();
       cleanup();

src/render-hook.tsx

@@ -10,6 +10,7 @@ export type RenderHookResult<Result, Props> = {
 
 export type RenderHookOptions<Props> = {
   initialProps?: Props;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   wrapper?: ComponentType<any>;
 };
 

src/render.tsx

@@ -19,8 +19,9 @@ import { getQueriesForElement } from './within';
 export interface RenderOptions {
   /**
    * Pass a React Component as the wrapper option to have it rendered around the inner element. This is most useful for creating
-   *  reusable custom render functions for common data providers.
+   * reusable custom render functions for common data providers.
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   wrapper?: React.ComponentType<any>;
 
   /**

src/wait-for.ts

@@ -127,10 +127,10 @@ function waitForInternal<T>(
       try {
         const result = expectation();
 
-        // @ts-ignore result can be a promise
+        // @ts-expect-error result can be a promise
         // eslint-disable-next-line promise/prefer-await-to-then
         if (typeof result?.then === 'function') {
-          const promiseResult: Promise<T> = result as any;
+          const promiseResult: Promise<T> = result as unknown as Promise<T>;
           promiseStatus = 'pending';
           // eslint-disable-next-line promise/catch-or-return, promise/prefer-await-to-then
           promiseResult.then(