chore: run typecheck in unit tests (#18858)

vitejs · Dec 2, 2024 · 49f20bb · 49f20bb
1 parent b39e696
commit 49f20bb
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 13 deletions.
diff --git a/packages/vite/package.json b/packages/vite/package.json
@@ -75,7 +75,7 @@
     "build": "premove dist && pnpm build-bundle && pnpm build-types",
     "build-bundle": "rollup --config rollup.config.ts --configPlugin esbuild",
     "build-types": "pnpm build-types-temp && pnpm build-types-roll && pnpm build-types-check",
-    "build-types-temp": "tsc --emitDeclarationOnly --outDir temp -p src/node",
+    "build-types-temp": "tsc --emitDeclarationOnly --outDir temp -p src/node/tsconfig.build.json",
     "build-types-roll": "rollup --config rollup.dts.config.ts --configPlugin esbuild && premove temp",
     "build-types-check": "tsc --project tsconfig.check.json",
     "typecheck": "tsc --noEmit && tsc --noEmit -p src/node",

diff --git a/packages/vite/src/node/__tests__/config.spec.ts b/packages/vite/src/node/__tests__/config.spec.ts
@@ -1,7 +1,7 @@
 import http from 'node:http'
 import { describe, expect, test } from 'vitest'
-import type { InlineConfig } from '..'
-import type { PluginOption, UserConfig, UserConfigExport } from '../config'
+import type { InlineConfig, PluginOption } from '..'
+import type { UserConfig, UserConfigExport } from '../config'
 import { defineConfig, resolveConfig } from '../config'
 import { resolveEnvPrefix } from '../env'
 import { createLogger, mergeConfig } from '../publicUtils'

diff --git a/packages/vite/src/node/__tests__/plugins/json.spec.ts b/packages/vite/src/node/__tests__/plugins/json.spec.ts
@@ -30,7 +30,11 @@ test('can extract json error position', () => {
 })
 
 describe('transform', () => {
-  const transform = (input: string, opts: JsonOptions, isBuild: boolean) => {
+  const transform = (
+    input: string,
+    opts: Required<JsonOptions>,
+    isBuild: boolean,
+  ) => {
     const plugin = jsonPlugin(opts, isBuild)
     return (plugin.transform! as Function)(input, 'test.json').code
   }

diff --git a/packages/vite/src/node/__tests__/tsconfig.json b/packages/vite/src/node/__tests__/tsconfig.json
diff --git a/packages/vite/src/node/ssr/runtime/__tests__/utils.ts b/packages/vite/src/node/ssr/runtime/__tests__/utils.ts
@@ -32,6 +32,7 @@ export async function createModuleRunnerTester(
   }
 
   beforeEach<TestClient>(async (t) => {
+    // @ts-ignore
     globalThis.__HMR__ = {}
 
     t.server = await createServer({

diff --git a/packages/vite/src/node/tsconfig.build.json b/packages/vite/src/node/tsconfig.build.json
@@ -0,0 +1,4 @@
+{
+  "extends": "./tsconfig.json",
+  "exclude": ["**/__tests__"]
+}
diff --git a/packages/vite/src/node/tsconfig.json b/packages/vite/src/node/tsconfig.json
@@ -1,7 +1,7 @@
 {
   "extends": "../../tsconfig.base.json",
   "include": ["./", "../module-runner", "../types"],
-  "exclude": ["../**/__tests__"],
+  "exclude": ["**/fixtures"],
   "compilerOptions": {
     // https://github.com/microsoft/TypeScript/wiki/Node-Target-Mapping#node-18
     "lib": ["ES2023", "DOM"], // DOM because we have client code that gets inlined via function.toString()