Switching to vitest

.github/workflows/node.js.yml

@@ -48,8 +48,8 @@ jobs:
         parallel: true
     - name: Build
       run: yarn build
-    - name: Vitest compatibility
-      run: yarn test:vi
+    - name: Compatibility test
+      run: yarn test:compat
     - name: Integration test
       run: yarn test:int
     - name: Issue 952

package.json

@@ -26,14 +26,14 @@
     "build:client": "yarn tsnode example/generate-client.ts && yarn prettier example/example.client.ts --write",
     "build:license": "yarn tsnode tools/license.ts",
     "build:logo": "yarn tsnode tools/startup-logo.ts",
-    "test": "yarn test:types && yarn test:jest && yarn test:badge",
-    "test:int": "yarn --cwd tests/integration && jest tests/integration",
+    "test": "yarn test:types && yarn test:unit && yarn test:badge",
+    "test:int": "yarn --cwd tests/integration && vitest run tests/integration",
     "test:952": "yarn --cwd tests/issue952 && yarn --cwd tests/issue952 test",
-    "test:esm": "yarn --cwd tests/esm && jest tests/esm",
-    "test:vi": "yarn --cwd tests/vitest && yarn --cwd tests/vitest test",
-    "test:u": "yarn test:jest -u && yarn test:badge",
+    "test:esm": "yarn --cwd tests/esm && vitest run tests/esm",
+    "test:compat": "yarn --cwd tests/compat && yarn --cwd tests/compat test",
+    "test:u": "yarn test:unit -u && yarn test:badge",
     "test:types": "tsc --noEmit",
-    "test:jest": "jest tests/unit tests/system",
+    "test:unit": "vitest run tests/unit tests/system",
     "test:badge": "make-coverage-badge --output-path coverage.svg",
     "lint": "eslint src example tests tools && yarn prettier *.md --check",
     "mdfix": "prettier *.md --write",
@@ -126,20 +126,19 @@
     "@arethetypeswrong/cli": "^0.13.0",
     "@swc-node/register": "^1.6.8",
     "@swc/core": "^1.3.92",
-    "@swc/jest": "^0.2.29",
     "@tsconfig/node18": "^18.2.1",
     "@types/compression": "^1.7.5",
     "@types/cors": "^2.8.14",
     "@types/express": "^4.17.17",
     "@types/express-fileupload": "^1.4.4",
     "@types/has-ansi": "^5.0.0",
     "@types/http-errors": "^2.0.2",
-    "@types/jest": "^29.5.4",
     "@types/node": "^20.8.4",
     "@types/ramda": "^0.29.3",
     "@types/triple-beam": "^1.3.2",
     "@typescript-eslint/eslint-plugin": "^6.6.0",
     "@typescript-eslint/parser": "^6.6.0",
+    "@vitest/coverage-istanbul": "^1.0.4",
     "chalk": "^4.1.2",
     "compression": "^1.7.4",
     "cors": "^2.8.5",
@@ -156,14 +155,14 @@
     "has-ansi": "^4.0.1",
     "http-errors": "^2.0.0",
     "husky": "^8.0.3",
-    "jest": "^29.6.4",
     "make-coverage-badge": "^1.2.0",
     "mockdate": "^3.0.5",
     "prettier": "3.1.0",
     "strip-ansi": "^6.0.1",
     "tsd": "^0.29.0",
     "tsup": "^8.0.0",
     "typescript": "^5.2.2",
+    "vitest": "^1.0.4",
     "winston": "^3.10.0",
     "zod": "^3.22.3"
   },

tests/compat/jest.config.json

@@ -0,0 +1,7 @@
+{
+  "transform": {
+    "^.+\\.ts$": "@swc/jest"
+  },
+  "testEnvironment": "node",
+  "verbose": true
+}

tests/compat/jest.spec.ts

@@ -0,0 +1,32 @@
+import { defaultEndpointsFactory, testEndpoint } from "express-zod-api";
+import { z } from "zod";
+
+declare module "express-zod-api" {
+  interface MockOverrides extends jest.Mock {}
+}
+
+describe("Jest compatibility test", () => {
+  describe("testEndpoint()", () => {
+    test.each([undefined, jest.fn])(
+      "should support jest.fn %#",
+      async (fnMethod) => {
+        const endpoint = defaultEndpointsFactory.build({
+          method: "post",
+          input: z.object({ n: z.number() }),
+          output: z.object({ inc: z.number() }),
+          handler: async ({ input }) => ({ inc: input.n + 1 }),
+        });
+        const { responseMock } = await testEndpoint({
+          endpoint,
+          requestProps: { method: "POST", body: { n: 123 } },
+          fnMethod,
+        });
+        expect(responseMock.status).toHaveBeenCalledWith(200);
+        expect(responseMock.json).toHaveBeenCalledWith({
+          status: "success",
+          data: { inc: 124 },
+        });
+      },
+    );
+  });
+});

