Replace all the occurrences of the deprecated toThrowError API with the appropriate toThrow one

fixtures/additional-modules/test/index.test.ts

@@ -87,7 +87,7 @@ describe("find_additional_modules dev", () => {
 		await fs.rm(path.join(srcDir, "lang", "en.js"));
 
 		await vi.waitFor(async () => {
-			await expect(get(worker, "/lang/en")).rejects.toThrowError(
+			await expect(get(worker, "/lang/en")).rejects.toThrow(
 				'No such module "lang/en.js".'
 			);
 		});

fixtures/get-platform-proxy/tests/get-platform-proxy.caches.test.ts

@@ -39,31 +39,31 @@ describe("getPlatformProxy - caches", () => {
 
 			expect(() => {
 				caches.has("my-cache");
-			}).toThrowError(
+			}).toThrow(
 				"Failed to execute 'has' on 'CacheStorage': the method is not implemented."
 			);
 
 			expect(() => {
 				caches.delete("my-cache");
-			}).toThrowError(
+			}).toThrow(
 				"Failed to execute 'delete' on 'CacheStorage': the method is not implemented."
 			);
 
 			expect(() => {
 				caches.keys();
-			}).toThrowError(
+			}).toThrow(
 				"Failed to execute 'keys' on 'CacheStorage': the method is not implemented."
 			);
 
 			expect(() => {
 				caches.match(new URL("https://localhost"));
-			}).toThrowError(
+			}).toThrow(
 				"Failed to execute 'match' on 'CacheStorage': the method is not implemented."
 			);
 
 			expect(() => {
 				caches.nonExistentMethod();
-			}).toThrowError("caches.nonExistentMethod is not a function");
+			}).toThrow("caches.nonExistentMethod is not a function");
 		} finally {
 			await dispose();
 		}

fixtures/get-platform-proxy/tests/get-platform-proxy.cf.test.ts

@@ -22,14 +22,14 @@ describe("getPlatformProxy - cf", () => {
 
 			expect(() => {
 				cf.city = "test city";
-			}).toThrowError(
+			}).toThrow(
 				"Cannot assign to read only property 'city' of object '#<Object>'"
 			);
 			expect(cf.city).not.toBe("test city");
 
 			expect(() => {
 				cf.newField = "test new field";
-			}).toThrowError("Cannot add property newField, object is not extensible");
+			}).toThrow("Cannot add property newField, object is not extensible");
 			expect("newField" in cf).toBe(false);
 
 			expect(cf.botManagement).toMatchObject({

fixtures/get-platform-proxy/tests/get-platform-proxy.ctx.test.ts

@@ -67,11 +67,11 @@ describe("getPlatformProxy - ctx", () => {
 			try {
 				expect(() => {
 					waitUntil(new Promise(() => {}));
-				}).toThrowError("Illegal invocation");
+				}).toThrow("Illegal invocation");
 
 				expect(() => {
 					passThroughOnException();
-				}).toThrowError("Illegal invocation");
+				}).toThrow("Illegal invocation");
 			} finally {
 				await dispose();
 			}
@@ -87,11 +87,11 @@ describe("getPlatformProxy - ctx", () => {
 			try {
 				expect(() => {
 					waitUntil(new Promise(() => {}));
-				}).toThrowError("Illegal invocation");
+				}).toThrow("Illegal invocation");
 
 				expect(() => {
 					passThroughOnException();
-				}).toThrowError("Illegal invocation");
+				}).toThrow("Illegal invocation");
 			} finally {
 				await dispose();
 			}
@@ -107,15 +107,15 @@ describe("getPlatformProxy - ctx", () => {
 			try {
 				expect(() => {
 					waitUntil(new Promise(() => {}));
-				}).not.toThrowError("Illegal invocation");
+				}).not.toThrow("Illegal invocation");
 
 				expect(() => {
 					passThroughOnException.apply(ctx, []);
-				}).not.toThrowError("Illegal invocation");
+				}).not.toThrow("Illegal invocation");
 
 				expect(() => {
 					passThroughOnException.call(ctx);
-				}).not.toThrowError("Illegal invocation");
+				}).not.toThrow("Illegal invocation");
 			} finally {
 				await dispose();
 			}
