Single global vitest config, remove turbo

thetarnav · thetarnav · commit b9a99e173b12 · 2024-12-22T20:45:06.000+01:00
diff --git a/.github/workflows/page.yml b/.github/workflows/page.yml
@@ -27,14 +27,6 @@ jobs:
       - name: Install Dependencies
         run: pnpm install --ignore-scripts
 
-      - name: Cache turbo build setup
-        uses: actions/cache@v4
-        with:
-          path: node_modules/.cache/turbo
-          key: turbo-${{ github.sha }}
-          restore-keys: |
-            turbo-
-
       - name: Build
         run: pnpm build:packages
 
diff --git a/.github/workflows/release-ext.yml b/.github/workflows/release-ext.yml
@@ -24,14 +24,6 @@ jobs:
       - name: Install Dependencies
         run: pnpm install --ignore-scripts
 
-      - name: Cache turbo build setup
-        uses: actions/cache@v4
-        with:
-          path: node_modules/.cache/turbo
-          key: turbo-${{ github.sha }}
-          restore-keys: |
-            turbo-
-
       - name: Build
         run: pnpm build
 
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -26,14 +26,6 @@ jobs:
       - name: Install Dependencies
         run: pnpm install --no-frozen-lockfile --ignore-scripts
 
-      - name: Cache turbo build setup
-        uses: actions/cache@v4
-        with:
-          path: node_modules/.cache/turbo
-          key: turbo-${{ github.sha }}
-          restore-keys: |
-            turbo-
-
       - name: Create Release Pull Request or Publish to npm
         id: changesets
         uses: changesets/action@v1
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -27,14 +27,6 @@ jobs:
       - name: Install dependencies
         run: pnpm install --no-frozen-lockfile --ignore-scripts
 
-      - name: Cache turbo build setup
-        uses: actions/cache@v4
-        with:
-          path: node_modules/.cache/turbo
-          key: turbo-${{ github.sha }}
-          restore-keys: |
-            turbo-
-
       - name: Install Playwright browser and dependencies
         run: pnpm exec playwright install --with-deps chromium
 
diff --git a/.gitignore b/.gitignore
@@ -17,9 +17,6 @@ lerna-debug.log
 .env.test.local
 .env.production.local
 
-# turbo
-.turbo
-
 # solid
 .solid
 
