test: convert tests to vitest (#1440)

* test: convert tests to vitest

* remove karma

* test: fixing unit tests in vitest

* fetch browser unit tests

* fix node http tests

* formatting

* convert nock to vitest mock in imds

* fix fetch browser tests

* fix empty test in readablestream

* use jsdom types temporarily

* skip test on node16

* formatting

Author: kuhe

.changeset/sweet-lions-design.md

@@ -0,0 +1,8 @@
+---
+"@smithy/fetch-http-handler": minor
+"@smithy/node-http-handler": minor
+"@smithy/util-stream": minor
+"@smithy/types": minor
+---
+
+vitest compatibility

.changeset/violet-ghosts-complain.md

@@ -0,0 +1,2 @@
+---
+---

Makefile

@@ -11,10 +11,15 @@ generate-protocol-tests:
 	./gradlew :smithy-typescript-protocol-test-codegen:build
 	rm -rf ./private/smithy-rpcv2-cbor
 	cp -r ./smithy-typescript-protocol-test-codegen/build/smithyprojections/smithy-typescript-protocol-test-codegen/smithy-rpcv2-cbor/typescript-codegen ./private/smithy-rpcv2-cbor
-	cp ./packages/core/jest.config.js ./private/smithy-rpcv2-cbor
+	cp ./packages/core/vitest.config.ts ./private/smithy-rpcv2-cbor/vitest.config.js
 	node ./scripts/post-protocol-test-codegen
 	npx prettier --write ./private/smithy-rpcv2-cbor
 	yarn
 
 test-protocols:
-	(cd ./private/smithy-rpcv2-cbor && npx jest)
+	(cd ./private/smithy-rpcv2-cbor && npx vitest run --globals)
+
+turbo-clean:
+	@read -p "Are you sure you want to delete your local cache? [y/N]: " ans && [ $${ans:-N} = y ]
+	@echo "\nDeleted cache folders: \n--------"
+	@find . -name '.turbo' -type d -prune -print -exec rm -rf '{}' + && echo '\n'

jest.config.js

@@ -20,9 +20,8 @@
     "extract:docs": "mkdir -p api-extractor-packages && turbo run extract:docs",
     "release": "yarn changeset publish",
     "build-test-packages": "./gradlew clean build && node ./scripts/build-generated-test-packages",
-    "g:jest": "cd $INIT_CWD && jest",
-    "g:karma": "cd $INIT_CWD && karma",
-    "g:tsc": "cd $INIT_CWD && tsc"
+    "g:tsc": "cd $INIT_CWD && tsc",
+    "g:vitest": "cd $INIT_CWD && vitest"
   },
   "repository": {
     "type": "git",
@@ -39,29 +38,33 @@
     "@microsoft/api-extractor": "7.34.4",
     "@tsconfig/recommended": "1.0.2",
     "@types/jest": "28.1.3",
+    "@types/jsdom": "20.0.1",
     "@typescript-eslint/eslint-plugin": "7.8.0",
     "@typescript-eslint/parser": "7.8.0",
     "esbuild": "0.19.11",
     "eslint": "8.57.0",
     "eslint-plugin-simple-import-sort": "7.0.0",
     "eslint-plugin-tsdoc": "0.2.17",
+    "get-port": "^7.1.0",
+    "happy-dom": "14.12.3",
     "husky": "^4.2.3",
     "jest": "29.7.0",
-    "jest-environment-jsdom": "29.7.0",
-    "karma": "6.4.3",
-    "karma-chrome-launcher": "3.2.0",
-    "karma-firefox-launcher": "2.1.3",
-    "karma-jasmine": "5.1.0",
-    "karma-typescript": "5.5.4",
-    "karma-webpack": "5.0.1",
     "prettier": "3.2.5",
     "puppeteer": "^19.2.0",
     "ts-jest": "29.1.2",
     "turbo": "latest",
     "typescript": "~5.2.2",
+    "vite": "4.5.5",
+    "vitest": "0.34.6",
     "webpack": "5.91.0",
     "yarn": "1.22.22"
   },
+  "overrides": {
+    "vite": "4.5.5"
+  },
+  "resolutions": {
+    "vite": "4.5.5"
+  },
   "workspaces": [
     "packages/*",
     "smithy-typescript-ssdk-libs/*",

package.json

@@ -16,7 +16,8 @@
     "lint": "eslint -c ../../.eslintrc.js \"src/**/*.ts\"",
     "format": "prettier --config ../../prettier.config.js --ignore-path ../.prettierignore --write \"**/*.{ts,md,json}\"",
     "extract:docs": "api-extractor run --local",
-    "test": "yarn g:jest"
+    "test": "yarn g:vitest run",
+    "test:watch": "yarn g:vitest watch"
   },
   "author": {
     "name": "AWS SDK for JavaScript Team",

packages/abort-controller/jest.config.js

@@ -1,3 +1,5 @@
+import { describe, expect, test as it } from "vitest";
+
 import { AbortController } from "./AbortController";
 import { AbortSignal } from "./AbortSignal";
 

packages/abort-controller/package.json

@@ -1,3 +1,5 @@
+import { describe, expect, test as it, vi } from "vitest";
+
 import { AbortController } from "./AbortController";
 
 describe("AbortSignal", () => {
@@ -13,7 +15,7 @@ describe("AbortSignal", () => {
   it("should invoke the onabort handler when the signal is aborted", () => {
     const controller = new AbortController();
     const { signal } = controller;
-    const abortHandler = jest.fn();
+    const abortHandler = vi.fn();
     signal.onabort = abortHandler;
     expect(abortHandler.mock.calls.length).toBe(0);
     controller.abort();
@@ -23,7 +25,7 @@ describe("AbortSignal", () => {
   it("should not invoke the onabort handler multiple time", () => {
     const controller = new AbortController();
     const { signal } = controller;
-    const abortHandler = jest.fn();
+    const abortHandler = vi.fn();
     signal.onabort = abortHandler;
     expect(abortHandler.mock.calls.length).toBe(0);
     controller.abort();

packages/abort-controller/src/AbortController.spec.ts

@@ -0,0 +1,9 @@
+import { defineConfig } from "vitest/config";
+
+export default defineConfig({
+  test: {
+    exclude: ["**/*.{integ,e2e,browser}.spec.ts"],
+    include: ["**/*.spec.ts"],
+    environment: "node",
+  },
+});