@@ -131,15 +131,15 @@ describe("getPlatformProxy - ctx", () => {
 			try {
 				expect(() => {
 					waitUntil(new Promise(() => {}));
-				}).toThrowError("Illegal invocation");
+				}).toThrow("Illegal invocation");
 
 				expect(() => {
 					passThroughOnException.apply(5, []);
-				}).toThrowError("Illegal invocation");
+				}).toThrow("Illegal invocation");
 
 				expect(() => {
 					passThroughOnException.call(new Boolean());
-				}).toThrowError("Illegal invocation");
+				}).toThrow("Illegal invocation");
 			} finally {
 				await dispose();
 			}

fixtures/pages-workerjs-app/tests/index.test.ts

@@ -15,7 +15,7 @@ describe("Pages _worker.js", () => {
 				cwd: path.resolve(__dirname, ".."),
 				stdio: "ignore",
 			})
-		).toThrowError();
+		).toThrow();
 	});
 
 	it("should throw an error when the _worker.js file imports something and --no-bundle is true", ({
@@ -26,7 +26,7 @@ describe("Pages _worker.js", () => {
 				cwd: path.resolve(__dirname, ".."),
 				stdio: "ignore",
 			})
-		).toThrowError();
+		).toThrow();
 	});
 
 	it("should not throw an error when the _worker.js file imports something if --no-bundle is false", async ({

packages/autoconfig/tests/details/framework-detection/multiple-frameworks-detected.test.ts

@@ -136,9 +136,7 @@ describe("detectFramework() / multiple frameworks detected", () => {
 				"package-lock.json": JSON.stringify({ lockfileVersion: 3 }),
 			});
 
-			await expect(
-				detectFramework(process.cwd(), ciContext)
-			).rejects.toThrowError(
+			await expect(detectFramework(process.cwd(), ciContext)).rejects.toThrow(
 				/Cloudflare's tooling was unable to automatically configure your project, since multiple frameworks were found/
 			);
 		});

packages/autoconfig/tests/details/get-details-for-auto-config.test.ts

@@ -475,7 +475,7 @@ describe("autoconfig details - getDetailsForAutoConfig()", () => {
 
 				await expect(
 					details.getDetailsForAutoConfig({ context: ciContext })
-				).rejects.toThrowError(
+				).rejects.toThrow(
 					/Cloudflare's tooling was unable to automatically configure your project, since multiple frameworks were found/
 				);
 			});

packages/autoconfig/tests/vite-config.test.ts

@@ -260,7 +260,7 @@ export default defineConfig({
 `
 			);
 
-			expect(() => transformViteConfig(".")).toThrowError(
+			expect(() => transformViteConfig(".")).toThrow(
 				/Cannot modify Vite config: could not find a valid plugins array/
 			);
 		});
@@ -279,7 +279,7 @@ export default defineConfig(() => ({
 `
 			);
 
-			expect(() => transformViteConfig(".")).toThrowError(
+			expect(() => transformViteConfig(".")).toThrow(
 				/Cannot modify Vite config: could not find a valid plugins array/
 			);
 		});

packages/containers-shared/tests/images.test.ts

@@ -116,7 +116,7 @@ describe("getAndValidateRegistryType - GAR", () => {
 	];
 	for (const domain of invalidGarDomains) {
 		it(`rejects ${domain}`, ({ expect }) => {
-			expect(() => getAndValidateRegistryType(domain)).toThrowError(
+			expect(() => getAndValidateRegistryType(domain)).toThrow(
 				`${domain} is not a supported image registry`
 			);
 		});

packages/create-cloudflare/src/helpers/__tests__/retry.test.ts

@@ -44,7 +44,7 @@ describe("retry", () => {
 			await retry({ times: 3 }, async () => {
 				throw Error("testing");
 			});
-		}).rejects.toThrowError("testing");
+		}).rejects.toThrow("testing");
 	});
 
 	test("exit condition encountered", async ({ expect }) => {
@@ -65,6 +65,6 @@ describe("retry", () => {
 					throw Error("special condition");
 				}
 			);
-		}).rejects.toThrowError("special condition");
+		}).rejects.toThrow("special condition");
 	});
 });

