Skip to content

Commit cf6eb01

Browse files
authored
test: Remove explict casting of baseconfig in vite configs (#12979)
Something I noticed while doing my jest -> vitest conversions, we don't need this anymore.
1 parent 789231c commit cf6eb01

File tree

7 files changed

+7
-28
lines changed

7 files changed

+7
-28
lines changed

Diff for: packages/astro/vite.config.ts

+1-5
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,9 @@
1-
import type { UserConfig } from 'vitest';
2-
31
import baseConfig from '../../vite/vite.config';
42

53
export default {
64
...baseConfig,
75
test: {
8-
// test exists, no idea why TS doesn't recognize it
9-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
10-
...(baseConfig as UserConfig & { test: any }).test,
6+
...baseConfig.test,
117
environment: 'jsdom',
128
},
139
};

Diff for: packages/nestjs/vite.config.ts

+1-4
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,10 @@
1-
import type { UserConfig } from 'vitest';
21
import { defineConfig } from 'vitest/config';
32
import baseConfig from '../../vite/vite.config';
43

54
export default defineConfig({
65
...baseConfig,
76
test: {
8-
// test exists, no idea why TS doesn't recognize it
9-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
10-
...(baseConfig as UserConfig & { test: any }).test,
7+
...baseConfig.test,
118
environment: 'node',
129
},
1310
});

Diff for: packages/replay-internal/vitest.config.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
1-
import type { UserConfig } from 'vitest';
21
import { defineConfig } from 'vitest/config';
32

43
import baseConfig from '../../vite/vite.config';
54

65
export default defineConfig({
76
...baseConfig,
87
test: {
9-
...(baseConfig as UserConfig & { test: any }).test,
8+
...baseConfig.test,
109
coverage: {},
1110
globals: true,
1211
setupFiles: ['./test.setup.ts'],

Diff for: packages/solid/vite.config.ts

+1-4
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,11 @@
11
import solidPlugin from 'vite-plugin-solid';
2-
import type { UserConfig } from 'vitest';
32
import baseConfig from '../../vite/vite.config';
43

54
export default {
65
...baseConfig,
76
plugins: [solidPlugin({ hot: !process.env.VITEST })],
87
test: {
9-
// test exists, no idea why TS doesn't recognize it
10-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
11-
...(baseConfig as UserConfig & { test: any }).test,
8+
...baseConfig.test,
129
environment: 'jsdom',
1310
},
1411
};

Diff for: packages/solidstart/vite.config.ts

+1-4
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,11 @@
11
import solidPlugin from 'vite-plugin-solid';
2-
import type { UserConfig } from 'vitest';
32
import baseConfig from '../../vite/vite.config';
43

54
export default {
65
...baseConfig,
76
plugins: [solidPlugin({ hot: !process.env.VITEST })],
87
test: {
9-
// test exists, no idea why TS doesn't recognize it
10-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
11-
...(baseConfig as UserConfig & { test: any }).test,
8+
...baseConfig.test,
129
environment: 'jsdom',
1310
},
1411
};

Diff for: packages/svelte/vite.config.ts

+1-4
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,11 @@
11
import { svelte } from '@sveltejs/vite-plugin-svelte';
2-
import type { UserConfig } from 'vitest';
32
import baseConfig from '../../vite/vite.config';
43

54
export default {
65
...baseConfig,
76
plugins: [svelte({ hot: !process.env.VITEST })],
87
test: {
9-
// test exists, no idea why TS doesn't recognize it
10-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
11-
...(baseConfig as UserConfig & { test: any }).test,
8+
...baseConfig.test,
129
environment: 'jsdom',
1310
alias: [{ find: /^svelte$/, replacement: 'svelte/internal' }],
1411
},

Diff for: packages/sveltekit/vite.config.ts

+1-5
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,12 @@
11
import { dirname, resolve } from 'path';
22
import { fileURLToPath } from 'url';
33

4-
import type { UserConfig } from 'vitest';
5-
64
import baseConfig from '../../vite/vite.config';
75

86
export default {
97
...baseConfig,
108
test: {
11-
// test exists, no idea why TS doesn't recognize it
12-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
13-
...(baseConfig as UserConfig & { test: any }).test,
9+
...baseConfig.test,
1410
setupFiles: ['./test/vitest.setup.ts'],
1511
alias: [
1612
{

0 commit comments

Comments
 (0)