fix(data-migrate): Revert "chore(dataMigrate): Switch from jest to vitest (#11978)" (#135)

This reverts commit f16f0dd.

When I switched from jest to vitest I also switched from `require(...)`
to `await import(...)`.
That also meant not running babel anymore, which we actually need for TS
support.

So this PR reverts back to `require` (and jest) to get TS data
migrations working again

Fixes #132

Author: Tobbe

packages/cli-packages/dataMigrate/dist.test.ts

@@ -1,7 +1,5 @@
-import fs from 'node:fs'
-import path from 'node:path'
-
-import { describe, expect, it } from 'vitest'
+import fs from 'fs'
+import path from 'path'
 
 const distPath = path.join(__dirname, 'dist')
 

packages/cli-packages/dataMigrate/package.json

@@ -21,8 +21,8 @@
     "build:types": "tsc --build --verbose",
     "prepublishOnly": "NODE_ENV=production yarn build",
     "test": "yarn test:unit && yarn test:dist",
-    "test:dist": "yarn vitest run ./dist.test.ts",
-    "test:unit": "yarn vitest run src/"
+    "test:dist": "yarn jest ./dist.test.ts",
+    "test:unit": "yarn jest src"
   },
   "dependencies": {
     "@cedarjs/babel-config": "workspace:*",
@@ -40,10 +40,10 @@
     "@prisma/client": "5.20.0",
     "@types/fs-extra": "11.0.4",
     "@types/yargs": "17.0.33",
+    "jest": "29.7.0",
     "memfs": "4.17.2",
     "tsx": "4.19.4",
-    "typescript": "5.6.2",
-    "vitest": "2.1.9"
+    "typescript": "5.6.2"
   },
   "gitHead": "3905ed045508b861b495f8d5630d76c7a157d8f1"
 }

packages/cli-packages/dataMigrate/src/__tests__/install.test.ts

@@ -1,12 +1,15 @@
-import { vi, describe, expect, it } from 'vitest'
-import type { Argv } from 'yargs'
+import type yargs from 'yargs'
 
 import * as installCommand from '../commands/install'
 import { handler as dataMigrateInstallHandler } from '../commands/installHandler.js'
 
-vi.mock('../commands/installHandler.js', () => ({
-  handler: vi.fn(),
-}))
+jest.mock(
+  '../commands/installHandler.js',
+  () => ({
+    handler: jest.fn(),
+  }),
+  { virtual: true },
+)
 
 describe('install', () => {
   it('exports `command`, `description`, `builder`, and `handler`', () => {
@@ -25,7 +28,7 @@ describe('install', () => {
   it('`builder` has an epilogue', () => {
     // The typecasting here is to make TS happy when calling `builder(yargs)`
     // further down. We know that only `epilogue` will be called.
-    const yargs = { epilogue: vi.fn() } as unknown as Argv
+    const yargs = { epilogue: jest.fn() } as unknown as yargs.Argv
 
     installCommand.builder(yargs)
 

packages/cli-packages/dataMigrate/src/__tests__/installHandler.test.ts

@@ -1,6 +1,7 @@
+import fs from 'fs'
+
 import execa from 'execa'
-import { vol, fs as memfs } from 'memfs'
-import { vi, describe, expect, it } from 'vitest'
+import { vol } from 'memfs'
 
 import { getPaths } from '@cedarjs/project-config'
 
@@ -11,18 +12,15 @@ import {
   notes,
 } from '../commands/installHandler'
 
-vi.mock('fs', async () => ({ ...memfs, default: { ...memfs } }))
-vi.mock('node:fs', async () => ({ ...memfs, default: { ...memfs } }))
+jest.mock('fs', () => require('memfs').fs)
 
-vi.mock('execa', () => {
+jest.mock('execa', () => {
   return {
-    default: {
-      command: vi.fn(() => {
-        return {
-          stdout: 42,
-        }
-      }),
-    },
+    command: jest.fn(() => {
+      return {
+        stdout: 42,
+      }
+    }),
   }
 })
 
@@ -60,14 +58,14 @@ describe('installHandler', () => {
       redwoodProjectPath,
     )
 
-    console.log = vi.fn()
+    console.log = jest.fn()
 
     await handler()
 
     const dataMigrationsPath = getPaths().api.dataMigrations
 
-    expect(memfs.readdirSync(dataMigrationsPath)).toEqual(['.keep'])
-    expect(memfs.readFileSync(getPaths().api.dbSchema, 'utf-8')).toMatch(
+    expect(fs.readdirSync(dataMigrationsPath)).toEqual(['.keep'])
+    expect(fs.readFileSync(getPaths().api.dbSchema, 'utf-8')).toMatch(
       RW_DATA_MIGRATION_MODEL,
     )
     expect(execa.command).toHaveBeenCalledWith(createDatabaseMigrationCommand, {

packages/cli-packages/dataMigrate/src/__tests__/up.test.ts

@@ -1,16 +1,19 @@
-import { fs as memfs, vol } from 'memfs'
-import { vi, describe, expect, it } from 'vitest'
-import yargs from 'yargs'
+import { vol } from 'memfs'
+import yargs from 'yargs/yargs'
 
 import { getPaths } from '@cedarjs/project-config'
 
 import * as upCommand from '../commands/up'
 import { handler as dataMigrateUpHandler } from '../commands/upHandler.js'
 
-vi.mock('fs', () => ({ default: memfs }))
-vi.mock('../commands/upHandler.js', () => ({
-  handler: vi.fn(),
-}))
+jest.mock('fs', () => require('memfs').fs)
+jest.mock(
+  '../commands/upHandler.js',
+  () => ({
+    handler: jest.fn(),
+  }),
+  { virtual: true },
+)
 
 describe('up', () => {
   it('exports `command`, `description`, `builder`, and `handler`', () => {
@@ -36,17 +39,14 @@ describe('up', () => {
 
     process.env.RWJS_CWD = '/redwood-app'
 
-    const { argv } = upCommand.builder(yargs())
+    const { argv } = upCommand.builder(yargs)
 
     expect(argv).toHaveProperty('import-db-client-from-dist', false)
     expect(argv).toHaveProperty('dist-path', getPaths().api.dist)
   })
 
   it('`handler` proxies to `./upHandler.js`', async () => {
-    await upCommand.handler({
-      importDbClientFromDist: false,
-      distPath: '',
-    })
+    await upCommand.handler({})
     expect(dataMigrateUpHandler).toHaveBeenCalled()
   })
 })