packages/kv-asset-handler/test/getAssetFromKV.test.ts

@@ -66,7 +66,7 @@ test("getAssetFromKV if no asset manifest /client -> client fails", async ({
 	const event = getEvent(new Request(`https://foo.com/client`));
 	await expect(() =>
 		getAssetFromKV(event, { ASSET_MANIFEST: {} })
-	).rejects.toThrowError(expect.objectContaining({ status: 404 }));
+	).rejects.toThrow(expect.objectContaining({ status: 404 }));
 });
 
 test("getAssetFromKV if sub/ -> sub/index.html served", async ({ expect }) => {

packages/vite-plugin-cloudflare/src/__tests__/dev-vars.spec.ts

@@ -75,7 +75,7 @@ describe("quoteForDotenv", () => {
 
 	test("throws when a value cannot be losslessly serialized", ({ expect }) => {
 		// Contains all three quote characters — no safe encoding under dotenv.
-		expect(() => quoteForDotenv("'`\"")).toThrowError(
+		expect(() => quoteForDotenv("'`\"")).toThrow(
 			/Unable to serialize value to \.dev\.vars/
 		);
 	});

packages/vite-plugin-cloudflare/src/__tests__/get-worker-config.spec.ts

@@ -177,7 +177,7 @@ describe("resolveWorkerType", () => {
 		);
 		expect(() =>
 			resolveWorkerType(config, raw, nonApplicable, { configPath })
-		).toThrowError(
+		).toThrow(
 			/The provided Wrangler config main field \(.*?index\.ts\) doesn't point to an existing file/
 		);
 	});

packages/vitest-pool-workers/test/compatibility-flag-assertions.test.ts

@@ -145,7 +145,7 @@ describe("FlagAssertions", () => {
 					enableFlag: "enable-flag",
 					defaultOnDate: "invalid-date",
 				});
-			}).toThrowError('Invalid date format: "invalid-date"');
+			}).toThrow('Invalid date format: "invalid-date"');
 		});
 
 		it("throws error when compatibilityDate is invalid", ({ expect }) => {
@@ -161,7 +161,7 @@ describe("FlagAssertions", () => {
 					enableFlag: "enable-flag",
 					defaultOnDate: "2023-01-01",
 				});
-			}).toThrowError('Invalid date format: "invalid-date"');
+			}).toThrow('Invalid date format: "invalid-date"');
 		});
 	});
 

packages/workers-shared/asset-worker/tests/kv.test.ts

@@ -50,7 +50,7 @@ describe("[Asset Worker] Fetching assets from KV", () => {
 
 			await expect(() =>
 				getAssetWithMetadataFromKV(mockKVNamespace, "abcd")
-			).rejects.toThrowError("KV GET abcd failed.");
+			).rejects.toThrow("KV GET abcd failed.");
 		});
 
 		it("should retry once by default if something went wrong while fetching the asset", async ({
@@ -60,7 +60,7 @@ describe("[Asset Worker] Fetching assets from KV", () => {
 
 			await expect(() =>
 				getAssetWithMetadataFromKV(mockKVNamespace, "abcd")
-			).rejects.toThrowError("KV GET abcd failed.");
+			).rejects.toThrow("KV GET abcd failed.");
 			expect(spy).toHaveBeenCalledTimes(2);
 		});
 
@@ -69,7 +69,7 @@ describe("[Asset Worker] Fetching assets from KV", () => {
 
 			await expect(() =>
 				getAssetWithMetadataFromKV(mockKVNamespace, "abcd", undefined, 2)
-			).rejects.toThrowError("KV GET abcd failed.");
+			).rejects.toThrow("KV GET abcd failed.");
 			expect(spy).toHaveBeenCalledTimes(3);
 		});
 
@@ -80,7 +80,7 @@ describe("[Asset Worker] Fetching assets from KV", () => {
 
 			await expect(() =>
 				getAssetWithMetadataFromKV(mockKVNamespace, "abcd")
-			).rejects.toThrowError("KV GET abcd failed: Oeps! Something went wrong");
+			).rejects.toThrow("KV GET abcd failed: Oeps! Something went wrong");
 			expect(spy).toHaveBeenCalledTimes(2);
 		});
 

