Support, organize & pick tests to run tests in Bun + Node

.github/workflows/ci-bun.yml

@@ -22,8 +22,10 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - uses: oven-sh/setup-bun@v1
+      # With the lockfile, Bun collapses over the `sharp` dependency in docs
+      - run: rm bun.lockb
       - name: Install dependencies
-        run: bun install --frozen-lockfile
+        run: bun install
       - name: Build knip
         run: bun run build
         working-directory: packages/knip

.github/workflows/ci-ts-latest.yml

@@ -22,10 +22,8 @@ jobs:
       - name: Build knip
         run: npm run build
         working-directory: packages/knip
-      - name: QA (format, lint, organize imports)
-        run: npm run ci
       - name: Test knip
-        run: npm run test
+        run: npm run test:node
         working-directory: packages/knip
       - name: Run knip
         run: npx knip

.github/workflows/ci.yml

@@ -39,8 +39,5 @@ jobs:
       - name: Run knip (production/strict)
         run: npx knip --production --strict
       - name: Test knip
-        run: npm run test
+        run: npm run test:node
         working-directory: packages/knip
-      - name: Build docs
-        run: npm run build
-        working-directory: packages/docs

knip.ts

@@ -11,6 +11,7 @@ const config: KnipConfig = {
       entry: ['src/{index,cli}.ts!', 'src/plugins/*/index.ts!'],
       project: ['src/**/*.ts!'],
       ignore: ['**/fixtures/**', '**/_template/**', '**/dist/**'],
+      ignoreBinaries: ['tsx'],
       ignoreDependencies: ['@pnpm/logger'],
       'node-test-runner': {
         entry: ['test/**/*.test.ts'],

packages/knip/package.json

@@ -40,6 +40,7 @@
     "lint": "biome lint .",
     "format": "biome format --write .",
     "test": "bun test test/**/*.test.ts",
+    "test:node": "glob -c \"tsx --test --import ./transform-test.js\" \"test/**/*.test.ts\"",
     "watch": "npm link && tsc --watch",
     "prebuild": "node rmdir.js dist",
     "build": "tsc",
@@ -89,6 +90,7 @@
     "@types/npmcli__package-json": "^4.0.4",
     "@types/resolve": "^1.20.6",
     "@types/webpack": "^5.28.5",
+    "glob": "^10.3.12",
     "playwright": "^1.43.0",
     "release-it": "^17.1.1",
     "tsx": "^4.7.2",

packages/knip/test/plugins/lefthook-v1.test.ts

@@ -1,3 +1,4 @@
+import os from 'node:os';
 import { test } from 'bun:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs/promises';
@@ -6,9 +7,11 @@ import { join, resolve } from '../../src/util/path.js';
 import baseArguments from '../helpers/baseArguments.js';
 import baseCounters from '../helpers/baseCounters.js';
 
+const skipIf = typeof Bun !== 'undefined' && os.platform() === 'win32' ? test.skip : test;
+
 const cwd = resolve('fixtures/plugins/lefthook-v1');
 
-test('Find dependencies with the lefthook v1 plugin', async () => {
+skipIf('Find dependencies with the lefthook v1 plugin', async () => {
   const CI = process.env.CI;
   process.env.CI = '';
   await fs.rename(join(cwd, '_git'), join(cwd, '.git')); // Can't add .git folder to repo

packages/knip/test/self-reference-from-plugin.test.ts

@@ -5,9 +5,11 @@ import { resolve } from '../src/util/path.js';
 import baseArguments from './helpers/baseArguments.js';
 import baseCounters from './helpers/baseCounters.js';
 
+const skipIf = typeof Bun !== 'undefined' ? test.skip : test;
+
 const cwd = resolve('fixtures/self-reference-from-plugin');
 
-test.skip('Allows self-references from plugin', async () => {
+skipIf('Allows self-references from plugin', async () => {
   const { counters } = await main({
     ...baseArguments,
     cwd,

packages/knip/test/workspaces-self-reference.test.ts

@@ -5,7 +5,9 @@ import { resolve } from '../src/util/path.js';
 import baseArguments from './helpers/baseArguments.js';
 import baseCounters from './helpers/baseCounters.js';
 
-test.skip('Find unused files, dependencies and exports in workspaces with cross self-references', async () => {
+const skipIf = typeof Bun !== 'undefined' ? test.skip : test;
+
+skipIf('Find unused files, dependencies and exports in workspaces with cross self-references', async () => {
   const cwd = resolve('fixtures/workspaces-self-reference');
 
   const { counters } = await main({

packages/knip/transform-test.js

@@ -0,0 +1,13 @@
+import { readFile } from 'node:fs/promises';
+import module from 'node:module';
+
+module.register('./transform-test.js', { parentURL: import.meta.url });
+
+export async function load(url, context, next) {
+  if (url.endsWith('.test.ts')) {
+    const content = await readFile(new URL(url), 'utf8');
+    const source = content.replace(/import { test } from 'bun:test';/, `import test from 'node:test';`);
+    return next(url, { ...context, source });
+  }
+  return next(url, context);
+}