refactor: migrate from jest to vitest (#812)

Author: satya164

eslint.config.mjs

@@ -1,10 +1,35 @@
 // eslint-disable-next-line import-x/extensions,import-x/no-unresolved
 import { defineConfig, globalIgnores } from 'eslint/config';
 import satya164 from 'eslint-config-satya164';
+import vitest from '@vitest/eslint-plugin';
 
 export default defineConfig([
   satya164,
 
+  {
+    files: ['**/*.{spec,test}.{js,ts,tsx}', '**/__tests__/**/*.{js,ts,tsx}'],
+
+    plugins: {
+      vitest,
+    },
+
+    rules: {
+      ...vitest.configs.recommended.rules,
+
+      'vitest/consistent-test-it': ['error', { fn: 'test' }],
+      'vitest/expect-expect': 'error',
+      'vitest/no-disabled-tests': 'error',
+      'vitest/no-duplicate-hooks': 'error',
+      'vitest/no-test-prefixes': 'error',
+      'vitest/no-test-return-statement': 'error',
+      'vitest/prefer-to-be': 'error',
+      'vitest/prefer-todo': 'error',
+      'vitest/require-to-throw-message': 'error',
+
+      'jest/no-deprecated-functions': 'off',
+    },
+  },
+
   globalIgnores([
     '**/.next/',
     '**/.expo/',

package.json

@@ -26,6 +26,7 @@
     "@evilmartians/lefthook": "^1.5.0",
     "@lerna-lite/cli": "^1.13.0",
     "@lerna-lite/run": "^1.13.0",
+    "@vitest/eslint-plugin": "^1.1.39",
     "commitlint": "^17.0.2",
     "concurrently": "^7.2.2",
     "eslint": "^9.23.0",

packages/react-native-builder-bob/package.json

@@ -37,12 +37,7 @@
   },
   "scripts": {
     "prepare": "babel --extensions .ts,.tsx src --out-dir lib --source-maps --delete-dir-on-start",
-    "test": "jest"
-  },
-  "jest": {
-    "testPathIgnorePatterns": [
-      "/lib/"
-    ]
+    "test": "vitest"
   },
   "dependencies": {
     "@babel/core": "^7.25.2",
@@ -71,7 +66,6 @@
   },
   "devDependencies": {
     "@babel/cli": "^7.24.8",
-    "@jest/globals": "^29.7.0",
     "@types/babel__core": "^7.1.19",
     "@types/browserslist": "^4.15.0",
     "@types/cross-spawn": "^6.0.2",
@@ -85,8 +79,8 @@
     "@types/which": "^2.0.1",
     "@types/yargs": "^17.0.10",
     "concurrently": "^7.2.2",
-    "jest": "^29.7.0",
     "mock-fs": "^5.2.0",
-    "mock-stdin": "^1.0.0"
+    "mock-stdin": "^1.0.0",
+    "vitest": "^3.1.1"
   }
 }

packages/react-native-builder-bob/src/__tests__/__snapshots__/init.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
 exports[`initializes the configuration 1`] = `
 "{

packages/react-native-builder-bob/src/__tests__/babel.test.ts

@@ -1,7 +1,8 @@
-import { expect, test } from '@jest/globals';
+import { expect, test } from 'vitest';
 import { transformFileAsync } from '@babel/core';
 import fs from 'node:fs';
 import path from 'node:path';
+import plugin from '../babel';
 
 test.each(['imports', 'exports'])(`adds extension to %s`, async (name) => {
   const filepath = path.resolve(
@@ -21,7 +22,7 @@ test.each(['imports', 'exports'])(`adds extension to %s`, async (name) => {
     babelrc: false,
     plugins: [
       '@babel/plugin-syntax-typescript',
-      [require.resolve('../babel.ts'), { extension: 'mjs' }],
+      [plugin, { extension: 'mjs' }],
     ],
   });
 

packages/react-native-builder-bob/src/__tests__/init.test.ts

@@ -1,13 +1,15 @@
-import { afterEach, beforeEach, expect, test, jest } from '@jest/globals';
+import { afterEach, beforeEach, expect, test, vi, type Mock } from 'vitest';
 import { readFile } from 'fs-extra';
 import mockFs from 'mock-fs';
 import { stdin } from 'mock-stdin';
-import { join } from 'path';
+import { join } from 'node:path';
 import { init } from '../init';
 
-jest.mock('../../package.json', () => ({
-  name: 'react-native-builder-bob',
-  version: '0.5.0',
+vi.mock('../../package.json', () => ({
+  default: {
+    name: 'react-native-builder-bob',
+    version: '0.5.0',
+  },
 }));
 
 let io: ReturnType<typeof stdin> | undefined;
@@ -58,17 +60,17 @@ beforeEach(() => {
 afterEach(() => {
   io?.restore();
   mockFs.restore();
-  jest.restoreAllMocks();
+  vi.restoreAllMocks();
 });
 
 test('initializes the configuration', async () => {
-  jest.spyOn(process.stdout, 'write').mockImplementation(() => true);
+  vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
 
   process.chdir(root);
 
   const run = async () => {
     await waitFor(() => {
-      const lastCall = (process.stdout.write as jest.Mock).mock.lastCall;
+      const lastCall = (process.stdout.write as Mock).mock.lastCall;
 
       if (lastCall == null) {
         throw new Error('No output');

packages/react-native-builder-bob/src/init.ts

@@ -5,9 +5,7 @@ import dedent from 'dedent';
 import isGitDirty from 'is-git-dirty';
 import prompts, { type PromptObject } from './utils/prompts';
 import { loadConfig } from './utils/loadConfig';
-
-// eslint-disable-next-line @typescript-eslint/no-require-imports,import-x/no-commonjs
-const { name, version } = require('../package.json');
+import pack from '../package.json';
 
 const FLOW_PRGAMA_REGEX = /\*?\s*@(flow)\b/m;
 
@@ -37,7 +35,11 @@ export async function init() {
   const pkg = JSON.parse(await fs.readFile(projectPackagePath, 'utf-8'));
   const result = loadConfig(root);
 
-  if (result?.config && pkg.devDependencies && name in pkg.devDependencies) {
+  if (
+    result?.config &&
+    pkg.devDependencies &&
+    pack.name in pkg.devDependencies
+  ) {
     const { shouldContinue } = await prompts({
       type: 'confirm',
       name: 'shouldContinue',
@@ -75,9 +77,10 @@ export async function init() {
   }
 
   pkg.devDependencies = Object.fromEntries(
-    [...Object.entries(pkg.devDependencies || {}), [name, `^${version}`]].sort(
-      ([a], [b]) => a.localeCompare(b)
-    )
+    [
+      ...Object.entries(pkg.devDependencies || {}),
+      [pack.name, `^${pack.version}`],
+    ].sort(([a], [b]) => a.localeCompare(b))
   );
 
   const questions: PromptObject[] = [
@@ -385,7 +388,7 @@ export async function init() {
     pkg.files = files;
   }
 
-  pkg[name] = {
+  pkg[pack.name] = {
     source,
     output,
     targets: targets.map((t: string) => {

packages/react-native-builder-bob/src/targets/codegen/patches/patchCodegenAndroidPackage.test.ts

@@ -1,4 +1,4 @@
-import { expect, test, describe, beforeEach, afterEach } from '@jest/globals';
+import { expect, test, describe, beforeEach, afterEach } from 'vitest';
 import fs from 'fs-extra';
 import path from 'node:path';
 import { patchCodegenAndroidPackage } from './patchCodegenAndroidPackage';

packages/react-native-builder-bob/src/targets/codegen/patches/removeCodegenAppLevelCode.test.ts

@@ -1,4 +1,4 @@
-import { expect, test, describe, beforeEach, afterEach } from '@jest/globals';
+import { expect, describe, beforeEach, afterEach, test } from 'vitest';
 import fs from 'fs-extra';
 import path from 'node:path';
 import { removeCodegenAppLevelCode } from './removeCodegenAppLevelCode';

packages/react-native-builder-bob/vitest.config.mjs

@@ -0,0 +1,7 @@
+import { defineConfig } from 'vitest/config';
+
+export default defineConfig({
+  test: {
+    exclude: ['lib/'],
+  },
+});