chore(repo): test migration to vitest. phase 3 (#1977)

* test(auto-install,pluginutils,run): migrate phase 3 packages to vitest

* chore: remove redundant vitest include glob

---------

Co-authored-by: CharlieHelps <charlie@charlielabs.ai>

Author: charliecreates[bot]

.config/vitest.config.mts

@@ -1,13 +1,15 @@
 import { defineConfig } from 'vitest/config';
 import path from 'node:path';
 
+const isAutoInstallPackage = path.basename(process.cwd()) === 'auto-install';
+
 export default defineConfig({
   test: {
     // Enable global APIs for CommonJS test files.
     globals: true,
-    // Phase 1/2 packages use runtime-style, *.test, and a few named entrypoints.
+    // Phase 1/2/3 packages use runtime-style, top-level test files, *.test, and a few named entrypoints.
     include: [
-      'test/test.{js,mjs,cjs,ts,mts,cts}',
+      'test/*.{js,mjs,cjs,ts,mts,cts}',
       'test/**/*.{test,spec}.{js,mjs,cjs,ts,mts,cts}',
       'test/{as-input-plugin,as-output-plugin,form,function,misc,sourcemaps}.{js,mjs,cjs,ts,mts,cts}'
     ],
@@ -20,6 +22,8 @@ export default defineConfig({
       '**/test/snapshots/**',
       '**/test/types.ts'
     ],
+    // These tests switch process cwd and invoke package managers; run files serially there.
+    fileParallelism: !isAutoInstallPackage,
     // Keep snapshots in the same location used by Ava.
     resolveSnapshotPath: (testPath, snapExt) =>
       path.join(path.dirname(testPath), 'snapshots', path.basename(testPath) + snapExt)

packages/auto-install/package.json

@@ -28,13 +28,13 @@
     "ci:coverage": "nyc pnpm test && nyc report --reporter=text-lcov > coverage.lcov",
     "ci:lint": "pnpm build && pnpm lint",
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
-    "ci:test": "pnpm test -- --verbose",
+    "ci:test": "pnpm test -- --reporter=verbose",
     "prebuild": "del-cli dist",
     "prepare": "if [ ! -d 'dist' ]; then pnpm build; fi",
     "prerelease": "pnpm build",
     "pretest": "pnpm build",
     "release": "pnpm --workspace-root package:release $(pwd)",
-    "test": "ava",
+    "test": "vitest --config ../../.config/vitest.config.mts run",
     "test:ts": "tsc --noEmit"
   },
   "files": [
@@ -69,15 +69,5 @@
     "rollup": "^4.0.0-24",
     "typescript": "^4.8.3"
   },
-  "types": "./types/index.d.ts",
-  "ava": {
-    "workerThreads": false,
-    "files": [
-      "!**/fixtures/**",
-      "!**/output/**",
-      "!**/helpers/**",
-      "!**/recipes/**",
-      "!**/types.ts"
-    ]
-  }
+  "types": "./types/index.d.ts"
 }

packages/auto-install/test/npm-bare.js

@@ -1,7 +1,6 @@
 const { readFileSync } = require('fs');
 const { join } = require('path');
 
-const test = require('ava');
 const del = require('del');
 const { nodeResolve } = require('@rollup/plugin-node-resolve');
 const { rollup } = require('rollup');
@@ -14,8 +13,7 @@ const input = join(cwd, '../input.js');
 
 process.chdir(cwd);
 
-test('npm, bare', async (t) => {
-  t.timeout(50000);
+test('npm, bare', async () => {
   await rollup({
     input,
     output: {
@@ -24,10 +22,10 @@ test('npm, bare', async (t) => {
     },
     plugins: [autoInstall(), nodeResolve()]
   });
-  t.snapshot(readFileSync('package.json', 'utf-8'));
-  t.truthy(readFileSync('package-lock.json', 'utf-8').includes('"node-noop"'));
-});
+  expect(readFileSync('package.json', 'utf-8')).toMatchSnapshot();
+  expect(readFileSync('package-lock.json', 'utf-8').includes('"node-noop"')).toBeTruthy();
+}, 50000);
 
-test.after(async () => {
+afterAll(async () => {
   await del(['node_modules', 'package.json', 'package-lock.json']);
 });

packages/auto-install/test/npm.js

@@ -1,7 +1,6 @@
 const { readFileSync, writeFileSync } = require('fs');
 const { join } = require('path');
 
-const test = require('ava');
 const del = require('del');
 const { nodeResolve } = require('@rollup/plugin-node-resolve');
 const { rollup } = require('rollup');
@@ -16,27 +15,29 @@ const pkgFile = join(cwd, 'package.json');
 
 process.chdir(cwd);
 
-test('invalid manager', (t) => {
-  t.timeout(50000);
-  const error = t.throws(
-    () =>
+test('invalid manager', () => {
+  const error = (() => {
+    try {
       rollup({
         input,
         output: {
           file,
           format: 'cjs'
         },
         plugins: [autoInstall({ pkgFile, manager: 'foo' }), nodeResolve()]
-      }),
-    {
-      instanceOf: RangeError
+      });
+    } catch (caught) {
+      return caught;
     }
-  );
-  t.snapshot(error.message);
-});
 
-test('npm', async (t) => {
-  t.timeout(50000);
+    return null;
+  })();
+
+  expect(error).toBeInstanceOf(RangeError);
+  expect(error.message).toMatchSnapshot();
+}, 50000);
+
+test('npm', async () => {
   await rollup({
     input,
     output: {
@@ -45,10 +46,10 @@ test('npm', async (t) => {
     },
     plugins: [autoInstall({ pkgFile, manager }), nodeResolve()]
   });
-  t.snapshot(readFileSync('package.json', 'utf-8'));
-});
+  expect(readFileSync('package.json', 'utf-8')).toMatchSnapshot();
+}, 50000);
 
-test.after(async () => {
+afterAll(async () => {
   await del(['node_modules', 'package-lock.json']);
   writeFileSync(pkgFile, '{}');
 });

packages/auto-install/test/pnpm-bare.js

@@ -1,7 +1,6 @@
 const { readFileSync } = require('fs');
 const { join } = require('path');
 
-const test = require('ava');
 const del = require('del');
 const { nodeResolve } = require('@rollup/plugin-node-resolve');
 const { rollup } = require('rollup');
@@ -14,8 +13,7 @@ const input = join(cwd, '../input.js');
 
 process.chdir(cwd);
 
-test('pnpm, bare', async (t) => {
-  t.timeout(50000);
+test('pnpm, bare', async () => {
   await rollup({
     input,
     output: {
@@ -26,9 +24,9 @@ test('pnpm, bare', async (t) => {
   });
   const json = JSON.parse(readFileSync('package.json', 'utf-8'));
   // snapshots for this are a nightmare cross-platform
-  t.truthy('node-noop' in json.dependencies);
-});
+  expect('node-noop' in json.dependencies).toBeTruthy();
+}, 50000);
 
-test.after(async () => {
+afterAll(async () => {
   await del(['node_modules', 'package.json', 'pnpm-lock.yaml']);
 });

packages/auto-install/test/pnpm.js

@@ -1,7 +1,6 @@
 const { readFileSync, writeFileSync } = require('fs');
 const { join } = require('path');
 
-const test = require('ava');
 const del = require('del');
 const { nodeResolve } = require('@rollup/plugin-node-resolve');
 const { rollup } = require('rollup');
@@ -14,8 +13,7 @@ const input = join(cwd, '../input.js');
 
 process.chdir(cwd);
 
-test('pnpm', async (t) => {
-  t.timeout(50000);
+test('pnpm', async () => {
   await rollup({
     input,
     output: {
@@ -27,10 +25,10 @@ test('pnpm', async (t) => {
 
   const json = JSON.parse(readFileSync('package.json', 'utf-8'));
   // snapshots for this are a nightmare cross-platform
-  t.truthy('node-noop' in json.dependencies);
-});
+  expect('node-noop' in json.dependencies).toBeTruthy();
+}, 50000);
 
-test.after(async () => {
+afterAll(async () => {
   await del(['node_modules', 'package.json']);
   writeFileSync('pnpm-lock.yaml', '');
 });

packages/auto-install/test/snapshots/npm-bare.js.snap

@@ -1,7 +1,6 @@
 const { readFileSync } = require('fs');
 const { join } = require('path');
 
-const test = require('ava');
 const del = require('del');
 const { nodeResolve } = require('@rollup/plugin-node-resolve');
 const { rollup } = require('rollup');
@@ -14,8 +13,7 @@ const input = join(cwd, '../input.js');
 
 process.chdir(cwd);
 
-test('yarn, bare', async (t) => {
-  t.timeout(50000);
+test('yarn, bare', async () => {
   await rollup({
     input,
     output: {
@@ -30,9 +28,9 @@ test('yarn, bare', async (t) => {
   });
   const lockFile = readFileSync('yarn.lock', 'utf-8');
   // snapshots for this are a nightmare cross-platform
-  t.truthy(/yarn\.bare\s+node-noop/.test(lockFile));
-});
+  expect(/yarn\.bare\s+node-noop/.test(lockFile)).toBeTruthy();
+}, 50000);
 
-test.after(async () => {
+afterAll(async () => {
   await del(['node_modules', 'package.json', 'yarn.lock']);
 });

packages/auto-install/test/snapshots/npm.js.snap

@@ -1,7 +1,6 @@
 const { readFileSync, writeFileSync } = require('fs');
 const { join } = require('path');
 
-const test = require('ava');
 const del = require('del');
 const { nodeResolve } = require('@rollup/plugin-node-resolve');
 const { rollup } = require('rollup');
@@ -14,8 +13,7 @@ const input = join(cwd, '../input.js');
 
 process.chdir(cwd);
 
-test('yarn', async (t) => {
-  t.timeout(50000);
+test('yarn', async () => {
   await rollup({
     input,
     output: {
@@ -27,10 +25,10 @@ test('yarn', async (t) => {
   });
   const lockFile = readFileSync('yarn.lock', 'utf-8');
   // snapshots for this are a nightmare cross-platform
-  t.truthy(/yarn\s+node-noop/.test(lockFile));
-});
+  expect(/yarn\s+node-noop/.test(lockFile)).toBeTruthy();
+}, 50000);
 
-test.after(async () => {
+afterAll(async () => {
   await del(['node_modules', 'package.json']);
   writeFileSync('yarn.lock', '');
 });