Tweaked Lint rule and fixed a few warnings

.eslintrc.js

@@ -27,10 +27,9 @@ module.exports = {
   },
   plugins: ["react"],
   rules: {
+    "react/no-did-update-set-state": "off",
     "react/react-in-jsx-scope": "off",
     "react/prop-types": "off",
-    "react/no-did-update-set-state": "off",
-    "@typescript-eslint/ban-ts-comment": "warn",
-    "@typescript-eslint/no-empty-function": "warn",
+    "@typescript-eslint/no-explicit-any": "off",
   },
 };

package.json

@@ -58,10 +58,12 @@
     "@babel/preset-env": "^7.22.5",
     "@babel/preset-typescript": "^7.21.5",
     "@preconstruct/cli": "^2.8.1",
+    "@types/assert": "^1.5.10",
     "@types/jest": "^26.0.15",
     "@types/react": "^18",
     "@types/react-dom": "^18",
     "@typescript-eslint/eslint-plugin": "^5.52.0",
+    "assert": "^2.0.0",
     "eslint": "^8.0.1",
     "eslint-config-prettier": "^9.0.0",
     "eslint-plugin-import": "^2.25.2",

src/ErrorBoundary.test.tsx

@@ -2,6 +2,7 @@
  * @jest-environment jsdom
  */
 
+import assert from "assert";
 import { createRef, PropsWithChildren, ReactElement, RefObject } from "react";
 import { createRoot } from "react-dom/client";
 import { act } from "react-dom/test-utils";
@@ -20,11 +21,13 @@ describe("ErrorBoundary", () => {
   let valueToThrow: any;
 
   beforeEach(() => {
-    // @ts-ignore
+    // @ts-expect-error This is a React internal
     global.IS_REACT_ACT_ENVIRONMENT = true;
 
     // Don't clutter the console with expected error text
-    jest.spyOn(console, "error").mockImplementation(() => {});
+    jest.spyOn(console, "error").mockImplementation(() => {
+      // No-op
+    });
 
     container = document.createElement("div");
     root = createRoot(container);
@@ -202,7 +205,8 @@ describe("ErrorBoundary", () => {
       expect(lastRenderedResetErrorBoundary).not.toBeNull();
       act(() => {
         shouldThrow = false;
-        lastRenderedResetErrorBoundary!();
+        assert(lastRenderedResetErrorBoundary !== null);
+        lastRenderedResetErrorBoundary();
       });
 
       expect(container.textContent).toBe("Content");
@@ -270,7 +274,8 @@ describe("ErrorBoundary", () => {
 
       act(() => {
         shouldThrow = false;
-        lastRenderedResetErrorBoundary!();
+        assert(lastRenderedResetErrorBoundary !== null);
+        lastRenderedResetErrorBoundary();
       });
 
       expect(container.textContent).toBe("Content");

src/useErrorBoundary.test.tsx

@@ -2,6 +2,7 @@
  * @jest-environment jsdom
  */
 
+import assert from "assert";
 import { createRoot } from "react-dom/client";
 import { act } from "react-dom/test-utils";
 import { ErrorBoundary } from "./ErrorBoundary";
@@ -12,11 +13,13 @@ describe("useErrorBoundary", () => {
   let lastRenderedUseErrorBoundaryApi: UseErrorBoundaryApi<Error> | null = null;
 
   beforeEach(() => {
-    // @ts-ignore
+    // @ts-expect-error This is a React internal
     global.IS_REACT_ACT_ENVIRONMENT = true;
 
     // Don't clutter the console with expected error text
-    jest.spyOn(console, "error").mockImplementation(() => {});
+    jest.spyOn(console, "error").mockImplementation(() => {
+      // No-op
+    });
 
     container = document.createElement("div");
     lastRenderedUseErrorBoundaryApi = null;
@@ -89,12 +92,14 @@ describe("useErrorBoundary", () => {
     expect(container.textContent).toBe("Child");
 
     act(() => {
-      showBoundary!(new Error("Example"));
+      assert(showBoundary !== null);
+      showBoundary(new Error("Example"));
     });
     expect(container.textContent).toBe("Error");
 
     act(() => {
-      resetBoundary!();
+      assert(resetBoundary !== null);
+      resetBoundary();
     });
     expect(container.textContent).toBe("Child");
   });

src/withErrorBoundary.test.tsx

@@ -14,11 +14,13 @@ describe("withErrorBoundary", () => {
   let valueToThrow: any;
 
   beforeEach(() => {
-    // @ts-ignore
+    // @ts-expect-error This is a React internal
     global.IS_REACT_ACT_ENVIRONMENT = true;
 
     // Don't clutter the console with expected error text
-    jest.spyOn(console, "error").mockImplementation(() => {});
+    jest.spyOn(console, "error").mockImplementation(() => {
+      // No-op
+    });
 
     container = document.createElement("div");
     root = createRoot(container);
@@ -58,7 +60,9 @@ describe("withErrorBoundary", () => {
     type Props = { foo: string };
 
     class Inner extends Component<Props> {
-      test() {}
+      test() {
+        // No-op
+      }
       render() {
         return this.props.foo;
       }