test(event-handler): migrate to vitest (#3126)

aws-powertools · Sep 27, 2024 · 39c8608 · 39c8608
1 parent 6d4b3de
commit 39c8608
Show file tree

Hide file tree

Showing 8 changed files with 23 additions and 48 deletions.
diff --git a/.github/workflows/reusable-run-linting-check-and-unit-tests.yml b/.github/workflows/reusable-run-linting-check-and-unit-tests.yml
@@ -44,7 +44,8 @@ jobs:
           "packages/batch",
           "packages/commons",
           "packages/jmespath",
-          "packages/idempotency"
+          "packages/idempotency",
+          "packages/event-handler"
         ]
       fail-fast: false
     steps:
@@ -91,16 +92,14 @@ jobs:
             -w packages/tracer \
             -w packages/metrics \
             -w packages/parameters \
-            -w packages/parser \
-            -w packages/event-handler
+            -w packages/parser
       - name: Run unit tests
         run: |
           npm t -w packages/logger \
             -w packages/tracer \
             -w packages/metrics \
             -w packages/parameters \
-            -w packages/parser \
-            -w packages/event-handler
+            -w packages/parser
   check-examples:
     runs-on: ubuntu-latest
     env:

diff --git a/.husky/pre-push b/.husky/pre-push
@@ -3,7 +3,6 @@ npm t \
   -w packages/metrics \
   -w packages/tracer \
   -w packages/parameters \
-  -w packages/parser \
-  -w packages/event-handler
+  -w packages/parser
 
 npx vitest --run --coverage --changed="$(git merge-base HEAD main)"
diff --git a/packages/event-handler/jest.config.cjs b/packages/event-handler/jest.config.cjs
diff --git a/packages/event-handler/package.json b/packages/event-handler/package.json
@@ -8,8 +8,10 @@
   },
   "private": true,
   "scripts": {
-    "test": "npm run test:unit",
-    "test:unit": "jest --group=unit --detectOpenHandles --coverage --verbose",
+    "test": "vitest --run",
+    "test:unit": "vitest --run",
+    "test:unit:coverage": "vitest --run tests/unit --coverage.enabled --coverage.thresholds.100 --coverage.include='src/**'",
+    "test:unit:types": "echo 'Not Implemented'",
     "test:e2e:nodejs18x": "echo \"Not implemented\"",
     "test:e2e:nodejs20x": "echo \"Not implemented\"",
     "test:e2e": "echo \"Not implemented\"",

diff --git a/packages/event-handler/tests/tsconfig.json b/packages/event-handler/tests/tsconfig.json
@@ -1,8 +1,8 @@
 {
   "extends": "../tsconfig.json",
   "compilerOptions": {
-    "rootDir": "../",
+    "rootDir": "../../",
     "noEmit": true
   },
-  "include": ["../src/**/*", "./**/*"]
+  "include": ["../../testing/src/setupEnv.ts", "../src/**/*", "./**/*"]
 }
diff --git a/packages/event-handler/tests/unit/index.test.ts b/packages/event-handler/tests/unit/index.test.ts
@@ -1,16 +1,12 @@
-/**
- * Test Event Handler
- *
- * @group unit/event-handler
- */
+import { afterAll, beforeEach, describe, expect, it, vi } from 'vitest';
 import { foo } from '../../src/index.js';
 
 describe('Event Handler', () => {
   const ENVIRONMENT_VARIABLES = process.env;
 
   beforeEach(() => {
-    jest.clearAllMocks();
-    jest.resetModules();
+    vi.clearAllMocks();
+    vi.resetModules();
     process.env = { ...ENVIRONMENT_VARIABLES };
   });
 

diff --git a/packages/event-handler/vitest.config.ts b/packages/event-handler/vitest.config.ts
@@ -0,0 +1,8 @@
+import { defineProject } from 'vitest/config';
+
+export default defineProject({
+  test: {
+    environment: 'node',
+    setupFiles: ['../testing/src/setupEnv.ts'],
+  },
+});
diff --git a/vitest.config.ts b/vitest.config.ts
@@ -15,7 +15,7 @@ export default defineConfig({
         ...coverageConfigDefaults.exclude,
         'packages/batch/src/types.ts',
         'packages/commons/src/types/**',
-        'packages/event-handler/**',
+        'packages/event-handler/src/types/**',
         'packages/idempotency/src/types/**',
         'packages/jmespath/src/types.ts',
         'packages/logger/**',