packages/workers-shared/router-worker/tests/index.test.ts

@@ -54,7 +54,7 @@ describe("inner entrypoint unit tests", () => {
 
 		await expect(
 			async () => await fetchFromInnerEntrypoint(request, env, ctx)
-		).rejects.toThrowError(
+		).rejects.toThrow(
 			"Fetch for user worker without having a user worker binding"
 		);
 	});

packages/workers-utils/tests/prometheus-metrics.test.ts

@@ -69,9 +69,7 @@ describe("MetricsRegistry", () => {
 		it("throws on negative values", ({ expect }) => {
 			const registry = new MetricsRegistry();
 			const counter = registry.createCounter("test_total", "A test counter");
-			expect(() => counter.add(-1)).toThrowError(
-				"Counter value cannot decrease"
-			);
+			expect(() => counter.add(-1)).toThrow("Counter value cannot decrease");
 		});
 
 		it("can be combined with inc", ({ expect }) => {

packages/wrangler/e2e/dev.test.ts

@@ -1448,7 +1448,7 @@ describe("custom builds", () => {
 		// regression: https://github.com/cloudflare/workers-sdk/issues/6876
 		await expect(
 			worker.readUntil(/\[custom build\] Running/, 5_000)
-		).rejects.toThrowError();
+		).rejects.toThrow();
 
 		// now check assets are still fetchable, even after updates
 
@@ -2085,7 +2085,7 @@ describe("watch mode", () => {
 				await worker.waitForReload();
 
 				// The three changes should be debounced, so only one reload should occur
-				await expect(worker.waitForReload(5_000)).rejects.toThrowError();
+				await expect(worker.waitForReload(5_000)).rejects.toThrow();
 
 				// now check assets are still fetchable
 				await expect(fetchText(url)).resolves.toBe("Hello from Assets");

packages/wrangler/e2e/startWorker.test.ts

@@ -327,9 +327,9 @@ describe("DevEnv", { sequential: true }, () => {
 			undiciRes = await undici.fetch(`http://127.0.0.1:${newPort}`);
 			await expect(undiciRes.text()).resolves.toBe("body:1");
 
-			await expect(
-				undici.fetch(`http://127.0.0.1:${oldPort}`)
-			).rejects.toThrowError("fetch failed");
+			await expect(undici.fetch(`http://127.0.0.1:${oldPort}`)).rejects.toThrow(
+				"fetch failed"
+			);
 		});
 
 		it("liveReload", async ({ expect }) => {
@@ -456,7 +456,7 @@ describe("DevEnv", { sequential: true }, () => {
 			});
 			onTestFinished(worker?.dispose);
 
-			await expect(worker.fetch("http://dummy")).rejects.toThrowError("Boom!");
+			await expect(worker.fetch("http://dummy")).rejects.toThrow("Boom!");
 
 			await helper.seed({
 				"src/index.ts": dedent`
@@ -469,9 +469,7 @@ describe("DevEnv", { sequential: true }, () => {
 			});
 
 			await waitFor(async () => {
-				await expect(worker.fetch("http://dummy")).rejects.toThrowError(
-					"Boom 2!"
-				);
+				await expect(worker.fetch("http://dummy")).rejects.toThrow("Boom 2!");
 			});
 
 			// test eyeball requests receive the pretty error page