tests/vitest/package.json → tests/compat/package.json

@@ -3,7 +3,7 @@
   "private": true,
   "version": "0.0.0",
   "scripts": {
-    "test": "vitest run"
+    "test": "jest"
   },
   "dependencies": {
     "express-zod-api": "link:../..",
@@ -13,6 +13,9 @@
     "zod": "^3"
   },
   "devDependencies": {
-    "vitest": "^1.0.0-beta.4"
+    "jest": "^29.7.0",
+    "@types/jest": "^29.5.11",
+    "@swc/core": "^1.3.100",
+    "@swc/jest": "^0.2.29"
   }
 }

tests/esm/quick-start.spec.ts

@@ -1,5 +1,6 @@
 import { ChildProcessWithoutNullStreams, spawn } from "node:child_process";
 import { givePort, waitFor } from "../helpers";
+import { afterAll, afterEach, beforeAll, describe, expect, test } from "vitest";
 
 describe("ESM Test", () => {
   let quickStart: ChildProcessWithoutNullStreams;

tests/express-mock.ts

@@ -1,30 +1,32 @@
-// @see https://github.com/swc-project/jest/issues/14#issuecomment-970189585
+// @see https://github.com/swc-project/vi/issues/14#issuecomment-970189585
+import { Mock, vi } from "vitest";
 
-const expressJsonMock = jest.fn();
-const compressionMock = jest.fn();
-const fileUploadMock = jest.fn();
-jest.mock("compression", () => compressionMock);
-jest.mock("express-fileupload", () => fileUploadMock);
+const expressJsonMock = vi.fn();
+const compressionMock = vi.fn();
+const fileUploadMock = vi.fn();
 
-const staticHandler = jest.fn();
-const staticMock = jest.fn(() => staticHandler);
+vi.mock("compression", () => ({ default: compressionMock }));
+vi.mock("express-fileupload", () => ({ default: fileUploadMock }));
 
-let appMock: Record<"disable" | "use" | "get" | "post" | "options", jest.Mock>;
+const staticHandler = vi.fn();
+const staticMock = vi.fn(() => staticHandler);
 
-const appCreatorMock = () => {
+let appMock: Record<"disable" | "use" | "get" | "post" | "options", Mock>;
+
+const expressMock = () => {
   appMock = {
-    disable: jest.fn(() => appMock),
-    use: jest.fn(() => appMock),
-    get: jest.fn(),
-    post: jest.fn(),
-    options: jest.fn(),
+    disable: vi.fn(() => appMock),
+    use: vi.fn(() => appMock),
+    get: vi.fn(),
+    post: vi.fn(),
+    options: vi.fn(),
   };
   return appMock;
 };
-appCreatorMock.json = () => expressJsonMock;
-appCreatorMock.static = staticMock;
+expressMock.json = () => expressJsonMock;
+expressMock.static = staticMock;
 
-const expressMock = jest.mock("express", () => appCreatorMock);
+vi.mock("express", () => ({ default: expressMock }));
 
 export {
   compressionMock,

tests/helpers.ts

@@ -1,4 +1,3 @@
-import jestConfig from "../jest.config.json";
 import { z } from "zod";
 import { SchemaHandler, walkSchema } from "../src/schema-walker";
 
@@ -22,7 +21,7 @@ export const waitFor = async (cb: () => boolean) =>
     const timeout = setTimeout(() => {
       clearInterval(timer); // eslint-disable-line @typescript-eslint/no-use-before-define
       reject();
-    }, jestConfig.testTimeout);
+    }, 10000);
     const timer = setInterval(() => {
       if (cb()) {
         clearInterval(timer);

tests/http-mock.ts

@@ -1,31 +1,30 @@
 import http from "node:http";
 import https from "node:https";
+import { MockInstance, vi } from "vitest";
 
 const realHttpCreator = http.createServer;
 const realHttpsCreator = https.createServer;
 
-let httpListenSpy: jest.SpyInstance;
-let httpsListenSpy: jest.SpyInstance;
+let httpListenSpy: MockInstance;
+let httpsListenSpy: MockInstance;
 
-jest.spyOn(http, "createServer").mockImplementation((app) => {
+vi.spyOn(http, "createServer").mockImplementation((app) => {
   const server = realHttpCreator(app);
-  httpListenSpy = jest.spyOn(server, "listen").mockImplementation(({}, cb) => {
+  httpListenSpy = vi.spyOn(server, "listen").mockImplementation(({}, cb) => {
     cb?.call(null);
     return server;
   });
   return server;
 });
 
-const createHttpsServerSpy = jest
+const createHttpsServerSpy = vi
   .spyOn(https, "createServer")
   .mockImplementation(({}, app) => {
     const server = realHttpsCreator(app);
-    httpsListenSpy = jest
-      .spyOn(server, "listen")
-      .mockImplementation(({}, cb) => {
-        cb?.call(null);
-        return server;
-      });
+    httpsListenSpy = vi.spyOn(server, "listen").mockImplementation(({}, cb) => {
+      cb?.call(null);
+      return server;
+    });
     return server;
   });
 

tests/integration/quick-start.spec.ts

@@ -1,5 +1,6 @@
 import { ChildProcessWithoutNullStreams, spawn } from "node:child_process";
 import { givePort, waitFor } from "../helpers";
+import { afterAll, afterEach, beforeAll, describe, expect, test } from "vitest";
 
 describe("Integration Test", () => {
   let quickStart: ChildProcessWithoutNullStreams;

tests/system/__snapshots__/example.spec.ts.snap

@@ -1,6 +1,6 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
-exports[`Example Client Should perform the request with a positive response 1`] = `
+exports[`Example > Client > Should perform the request with a positive response 1`] = `
 {
   "data": {
     "features": [
@@ -39,7 +39,7 @@ exports[`Example Client Should perform the request with a positive response 1`]
 }
 `;
 
-exports[`Example Negative GET request should fail on missing input param 1`] = `
+exports[`Example > Negative > GET request should fail on missing input param 1`] = `
 {
   "error": {
     "message": "id: Required",
@@ -48,7 +48,7 @@ exports[`Example Negative GET request should fail on missing input param 1`] = `
 }
 `;
 
-exports[`Example Negative POST request should fail on schema validation 1`] = `
+exports[`Example > Negative > POST request should fail on schema validation 1`] = `
 {
   "error": {
     "message": "id: should be greater than or equal to 0",
@@ -57,7 +57,7 @@ exports[`Example Negative POST request should fail on schema validation 1`] = `
 }
 `;
 
-exports[`Example Negative Should respond with error to the missing static file request 1`] = `
+exports[`Example > Negative > Should respond with error to the missing static file request 1`] = `
 {
   "error": {
     "message": "Can not GET /public/missing.svg",
@@ -66,7 +66,7 @@ exports[`Example Negative Should respond with error to the missing static file r
 }
 `;
 
-exports[`Example Positive Should accept raw data 0 1`] = `
+exports[`Example > Positive > Should accept raw data 0 1`] = `
 {
   "data": {
     "length": 48687,
@@ -75,7 +75,7 @@ exports[`Example Positive Should accept raw data 0 1`] = `
 }
 `;
 
-exports[`Example Positive Should accept raw data 1 1`] = `
+exports[`Example > Positive > Should accept raw data 1 1`] = `
 {
   "data": {
     "length": 48687,
@@ -84,13 +84,13 @@ exports[`Example Positive Should accept raw data 1 1`] = `
 }
 `;
 
-exports[`Example Positive Should send an image with a correct header 1`] = `"f39beeff92379dc935586d726211c2620be6f879"`;
+exports[`Example > Positive > Should send an image with a correct header 1`] = `"f39beeff92379dc935586d726211c2620be6f879"`;
 
-exports[`Example Positive Should serve static files 1`] = `"f39beeff92379dc935586d726211c2620be6f879"`;
+exports[`Example > Positive > Should serve static files 1`] = `"f39beeff92379dc935586d726211c2620be6f879"`;
 
-exports[`Example Positive Should stream an image with a correct header 1`] = `"f39beeff92379dc935586d726211c2620be6f879"`;
+exports[`Example > Positive > Should stream an image with a correct header 1`] = `"f39beeff92379dc935586d726211c2620be6f879"`;
 
-exports[`Example Positive Should upload the file 1`] = `
+exports[`Example > Positive > Should upload the file 1`] = `
 {
   "data": {
     "hash": "f39beeff92379dc935586d726211c2620be6f879",