Skip to content

Commit 582861d

Browse files
authored
fix: Fix in-package test commands (#596)
Fixes running in-package test commands, which were inadvertently broken by #592.
1 parent f194333 commit 582861d

4 files changed

Lines changed: 10 additions & 13 deletions

File tree

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
1-
import { defineConfig, defineProject, mergeConfig } from 'vitest/config';
1+
import { mergeConfig } from '@ocap/test-utils/vitest-config';
2+
import { defineConfig, defineProject } from 'vitest/config';
23

34
import defaultConfig from '../../vitest.config.ts';
45

5-
export default defineConfig(() => {
6+
export default defineConfig((args) => {
67
delete defaultConfig.test?.setupFiles;
78

89
// We do not use our custom mergeConfig here
910
const config = mergeConfig(
11+
args,
1012
defaultConfig,
1113
defineProject({
1214
test: {
@@ -17,7 +19,6 @@ export default defineConfig(() => {
1719
);
1820

1921
delete config.test?.coverage;
20-
delete config.test?.projects;
2122

2223
return config;
2324
});

packages/kernel-language-model-service/vitest.config.e2e.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { defineConfig, defineProject } from 'vitest/config';
55
import defaultConfig from '../../vitest.config.ts';
66

77
export default defineConfig((args) => {
8-
const config = mergeConfig(
8+
return mergeConfig(
99
args,
1010
defaultConfig,
1111
defineProject({
@@ -17,8 +17,4 @@ export default defineConfig((args) => {
1717
},
1818
}),
1919
);
20-
21-
delete config.test?.projects;
22-
23-
return config;
2420
});

packages/nodejs/vitest.config.e2e.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { defineConfig, defineProject } from 'vitest/config';
44
import defaultConfig from '../../vitest.config.ts';
55

66
export default defineConfig((args) => {
7-
const config = mergeConfig(
7+
return mergeConfig(
88
args,
99
defaultConfig,
1010
defineProject({
@@ -16,8 +16,4 @@ export default defineConfig((args) => {
1616
},
1717
}),
1818
);
19-
20-
delete config.test?.projects;
21-
22-
return config;
2319
});

packages/test-utils/src/vitest-config.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,10 @@ import { mergeConfig as mergeVitestConfig } from 'vitest/config';
2323
*/
2424
export const mergeConfig = (args, ...rest) => {
2525
const config = mergeVitestConfig(...rest);
26+
27+
// This breaks tests in packages if set
28+
delete config.test?.projects;
29+
2630
if (args.mode === 'development') {
2731
delete config.test.coverage;
2832
} else {

0 commit comments

Comments
 (0)