Merge pull request #4668 from aryaemami59/modernize-vitest-config

Modernize vitest config

.github/workflows/test.yaml

@@ -89,7 +89,7 @@ jobs:
       - name: Install build artifact
         run: yarn  add ./package.tgz
 
-      - run: sed -i -e /@remap-prod-remove-line/d ./tsconfig.json ./vitest.config.ts ./test/tsconfig.json ./test/typescript/tsconfig.json
+      - run: sed -i -e /@remap-prod-remove-line/d ./tsconfig.json ./vitest.config.mts ./test/tsconfig.json ./test/typescript/tsconfig.json
 
       - name: Run tests, against dist
         run: yarn test

package.json

@@ -46,7 +46,7 @@
     "check-types": "tsc --noEmit && echo \"Types compiled\"",
     "test": "vitest --run",
     "test:types": "tsc -p test/typescript && echo \"Typetests passed\"",
-    "test:watch": "vitest",
+    "test:watch": "vitest --watch",
     "test:cov": "vitest --coverage",
     "test:typecheck": "tsc -p test && echo \"Types passed\"",
     "build": "tsup",
@@ -76,7 +76,7 @@
     "rxjs": "^7.5.6",
     "tsup": "7.0.0",
     "typescript": "5.2",
-    "vitest": "^0.34.0"
+    "vitest": "^1.2.1"
   },
   "resolutions": {
     "esbuild": "0.19.7",

tsconfig.json

@@ -15,7 +15,7 @@
     "noUnusedLocals": true,
     "noUnusedParameters": true,
     "baseUrl": "./",
-    "types": ["vitest/globals", "esbuild-extra/global"],
+    "types": ["vitest/globals", "esbuild-extra/global", "vitest/importMeta"],
     "paths": {
       "redux": ["src/index.ts"], // @remap-prod-remove-line
       "@internal/*": ["src/*"]

vitest.config.ts → vitest.config.mts

@@ -1,17 +1,21 @@
 import { defineConfig } from 'vitest/config'
 
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+// No __dirname under Node ESM
+const __filename = fileURLToPath(import.meta.url)
+const __dirname = path.dirname(__filename)
+
 export default defineConfig({
   test: {
     globals: true,
     include: ['./test/**/*.(spec|test).[jt]s?(x)'],
     alias: {
-      redux: './src/index.ts', // @remap-prod-remove-line
+      redux: path.join(__dirname, './src/index.ts'), // @remap-prod-remove-line
 
       // this mapping is disabled as we want `dist` imports in the tests only to be used for "type-only" imports which don't play a role for jest
-      '@internal/': './src/'
-    },
-    deps: {
-      interopDefault: true
+      '@internal': path.join(__dirname, 'src')
     }
   }
 })