Skip to content

Commit 39c8608

Browse files
authored
test(event-handler): migrate to vitest (#3126)
1 parent 6d4b3de commit 39c8608

File tree

8 files changed

+23
-48
lines changed

8 files changed

+23
-48
lines changed

Diff for: .github/workflows/reusable-run-linting-check-and-unit-tests.yml

+4-5
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,8 @@ jobs:
4444
"packages/batch",
4545
"packages/commons",
4646
"packages/jmespath",
47-
"packages/idempotency"
47+
"packages/idempotency",
48+
"packages/event-handler"
4849
]
4950
fail-fast: false
5051
steps:
@@ -91,16 +92,14 @@ jobs:
9192
-w packages/tracer \
9293
-w packages/metrics \
9394
-w packages/parameters \
94-
-w packages/parser \
95-
-w packages/event-handler
95+
-w packages/parser
9696
- name: Run unit tests
9797
run: |
9898
npm t -w packages/logger \
9999
-w packages/tracer \
100100
-w packages/metrics \
101101
-w packages/parameters \
102-
-w packages/parser \
103-
-w packages/event-handler
102+
-w packages/parser
104103
check-examples:
105104
runs-on: ubuntu-latest
106105
env:

Diff for: .husky/pre-push

+1-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ npm t \
33
-w packages/metrics \
44
-w packages/tracer \
55
-w packages/parameters \
6-
-w packages/parser \
7-
-w packages/event-handler
6+
-w packages/parser
87

98
npx vitest --run --coverage --changed="$(git merge-base HEAD main)"

Diff for: packages/event-handler/jest.config.cjs

-29
This file was deleted.

Diff for: packages/event-handler/package.json

+4-2
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,10 @@
88
},
99
"private": true,
1010
"scripts": {
11-
"test": "npm run test:unit",
12-
"test:unit": "jest --group=unit --detectOpenHandles --coverage --verbose",
11+
"test": "vitest --run",
12+
"test:unit": "vitest --run",
13+
"test:unit:coverage": "vitest --run tests/unit --coverage.enabled --coverage.thresholds.100 --coverage.include='src/**'",
14+
"test:unit:types": "echo 'Not Implemented'",
1315
"test:e2e:nodejs18x": "echo \"Not implemented\"",
1416
"test:e2e:nodejs20x": "echo \"Not implemented\"",
1517
"test:e2e": "echo \"Not implemented\"",

Diff for: packages/event-handler/tests/tsconfig.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
{
22
"extends": "../tsconfig.json",
33
"compilerOptions": {
4-
"rootDir": "../",
4+
"rootDir": "../../",
55
"noEmit": true
66
},
7-
"include": ["../src/**/*", "./**/*"]
7+
"include": ["../../testing/src/setupEnv.ts", "../src/**/*", "./**/*"]
88
}

Diff for: packages/event-handler/tests/unit/index.test.ts

+3-7
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,12 @@
1-
/**
2-
* Test Event Handler
3-
*
4-
* @group unit/event-handler
5-
*/
1+
import { afterAll, beforeEach, describe, expect, it, vi } from 'vitest';
62
import { foo } from '../../src/index.js';
73

84
describe('Event Handler', () => {
95
const ENVIRONMENT_VARIABLES = process.env;
106

117
beforeEach(() => {
12-
jest.clearAllMocks();
13-
jest.resetModules();
8+
vi.clearAllMocks();
9+
vi.resetModules();
1410
process.env = { ...ENVIRONMENT_VARIABLES };
1511
});
1612

Diff for: packages/event-handler/vitest.config.ts

+8
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
import { defineProject } from 'vitest/config';
2+
3+
export default defineProject({
4+
test: {
5+
environment: 'node',
6+
setupFiles: ['../testing/src/setupEnv.ts'],
7+
},
8+
});

Diff for: vitest.config.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ export default defineConfig({
1515
...coverageConfigDefaults.exclude,
1616
'packages/batch/src/types.ts',
1717
'packages/commons/src/types/**',
18-
'packages/event-handler/**',
18+
'packages/event-handler/src/types/**',
1919
'packages/idempotency/src/types/**',
2020
'packages/jmespath/src/types.ts',
2121
'packages/logger/**',

0 commit comments

Comments
 (0)