Bump to use node 20 (#4)

* bump to use node 20

* use fetch pool forks

to prevent leaving processes running in the background, see https://vitest.dev/guide/common-errors.html#failed-to-terminate-worker

.github/workflows/merge-request.yml

@@ -15,7 +15,7 @@ jobs:
       - name: Setup Node
         uses: actions/setup-node@v3
         with:
-          node-version: 18
+          node-version: 20
       - name: Install and build
         run: npm ci && npm run build
       - name: Test

.github/workflows/publish.yml

@@ -13,7 +13,7 @@ jobs:
       - name: Setup Node
         uses: actions/setup-node@v3
         with:
-          node-version: 18
+          node-version: 20
           registry-url: https://registry.npmjs.org/
       - name: Install and build
         run: npm ci && npm run build

lib/tests/hibp.test.ts

@@ -27,16 +27,19 @@ describe("Have I Been Pwned", () => {
 	test("non-node web crypto API hash", async () => {
 		const cryptoSafeBox = globalThis.crypto;
 
-		globalThis.crypto = {
-			getRandomValues: vi
-				.fn()
-				.mockImplementation(async () => new ArrayBuffer(16)),
-			randomUUID: vi.fn() as MockedFunction<typeof cryptoSafeBox.randomUUID>,
-			subtle: {
-				...cryptoSafeBox?.subtle,
-				digest: vi.fn().mockImplementation(async () => new ArrayBuffer(16)),
+		Object.defineProperty(globalThis, "crypto", {
+			value: {
+				getRandomValues: vi
+					.fn()
+					.mockImplementation(async () => new ArrayBuffer(16)),
+				randomUUID: vi.fn() as MockedFunction<typeof cryptoSafeBox.randomUUID>,
+				subtle: {
+					...cryptoSafeBox?.subtle,
+					digest: vi.fn().mockImplementation(async () => new ArrayBuffer(16)),
+				},
 			},
-		};
+			writable: true,
+		});
 
 		await createSha1Hash("123934534f", {} as NodeJS.Process);
 

vite.config.ts

@@ -10,5 +10,8 @@ module.exports = defineConfig({
       fileName: (format) => `password-tester.${format}.js`
     }
   },
-  plugins: [dts()]
+  plugins: [dts()],
+  test: {
+    pool: 'forks',
+  },
 })