done

Author: mdjastrzebski

src/__tests__/react-native-animated.tsx renamed to src/__tests__/react-native-animated.test.tsx

@@ -46,13 +46,11 @@ function withGlobalActEnvironment(actImplementation: ReactAct) {
               // eslint-disable-next-line promise/always-return
               (returnValue) => {
                 setIsReactActEnvironment(previousActEnvironment);
-                // @ts-expect-error
-                resolve(returnValue);
+                resolve(returnValue as never);
               },
               (error) => {
                 setIsReactActEnvironment(previousActEnvironment);
-                // @ts-expect-error
-                reject(error);
+                reject(error as never);
               },
             );
           },
@@ -70,7 +68,7 @@ function withGlobalActEnvironment(actImplementation: ReactAct) {
   };
 }
 
-// @ts-expect-error
+// @ts-expect-error: typings get too complex
 const act = withGlobalActEnvironment(reactAct) as ReactAct;
 
 export default act;

src/act.ts

@@ -123,7 +123,7 @@ test('toContainElement() handles non-element element', () => {
   const view = screen.getByTestId('view');
 
   expect(() =>
-    // @ts-expect-error
+    // @ts-expect-error: intentionally passing wrong element shape
     expect(view).not.toContainElement({ name: 'non-element' }),
   ).toThrowErrorMatchingInlineSnapshot(`
     "expect(received).not.toContainElement()

src/matchers/__tests__/to-contain-element.test.tsx

@@ -4,7 +4,7 @@ import { render, screen } from '../..';
 import { checkHostElement, formatElement } from '../utils';
 
 function fakeMatcher() {
-  // Do nothing.
+  return { pass: true, message: () => 'fake' };
 }
 
 test('formatElement', () => {
@@ -15,7 +15,7 @@ test('checkHostElement allows host element', () => {
   render(<View testID="view" />);
 
   expect(() => {
-    // @ts-expect-error
+    // @ts-expect-error: intentionally passing wrong element shape
     checkHostElement(screen.getByTestId('view'), fakeMatcher, {});
   }).not.toThrow();
 });
@@ -24,14 +24,14 @@ test('checkHostElement allows rejects composite element', () => {
   render(<View testID="view" />);
 
   expect(() => {
-    // @ts-expect-error
+    // @ts-expect-error: intentionally passing wrong element shape
     checkHostElement(screen.UNSAFE_root, fakeMatcher, {});
   }).toThrow(/value must be a host element./);
 });
 
 test('checkHostElement allows rejects null element', () => {
   expect(() => {
-    // @ts-expect-error
+    // @ts-expect-error: intentionally passing wrong element shape
     checkHostElement(null, fakeMatcher, {});
   }).toThrowErrorMatchingInlineSnapshot(`
     "expect(received).fakeMatcher()

src/matchers/__tests__/utils.test.tsx

@@ -43,7 +43,6 @@ function waitForInternal<T>(
       // infinite loop. However, eslint isn't smart enough to know that we're
       // setting finished inside `onDone` which will be called when we're done
       // waiting or when we've timed out.
-      // eslint-disable-next-line no-unmodified-loop-condition
       let fakeTimeRemaining = timeout;
       while (!finished) {
         if (!jestFakeTimersAreEnabled()) {
@@ -82,7 +81,6 @@ function waitForInternal<T>(
         // to resolve before continuing. We don't need to take advantage
         // of parallelization so we're fine.
         // https://stackoverflow.com/a/59243586/971592
-        // eslint-disable-next-line no-await-in-loop
         await flushMicroTasks();
       }
     } else {
@@ -128,7 +126,6 @@ function waitForInternal<T>(
         const result = expectation();
 
         // @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 unknown as Promise<T>;
           promiseStatus = 'pending';