fix tests

Author: penalosa

packages/wrangler/src/__tests__/provision.test.ts

@@ -192,6 +192,7 @@ describe("--x-provision", () => {
 				Provisioning R2 (R2 Bucket)...
 				✨ R2 provisioned 🎉
 
+				Your Worker was deployed with provisioned resources. We've written the IDs of these resources to your config file, which you can choose to save or discard—either way future deploys will continue to work.
 				🎉 All resources provisioned, continuing with deployment...
 
 				Worker Startup Time: 100 ms
@@ -208,9 +209,6 @@ describe("--x-provision", () => {
 			`);
 			expect(std.err).toMatchInlineSnapshot(`""`);
 			expect(std.warn).toMatchInlineSnapshot(`""`);
-
-			// IDs should be written back to the config file
-			expect(await readFile("wrangler.toml", "utf-8")).toMatchInlineSnapshot();
 		});
 
 		it("can provision KV, R2 and D1 bindings with existing resources, and lets you search when there are too many to list", async () => {
@@ -312,6 +310,7 @@ describe("--x-provision", () => {
 				Provisioning R2 (R2 Bucket)...
 				✨ R2 provisioned 🎉
 
+				Your Worker was deployed with provisioned resources. We've written the IDs of these resources to your config file, which you can choose to save or discard—either way future deploys will continue to work.
 				🎉 All resources provisioned, continuing with deployment...
 
 				Worker Startup Time: 100 ms
@@ -442,6 +441,7 @@ describe("--x-provision", () => {
 				🌀 Creating new R2 Bucket \\"new-r2\\"...
 				✨ R2 provisioned 🎉
 
+				Your Worker was deployed with provisioned resources. We've written the IDs of these resources to your config file, which you can choose to save or discard—either way future deploys will continue to work.
 				🎉 All resources provisioned, continuing with deployment...
 
 				Worker Startup Time: 100 ms
@@ -458,6 +458,26 @@ describe("--x-provision", () => {
 			`);
 			expect(std.err).toMatchInlineSnapshot(`""`);
 			expect(std.warn).toMatchInlineSnapshot(`""`);
+
+			// IDs should be written back to the config file
+			expect(await readFile("wrangler.toml", "utf-8")).toMatchInlineSnapshot(`
+				"compatibility_date = \\"2022-01-12\\"
+				name = \\"test-name\\"
+				main = \\"index.js\\"
+
+				[[kv_namespaces]]
+				binding = \\"KV\\"
+				id = \\"new-kv-id\\"
+
+				[[r2_buckets]]
+				binding = \\"R2\\"
+				bucket_name = \\"new-r2\\"
+
+				[[d1_databases]]
+				binding = \\"D1\\"
+				database_id = \\"new-d1-id\\"
+				"
+			`);
 		});
 
 		it("can prefill d1 database name from config file if provided", async () => {
@@ -511,6 +531,7 @@ describe("--x-provision", () => {
 				🌀 Creating new D1 Database \\"prefilled-d1-name\\"...
 				✨ D1 provisioned 🎉
 
+				Your Worker was deployed with provisioned resources. We've written the IDs of these resources to your config file, which you can choose to save or discard—either way future deploys will continue to work.
 				🎉 All resources provisioned, continuing with deployment...
 
 				Worker Startup Time: 100 ms
@@ -633,6 +654,7 @@ describe("--x-provision", () => {
 				🌀 Creating new D1 Database \\"new-d1-name\\"...
 				✨ D1 provisioned 🎉
 
+				Your Worker was deployed with provisioned resources. We've written the IDs of these resources to your config file, which you can choose to save or discard—either way future deploys will continue to work.
 				🎉 All resources provisioned, continuing with deployment...
 
 				Worker Startup Time: 100 ms
@@ -708,6 +730,7 @@ describe("--x-provision", () => {
 				🌀 Creating new R2 Bucket \\"prefilled-r2-name\\"...
 				✨ BUCKET provisioned 🎉
 
+				Your Worker was deployed with provisioned resources. We've written the IDs of these resources to your config file, which you can choose to save or discard—either way future deploys will continue to work.
 				🎉 All resources provisioned, continuing with deployment...
 
 				Worker Startup Time: 100 ms
@@ -894,6 +917,7 @@ describe("--x-provision", () => {
 				🌀 Creating new R2 Bucket \\"existing-bucket-name\\"...
 				✨ BUCKET provisioned 🎉
 
+				Your Worker was deployed with provisioned resources. We've written the IDs of these resources to your config file, which you can choose to save or discard—either way future deploys will continue to work.
 				🎉 All resources provisioned, continuing with deployment...
 
 				Worker Startup Time: 100 ms

packages/wrangler/src/deployment-bundle/bindings.ts

@@ -1,7 +1,9 @@
 import assert from "node:assert";
 import { fetchResult } from "../cfetch";
-import { experimental_patchConfig } from "../cli";
-import { PatchConfigError } from "../config/patch-config";
+import {
+	experimental_patchConfig,
+	PatchConfigError,
+} from "../config/patch-config";
 import { createD1Database } from "../d1/create";
 import { listDatabases } from "../d1/list";
 import { getDatabaseInfoFromIdOrName } from "../d1/utils";