diff --git a/README.md b/README.md
@@ -5,7 +5,6 @@
 # Solid Developer Tools
 
 [![pnpm](https://img.shields.io/badge/maintained%20with-pnpm-cc00ff.svg?style=for-the-badge&logo=pnpm)](https://pnpm.io/)
-[![turborepo](https://img.shields.io/badge/built%20with-turborepo-cc00ff.svg?style=for-the-badge&logo=turborepo)](https://turborepo.org/)
 [![npm](https://img.shields.io/npm/dw/@solid-devtools/shared?color=blue&style=for-the-badge)](https://www.npmjs.com/package/solid-devtools)
 
 Library of developer tools, reactivity debugger & Devtools Chrome extension for visualizing SolidJS reactivity graph.
diff --git a/build_shared.ts b/build_shared.ts
@@ -13,7 +13,6 @@ function is_env_truthy(value: string | undefined): boolean {
 
 export const CI = is_env_truthy(process.env['CI'])
                || is_env_truthy(process.env['GITHUB_ACTIONS'])
-               || !!process.env['TURBO_HASH']
 
 // type Worker_Data = {
 //     is_dev:     boolean,
diff --git a/package.json b/package.json
@@ -18,7 +18,7 @@
         "build:types": "tsc -b packages/*",
         "build": "pnpm build:packages && pnpm build:ext",
         "----------------------TEST----------------------": "",
-        "test:unit": "turbo run test:unit --filter=\"./packages/*\" --filter=\"./extension\"",
+        "test:unit": "vitest",
         "test:types": "pnpm -dir extension run test:types",
         "test:lint": "eslint --max-warnings 0",
         "test:e2e": "playwright test -c e2e/playwright.config.ts",
@@ -44,7 +44,6 @@
         "eslint-plugin-no-only-tests": "^3.3.0",
         "jsdom": "^25.0.1",
         "solid-js": "^1.9.3",
-        "turbo": "^1.10.16",
         "typescript": "^5.7.2",
         "@unocss/core": "0.65.1",
         "@unocss/preset-uno": "0.65.1",
diff --git a/packages/debugger/package.json b/packages/debugger/package.json
@@ -51,9 +51,6 @@
             }
         }
     },
-    "scripts": {
-        "test:unit": "vitest"
-    },
     "dependencies": {
         "@nothing-but/utils": "~0.17.0",
         "@solid-devtools/shared": "workspace:^",
diff --git a/packages/debugger/src/dependency/test/collect.test.ts b/packages/debugger/src/dependency/test/collect.test.ts
@@ -1,3 +1,5 @@
+import '../../setup.ts'
+
 import * as s from 'solid-js'
 import * as test from 'vitest'
 import {NodeType} from '../../main/constants.ts'
diff --git a/packages/debugger/src/inspector/test/index.test.tsx b/packages/debugger/src/inspector/test/index.test.tsx
@@ -1,3 +1,5 @@
+import '../../setup.ts'
+
 import {
     createComputed,
     createMemo,
diff --git a/packages/debugger/src/inspector/test/serialize.test.ts b/packages/debugger/src/inspector/test/serialize.test.ts
@@ -1,3 +1,5 @@
+import '../../setup.ts'
+
 import {type Truthy} from '@solid-primitives/utils'
 import {createMutable, createStore} from 'solid-js/store'
 import {describe, expect, test, vi} from 'vitest'
diff --git a/packages/debugger/src/inspector/test/store.test.ts b/packages/debugger/src/inspector/test/store.test.ts
@@ -1,3 +1,5 @@
+import '../../setup.ts'
+
 import {createRoot} from 'solid-js'
 import {createMutable, createStore, modifyMutable, produce, reconcile, unwrap} from 'solid-js/store'
 import {beforeEach, describe, expect, it, vi} from 'vitest'
diff --git a/packages/debugger/src/main/test/update.test.ts b/packages/debugger/src/main/test/update.test.ts
@@ -1,3 +1,5 @@
+import '../../setup.ts'
+
 import {createComputed, createRoot, createSignal, onCleanup} from 'solid-js'
 import {describe, expect, it} from 'vitest'
 import {
diff --git a/packages/debugger/src/main/test/utils.test.ts b/packages/debugger/src/main/test/utils.test.ts
@@ -1,3 +1,5 @@
+import '../../setup.ts'
+
 import * as s from 'solid-js'
 import * as vi from 'vitest'
 import {NodeType} from '../constants.ts'
diff --git a/packages/debugger/src/structure/test/walker.test.tsx b/packages/debugger/src/structure/test/walker.test.tsx
@@ -1,3 +1,5 @@
+import '../../setup.ts'
+
 import * as s from 'solid-js'
 import * as test from 'vitest'
 import {NodeType, TreeWalkerMode} from '../../main/constants.ts'
diff --git a/packages/debugger/vite.config.ts b/packages/debugger/vite.config.ts
diff --git a/packages/frontend/package.json b/packages/frontend/package.json
@@ -28,9 +28,6 @@
     "main": "dist/index.js",
     "module": "dist/index.js",
     "types": "dist/index.d.ts",
-    "scripts": {
-        "test:unit": "vitest"
-    },
     "dependencies": {
         "@nothing-but/utils": "~0.17.0",
         "@solid-devtools/debugger": "workspace:^",
diff --git a/packages/frontend/vite.config.ts b/packages/frontend/vite.config.ts
diff --git a/packages/main/package.json b/packages/main/package.json
@@ -76,9 +76,6 @@
         },
         "./package.json": "./package.json"
     },
-    "scripts": {
-        "test:unit": "vitest"
-    },
     "devDependencies": {
         "@babel/core": "^7.26.0",
         "@babel/generator": "^7.26.3",
diff --git a/packages/main/vite.config.ts b/packages/main/vite.config.ts
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/turbo.json b/turbo.json
diff --git a/vitest.config.ts b/vitest.config.ts
@@ -4,30 +4,21 @@ import * as vi    from 'vitest/config'
 
 const cwd = process.cwd()
 
-export const testConfig: vi.ViteUserConfig['test'] = {
-    passWithNoTests: true,
-    watch:           false,
-    globals:         true,
-    clearMocks:      true,
-    environment:     'jsdom',
-}
-
-export const resolveConfig = {
-    conditions: ['browser', 'development'],
-    alias: {
-        'solid-js/web':   path.resolve(cwd, 'node_modules/solid-js/web/dist/dev.js'),
-        'solid-js/store': path.resolve(cwd, 'node_modules/solid-js/store/dist/dev.js'),
-        'solid-js':       path.resolve(cwd, 'node_modules/solid-js/dist/dev.js'),
+export default vi.defineConfig({
+    plugins: [solid() as any],
+    test:    {
+        passWithNoTests: true,
+        watch:           false,
+        environment:     'jsdom',
+        isolate:         false,
+        include:         ['packages/**/*.test.{ts,tsx,js,jsx}'],
     },
-} satisfies vi.ViteUserConfig['resolve']
-
-export const vitestFullConfig = (patch?: (config: vi.ViteUserConfig) => void) =>
-    vi.defineConfig(() => {
-        const config: vi.ViteUserConfig = {
-            plugins: [solid() as any],
-            test: testConfig,
-            resolve: resolveConfig,
-        }
-        patch?.(config)
-        return config
-    })
+    resolve: {
+        conditions: ['browser', 'development'],
+        alias:      {
+            'solid-js/web':   path.resolve(cwd, 'node_modules/solid-js/web/dist/dev.js'),
+            'solid-js/store': path.resolve(cwd, 'node_modules/solid-js/store/dist/dev.js'),
+            'solid-js':       path.resolve(cwd, 'node_modules/solid-js/dist/dev.js'),
+        },
+    },
+})

Original file line number	Diff line number	Diff line change
`@@ -51,9 +51,6 @@`
`51`	`51`	`}`
`52`	`52`	`}`
`53`	`53`	`},`
`54`		`- "scripts": {`
`55`		`- "test:unit": "vitest"`
`56`		`- },`
`57`	`54`	`"dependencies": {`
`58`	`55`	`"@nothing-but/utils": "~0.17.0",`
`59`	`56`	`"@solid-devtools/shared": "workspace:^",`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+import '../../setup.ts'`
	`2`	`+`
`1`	`3`	`import * as s from 'solid-js'`
`2`	`4`	`import * as test from 'vitest'`
`3`	`5`	`import {NodeType} from '../../main/constants.ts'`