Allow any in test files

vitest-dev/vitest#4543 (comment)

Author: steida

apps/web/src/components/Code.tsx

@@ -155,16 +155,19 @@ function CodePanel({
   const childrenArray = Children.toArray(children);
   const child = childrenArray.length === 1 ? childrenArray[0] : null;
 
-  if (isValidElement(child)) {
-    // @ts-expect-error TODO: Fix this somehow
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-    tag = child.props.tag ?? tag;
-    // @ts-expect-error TODO: Fix this somehow
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-    label = child.props.label ?? label;
-    // @ts-expect-error TODO: Fix this somehow
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-    code = child.props.code ?? code;
+  if (
+    isValidElement(child) &&
+    typeof child.props === "object" &&
+    child.props !== null
+  ) {
+    const props = child.props as {
+      tag?: string;
+      label?: string;
+      code?: string;
+    };
+    tag = props.tag ?? tag;
+    label = props.label ?? label;
+    code = props.code ?? code;
   }
 
   // If no code prop is provided, try to extract it from children

eslint.config.mjs

@@ -77,4 +77,15 @@ export default defineConfig(
       "react-hooks/exhaustive-deps": "error",
     },
   },
+  {
+    // https://github.com/vitest-dev/vitest/issues/4543#issuecomment-1824881253
+    files: ["**/*.test.ts", "**/*.test.tsx"],
+    rules: {
+      "@typescript-eslint/no-unsafe-assignment": "off",
+      "@typescript-eslint/no-unsafe-member-access": "off",
+      "@typescript-eslint/no-unsafe-argument": "off",
+      "@typescript-eslint/no-unsafe-call": "off",
+      "@typescript-eslint/no-unsafe-return": "off",
+    },
+  },
 );

packages/common/test/Console.test.ts

@@ -31,7 +31,6 @@ const mockDate = vi.fn();
 vi.stubGlobal(
   "Date",
   vi.fn().mockImplementation(function (this: any) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-return
     return mockDate();
   }),
 );

packages/common/test/Error.test.ts

@@ -9,7 +9,6 @@ describe("createTransferableError", () => {
     expect(result.type).toBe("TransferableError");
     expect(result.error).toMatchObject({
       message: "Test error",
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       stack: expect.any(String),
     });
   });
@@ -22,20 +21,19 @@ describe("createTransferableError", () => {
     expect(result.type).toBe("TransferableError");
     expect(result.error).toMatchObject({
       message: "Outer error",
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       stack: expect.any(String),
       cause: {
         message: "Inner error",
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
         stack: expect.any(String),
       },
     });
   });
 
   test("excludes non-transferable error properties", () => {
     const error = new Error("Test error");
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-empty-function
-    (error as any).nonTransferable = () => {};
+    (error as any).nonTransferable = () => {
+      //
+    };
     const result = createTransferableError(error);
 
     expect(result.type).toBe("TransferableError");
@@ -79,7 +77,6 @@ describe("createTransferableError", () => {
 
   test("handles circular references", () => {
     const error: any = {};
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
     error.self = error; // Create a circular reference
     const result = createTransferableError(error);
 

packages/common/test/Result.test.ts

@@ -73,15 +73,13 @@ test("trySync", () => {
     ok: false,
     error: {
       type: "ParseError",
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       message: expect.stringContaining("SyntaxError"),
     },
   });
 });
 
 test("tryAsync", async () => {
-  // eslint-disable-next-line @typescript-eslint/require-await
-  const successfulPromise = async () => "success";
+  const successfulPromise = () => Promise.resolve("success");
 
   const successResult = await tryAsync(successfulPromise, (error) => ({
     type: "TestError",

packages/common/test/Type.test.ts

@@ -3073,9 +3073,7 @@ describe("Standard Schema V1", () => {
     };
 
     expect(user).toEqual({
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       name: expect.schemaMatching(NonEmptyTrimmedString100),
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       age: expect.schemaMatching(PositiveInt),
     });
   });

packages/web/test/SharedWebWorker.test.ts

@@ -1,7 +1,3 @@
-/* eslint-disable @typescript-eslint/no-unsafe-call */
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
-/* eslint-disable @typescript-eslint/no-unsafe-return */
-/* eslint-disable @typescript-eslint/no-unsafe-assignment */
 import { expect, test, vi, beforeEach, afterEach } from "vitest";
 import { createSharedWebWorker } from "../src/SharedWebWorker.js";
 import { SimpleName, wait } from "@evolu/common";