some more knip reportings (#12314)

apollographql · Jan 29, 2025 · b69d07b · b69d07b
1 parent 3623e28
commit b69d07b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 55 deletions.
diff --git a/knip.config.js b/knip.config.js
@@ -29,7 +29,7 @@ const config = {
       "src/react/types/types.documentation.ts",
       "eslint-local-rules/index.js",
     ]),
-  project: ["src/**/*.ts", "config/*.[jt]s", "eslint-local-rules/*.[jt]s"],
+  project: ["src/**/*.ts{,x}", "config/*.[jt]s", "eslint-local-rules/*.[jt]s"],
   ignore: ["integration-tests/**/*", ".yalc/**/*"],
   ignoreBinaries: ["jq"],
   ignoreDependencies: [

diff --git a/src/testing/internal/renderHelpers.tsx b/src/testing/internal/renderHelpers.tsx
@@ -1,20 +1,9 @@
 import * as React from "react";
-import type { ReactElement } from "react";
-import { render } from "@testing-library/react";
-import type { Queries, RenderOptions, queries } from "@testing-library/react";
 import type { ApolloClient } from "../../core/index.js";
 import { ApolloProvider } from "../../react/index.js";
 import type { MockedProviderProps } from "../react/MockedProvider.js";
 import { MockedProvider } from "../react/MockedProvider.js";
 
-export interface RenderWithClientOptions<
-  Q extends Queries = typeof queries,
-  Container extends Element | DocumentFragment = HTMLElement,
-  BaseElement extends Element | DocumentFragment = Container,
-> extends RenderOptions<Q, Container, BaseElement> {
-  client: ApolloClient<any>;
-}
-
 export function createClientWrapper(
   client: ApolloClient<any>,
   Wrapper: React.JSXElementConstructor<{
@@ -32,31 +21,6 @@ export function createClientWrapper(
   };
 }
 
-export function renderWithClient<
-  Q extends Queries = typeof queries,
-  Container extends Element | DocumentFragment = HTMLElement,
-  BaseElement extends Element | DocumentFragment = Container,
->(
-  ui: ReactElement,
-  {
-    client,
-    wrapper,
-    ...renderOptions
-  }: RenderWithClientOptions<Q, Container, BaseElement>
-) {
-  return render(ui, {
-    ...renderOptions,
-    wrapper: createClientWrapper(client, wrapper),
-  });
-}
-
-export interface RenderWithMocksOptions<
-  Q extends Queries = typeof queries,
-  Container extends Element | DocumentFragment = HTMLElement,
-  BaseElement extends Element | DocumentFragment = Container,
-> extends RenderOptions<Q, Container, BaseElement>,
-    MockedProviderProps<any> {}
-
 export function createMockWrapper(
   renderOptions: MockedProviderProps<any>,
   Wrapper: React.JSXElementConstructor<{
@@ -73,20 +37,3 @@ export function createMockWrapper(
     );
   };
 }
-
-export function renderWithMocks<
-  Q extends Queries = typeof queries,
-  Container extends Element | DocumentFragment = HTMLElement,
-  BaseElement extends Element | DocumentFragment = Container,
->(
-  ui: ReactElement,
-  {
-    wrapper,
-    ...renderOptions
-  }: RenderWithMocksOptions<Q, Container, BaseElement>
-) {
-  return render(ui, {
-    ...renderOptions,
-    wrapper: createMockWrapper(renderOptions, wrapper),
-  });
-}
diff --git a/src/testing/react/MockedProvider.tsx b/src/testing/react/MockedProvider.tsx
@@ -27,7 +27,7 @@ export interface MockedProviderProps<TSerializedCache = {}> {
   connectToDevTools?: boolean;
 }
 
-export interface MockedProviderState {
+interface MockedProviderState {
   client: ApolloClient<any>;
 }