Maybe really fix dist tests

Author: markerikson

.github/workflows/tests.yml

@@ -114,14 +114,21 @@ jobs:
       - name: Erase path aliases
         run: sed -i -e /@remap-prod-remove-line/d ./tsconfig.base.json
 
-      - name: Run tests, against dist
-        env:
-          TEST_DIST: true
-        run: yarn test
-
       - name: Run type tests with `moduleResolution Bundler`
         run: rm -rf dist && yarn tsc -p . --moduleResolution Bundler --module ESNext --noEmit false --declaration --emitDeclarationOnly --outDir dist --target ESNext && rm -rf dist
 
+      - name: Ensure there's no dist folder
+        run: rm -rf dist
+
+      - name: Change RTK package name to ensure node_modules is used
+        run: sed -i -e 's|@reduxjs/toolkit|@reduxjs/toolkit-test|' ./package.json
+
+      - name: Run tests, against installed artifact
+        env:
+          TEST_DIST: true
+        working-directory: ./packages/toolkit
+        run: ../../node_modules/.bin/vitest --run --typecheck
+
   test-types:
     name: 'Test Types: TS ${{ matrix.ts }} and React ${{ matrix.react.version }}'
 

packages/toolkit/vitest.config.mts

@@ -10,22 +10,15 @@ const __dirname = path.dirname(__filename)
 export default defineConfig({
   plugins: [tsconfigPaths({ root: __dirname })],
   test: {
-    alias: process.env.TEST_DIST
-      ? {
-          '@reduxjs/toolkit': new URL(
-            'node_modules/@reduxjs/toolkit',
-            import.meta.url,
-          ).pathname,
-        }
-      : undefined,
     globals: true,
     environment: 'jsdom',
     setupFiles: ['./vitest.setup.ts'],
     include: ['./src/**/*.(spec|test).[jt]s?(x)'],
-    server: { deps: { inline: ['redux', '@reduxjs/toolkit'] } },
+    server: {
+      deps: {
+        inline: ['redux', '@reduxjs/toolkit'],
+      },
+    },
     unstubEnvs: true,
   },
-  resolve: {
-    mainFields: ['module'],
-  },
 })