wrangler 0.0.0-e043479 → 0.0.0-e0b9366

package/src/__tests__/configuration.test.ts

@@ -65,8 +65,8 @@ describe("normalizeAndValidateConfig()", () => {
         dev: {
           ip: "255.255.255.255",
           port: 9999,
-          local_protocol: "HTTPS",
-          upstream_protocol: "HTTP",
+          local_protocol: "https",
+          upstream_protocol: "http",
         },
         build: {
           command: "COMMAND",
@@ -95,8 +95,8 @@ describe("normalizeAndValidateConfig()", () => {
         dev: {
           ip: 222,
           port: "FOO",
-          local_protocol: 333,
-          upstream_protocol: 444,
+          local_protocol: "wss",
+          upstream_protocol: "ws",
         },
         build: {
           command: 555,
@@ -115,20 +115,40 @@ describe("normalizeAndValidateConfig()", () => {
       );
       expect(diagnostics.hasWarnings()).toBe(false);
       expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
-          "Processing wrangler configuration:
-            - ERROR: \\"build.command\\" should be of type string but got 555.
-            - ERROR: \\"build.cwd\\" should be of type string but got 666.
-            - ERROR: \\"build.watch_dir\\" should be of type string but got 777.
-            - ERROR: \\"legacy_env\\" should be of type boolean but got \\"FOO\\".
-            - ERROR: \\"main\\" should be a string but got 111
-            - ERROR: \\"dev.ip\\" should be of type string but got 222.
-            - ERROR: \\"dev.port\\" should be of type number but got \\"FOO\\".
-            - ERROR: \\"dev.local_protocol\\" should be of type string but got 333.
-            - ERROR: \\"dev.upstream_protocol\\" should be of type string but got 444."
-        `);
+        "Processing wrangler configuration:
+          - Expected \\"build.command\\" to be of type string but got 555.
+          - Expected \\"build.cwd\\" to be of type string but got 666.
+          - Expected \\"build.watch_dir\\" to be of type string but got 777.
+          - Expected \\"legacy_env\\" to be of type boolean but got \\"FOO\\".
+          - Expected \\"main\\" to be a string but got 111
+          - Expected \\"dev.ip\\" to be of type string but got 222.
+          - Expected \\"dev.port\\" to be of type number but got \\"FOO\\".
+          - Expected \\"dev.local_protocol\\" field to be one of [\\"http\\",\\"https\\"] but got \\"wss\\".
+          - Expected \\"dev.upstream_protocol\\" field to be one of [\\"http\\",\\"https\\"] but got \\"ws\\"."
+      `);
     });
 
     it("should warn on and remove unexpected top level fields", () => {
+      const expectedConfig = {
+        unexpected: {
+          subkey: "some-value",
+        },
+      };
+
+      const { config, diagnostics } = normalizeAndValidateConfig(
+        expectedConfig as unknown as RawConfig,
+        undefined
+      );
+
+      expect("unexpected" in config).toBe(false);
+      expect(diagnostics.hasErrors()).toBe(false);
+      expect(diagnostics.renderWarnings()).toMatchInlineSnapshot(`
+        "Processing wrangler configuration:
+          - Unexpected fields found in top-level field: \\"unexpected\\""
+      `);
+    });
+
+    it("should ignore `miniflare` top level fields", () => {
       const expectedConfig = {
         miniflare: {
           host: "127.0.0.1",
@@ -144,7 +164,7 @@ describe("normalizeAndValidateConfig()", () => {
       expect(diagnostics.hasErrors()).toBe(false);
       expect(diagnostics.renderWarnings()).toMatchInlineSnapshot(`
         "Processing wrangler configuration:
-          - Unexpected fields found in top-level field: \\"miniflare\\""
+        "
       `);
     });
 
@@ -244,84 +264,115 @@ describe("normalizeAndValidateConfig()", () => {
       expect(config).toEqual(expect.objectContaining(expectedConfig));
       expect(diagnostics.hasWarnings()).toBe(false);
       expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
-          "Processing wrangler configuration:
-            - ERROR: \\"migrations[0].tag\\" should be of type string but got 111.
-            - ERROR: \\"migrations[0].new_classes.[0]\\" should be of type string but got 222.
-            - ERROR: \\"migrations[0].new_classes.[1]\\" should be of type string but got 333.
-            - \\"migrations[0].renamed_classes\\" should be an array of \\"{from: string, to: string}\\" objects but got [{\\"from\\":444,\\"to\\":555}].
-            - ERROR: \\"migrations[0].deleted_classes.[0]\\" should be of type string but got 666.
-            - ERROR: \\"migrations[0].deleted_classes.[1]\\" should be of type string but got 777."
-        `);
+        "Processing wrangler configuration:
+          - Expected \\"migrations[0].tag\\" to be of type string but got 111.
+          - Expected \\"migrations[0].new_classes.[0]\\" to be of type string but got 222.
+          - Expected \\"migrations[0].new_classes.[1]\\" to be of type string but got 333.
+          - Expected \\"migrations[0].renamed_classes\\" to be an array of \\"{from: string, to: string}\\" objects but got [{\\"from\\":444,\\"to\\":555}].
+          - Expected \\"migrations[0].deleted_classes.[0]\\" to be of type string but got 666.
+          - Expected \\"migrations[0].deleted_classes.[1]\\" to be of type string but got 777."
+      `);
     });
 
-    it("should override `site` config defaults with provided values", () => {
-      const expectedConfig: RawConfig = {
-        site: {
-          bucket: "BUCKET",
-          "entry-point": "ENTRY_POINT",
-          include: ["INCLUDE_1", "INCLUDE_2"],
-          exclude: ["EXCLUDE_1", "EXCLUDE_2"],
-        },
-      };
+    describe("site", () => {
+      it("should override `site` config defaults with provided values", () => {
+        const expectedConfig: RawConfig = {
+          site: {
+            bucket: "BUCKET",
+            include: ["INCLUDE_1", "INCLUDE_2"],
+            exclude: ["EXCLUDE_1", "EXCLUDE_2"],
+          },
+        };
 
-      const { config, diagnostics } = normalizeAndValidateConfig(
-        expectedConfig,
-        undefined
-      );
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          expectedConfig,
+          undefined
+        );
 
-      expect(config).toEqual(expect.objectContaining(expectedConfig));
-      expect(diagnostics.hasErrors()).toBe(false);
-      expect(diagnostics.hasWarnings()).toBe(false);
-    });
+        expect(config).toEqual(expect.objectContaining(expectedConfig));
+        expect(diagnostics.hasErrors()).toBe(false);
+        expect(diagnostics.hasWarnings()).toBe(false);
+      });
 
-    it("should error if `site` config is missing `bucket`", () => {
-      const expectedConfig: RawConfig = {
-        site: {
-          "entry-point": "ENTRY_POINT",
-          include: ["INCLUDE_1", "INCLUDE_2"],
-          exclude: ["EXCLUDE_1", "EXCLUDE_2"],
-        },
-      } as RawConfig;
+      it("should error if `site` config is missing `bucket`", () => {
+        const expectedConfig: RawConfig = {
+          // @ts-expect-error we're intentionally passing an invalid configuration here
+          site: {
+            include: ["INCLUDE_1", "INCLUDE_2"],
+            exclude: ["EXCLUDE_1", "EXCLUDE_2"],
+          },
+        };
 
-      const { config, diagnostics } = normalizeAndValidateConfig(
-        expectedConfig,
-        undefined
-      );
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          expectedConfig,
+          undefined
+        );
 
-      expect(config).toEqual(expect.objectContaining(expectedConfig));
-      expect(diagnostics.hasErrors()).toBe(true);
-      expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
-                "Processing wrangler configuration:
-                  - ERROR: \\"site.bucket\\" is a required field."
-            `);
-      expect(diagnostics.hasWarnings()).toBe(false);
-    });
+        expect(config).toEqual(expect.objectContaining(expectedConfig));
+        expect(diagnostics.hasErrors()).toBe(true);
+        expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+          "Processing wrangler configuration:
+            - \\"site.bucket\\" is a required field."
+        `);
+        expect(diagnostics.hasWarnings()).toBe(false);
+      });
 
-    it("should error on invalid `site` values", () => {
-      const expectedConfig = {
-        site: {
-          bucket: "BUCKET",
-          "entry-point": 111,
-          include: [222, 333],
-          exclude: [444, 555],
-        },
-      };
+      it("should error on invalid `site` values", () => {
+        const expectedConfig = {
+          site: {
+            bucket: "BUCKET",
+            include: [222, 333],
+            exclude: [444, 555],
+          },
+        };
 
-      const { config, diagnostics } = normalizeAndValidateConfig(
-        expectedConfig as unknown as RawConfig,
-        undefined
-      );
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          expectedConfig as unknown as RawConfig,
+          undefined
+        );
 
-      expect(config).toEqual(expect.objectContaining(expectedConfig));
-      expect(diagnostics.hasWarnings()).toBe(false);
-      expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+        expect(config).toEqual(expect.objectContaining(expectedConfig));
+        expect(diagnostics.hasWarnings()).toBe(false);
+        expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+          "Processing wrangler configuration:
+            - Expected \\"sites.include.[0]\\" to be of type string but got 222.
+            - Expected \\"sites.include.[1]\\" to be of type string but got 333.
+            - Expected \\"sites.exclude.[0]\\" to be of type string but got 444.
+            - Expected \\"sites.exclude.[1]\\" to be of type string but got 555."
+        `);
+      });
+
+      it("should error with a deprecation warning if entry-point is defined", async () => {
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          {
+            site: {
+              bucket: "some/path",
+              "entry-point": "some/other/script.js",
+            },
+          } as unknown as RawConfig,
+          undefined
+        );
+
+        expect(config.site).toMatchInlineSnapshot(`
+          Object {
+            "bucket": "some/path",
+            "exclude": Array [],
+            "include": Array [],
+          }
+        `);
+        expect(diagnostics.hasWarnings()).toBe(true);
+        expect(diagnostics.hasErrors()).toBe(true);
+        expect(diagnostics.renderWarnings()).toMatchInlineSnapshot(`
           "Processing wrangler configuration:
-            - ERROR: \\"site.entry_point\\" should be of type string but got 111.
-            - ERROR: \\"sites.include.[0]\\" should be of type string but got 222.
-            - ERROR: \\"sites.include.[1]\\" should be of type string but got 333.
-            - ERROR: \\"sites.exclude.[0]\\" should be of type string but got 444.
-            - ERROR: \\"sites.exclude.[1]\\" should be of type string but got 555."
+            - Unexpected fields found in site field: \\"entry-point\\""
         `);
+        expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+          "Processing wrangler configuration:
+            - DEPRECATION: \\"site.entry-point\\":
+              The \`site.entry-point\` config field is no longer used.
+              The entry-point should be specified via the command line or the \`main\` config field."
+        `);
+      });
     });
 
     it("should map `wasm_module` paths from relative to the config path to relative to the cwd", () => {
@@ -467,7 +518,6 @@ describe("normalizeAndValidateConfig()", () => {
         compatibility_flags: ["FLAG1", "FLAG2"],
         workers_dev: false,
         routes: ["ROUTE_1", "ROUTE_2"],
-        route: "ROUTE_3",
         jsx_factory: "JSX_FACTORY",
         jsx_fragment: "JSX_FRAGMENT",
         triggers: { crons: ["CRON_1", "CRON_2"] },
@@ -553,6 +603,7 @@ describe("normalizeAndValidateConfig()", () => {
         "Processing wrangler configuration:
           - Expected \\"route\\" field to be a string but got 888.
           - Expected \\"routes\\" field to be an array of strings but got [666,777].
+          - Expected exactly one of [\\"routes\\",\\"route\\"].
           - Expected \\"workers_dev\\" field to be a boolean but got \\"BAD\\".
           - Expected \\"account_id\\" field to be a string but got 222.
           - Expected \\"compatibility_date\\" field to be a string but got 333.
@@ -560,7 +611,7 @@ describe("normalizeAndValidateConfig()", () => {
           - Expected \\"jsx_factory\\" field to be a string but got 999.
           - Expected \\"jsx_fragment\\" field to be a string but got 1000.
           - Expected \\"name\\" field to be a string but got 111.
-          - Expected \\"usage_model\\" field to be on of bundled,unbound but got \\"INVALID\\"."
+          - Expected \\"usage_model\\" field to be one of [\\"bundled\\",\\"unbound\\"] but got \\"INVALID\\"."
       `);
     });
 
@@ -746,20 +797,20 @@ describe("normalizeAndValidateConfig()", () => {
           "Processing wrangler configuration:
 
             - \\"durable_objects.bindings[0]\\": {}
-              - ERROR: bindings should have a string \\"name\\" field.
-              - ERROR: should have a string \\"class_name\\" field.
+              - binding should have a string \\"name\\" field.
+              - binding should have a string \\"class_name\\" field.
 
             - \\"durable_objects.bindings[1]\\": {\\"name\\":\\"VALID\\"}
-              - ERROR: should have a string \\"class_name\\" field.
+              - binding should have a string \\"class_name\\" field.
 
             - \\"durable_objects.bindings[2]\\": {\\"name\\":1555,\\"class_name\\":1666}
-              - ERROR: bindings should have a string \\"name\\" field.
-              - ERROR: should have a string \\"class_name\\" field.
+              - binding should have a string \\"name\\" field.
+              - binding should have a string \\"class_name\\" field.
 
             - \\"durable_objects.bindings[3]\\": {\\"name\\":1777,\\"class_name\\":1888,\\"script_name\\":1999}
-              - ERROR: bindings should have a string \\"name\\" field.
-              - ERROR: should have a string \\"class_name\\" field.
-              - ERROR: should, optionally, have a string \\"script_name\\" field."
+              - binding should have a string \\"name\\" field.
+              - binding should have a string \\"class_name\\" field.
+              - binding should, optionally, have a string \\"script_name\\" field."
         `);
       });
     });
@@ -1176,15 +1227,15 @@ describe("normalizeAndValidateConfig()", () => {
           "Processing wrangler configuration:
 
             - \\"unsafe.bindings[0]\\": {}
-              - ERROR: should have a string \\"name\\" field.
-              - ERROR: should have a string \\"type\\" field.
+              - binding should have a string \\"name\\" field.
+              - binding should have a string \\"type\\" field.
 
             - \\"unsafe.bindings[1]\\": {\\"name\\":\\"UNSAFE_BINDING_1\\"}
-              - ERROR: should have a string \\"type\\" field.
+              - binding should have a string \\"type\\" field.
 
             - \\"unsafe.bindings[2]\\": {\\"name\\":2666,\\"type\\":2777}
-              - ERROR: should have a string \\"name\\" field.
-              - ERROR: should have a string \\"type\\" field."
+              - binding should have a string \\"name\\" field.
+              - binding should have a string \\"type\\" field."
         `);
       });
     });
@@ -1228,6 +1279,25 @@ describe("normalizeAndValidateConfig()", () => {
           `);
       });
     });
+
+    describe("route & routes fields", () => {
+      it("should error if both route and routes are specified", () => {
+        const rawConfig: RawConfig = {
+          route: "route1",
+          routes: ["route2", "route3"],
+        };
+
+        const { diagnostics } = normalizeAndValidateConfig(
+          rawConfig,
+          undefined
+        );
+
+        expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+          "Processing wrangler configuration:
+            - Expected exactly one of [\\"routes\\",\\"route\\"]."
+        `);
+      });
+    });
   });
 
   describe("named environments", () => {
@@ -1239,7 +1309,6 @@ describe("normalizeAndValidateConfig()", () => {
         compatibility_flags: ["FLAG1", "FLAG2"],
         workers_dev: false,
         routes: ["ROUTE_1", "ROUTE_2"],
-        route: "ROUTE_3",
         jsx_factory: "JSX_FACTORY",
         jsx_fragment: "JSX_FRAGMENT",
         triggers: { crons: ["CRON_1", "CRON_2"] },
@@ -1264,7 +1333,6 @@ describe("normalizeAndValidateConfig()", () => {
         compatibility_flags: ["ENV_FLAG1", "ENV_FLAG2"],
         workers_dev: true,
         routes: ["ENV_ROUTE_1", "ENV_ROUTE_2"],
-        route: "ENV_ROUTE_3",
         jsx_factory: "ENV_JSX_FACTORY",
         jsx_fragment: "ENV_JSX_FRAGMENT",
         triggers: { crons: ["ENV_CRON_1", "ENV_CRON_2"] },
@@ -1277,7 +1345,6 @@ describe("normalizeAndValidateConfig()", () => {
         compatibility_flags: ["FLAG1", "FLAG2"],
         workers_dev: false,
         routes: ["ROUTE_1", "ROUTE_2"],
-        route: "ROUTE_3",
         jsx_factory: "JSX_FACTORY",
         jsx_fragment: "JSX_FRAGMENT",
         triggers: { crons: ["CRON_1", "CRON_2"] },
@@ -1297,6 +1364,78 @@ describe("normalizeAndValidateConfig()", () => {
       expect(diagnostics.hasWarnings()).toBe(false);
     });
 
+    describe("non-legacy", () => {
+      it("should use top-level `name` field", () => {
+        const rawConfig: RawConfig = {
+          name: "NAME",
+          legacy_env: false,
+          env: { DEV: {} },
+        };
+
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          rawConfig,
+          undefined
+        );
+
+        expect(config.env.DEV.name).toEqual("NAME");
+        expect(diagnostics.hasErrors()).toBe(false);
+        expect(diagnostics.hasWarnings()).toBe(false);
+      });
+
+      it("should error if named environment contains a `name` field, even if there is no top-level name", () => {
+        const rawConfig: RawConfig = {
+          legacy_env: false,
+          env: {
+            DEV: {
+              name: "ENV_NAME",
+            },
+          },
+        };
+
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          rawConfig,
+          undefined
+        );
+
+        expect(config.env.DEV.name).toBeUndefined();
+        expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+          "Processing wrangler configuration:
+
+            - \\"env.DEV\\" environment configuration
+              - The \\"name\\" field is not allowed in named service environments.
+                Please remove the field from this environment."
+        `);
+        expect(diagnostics.hasWarnings()).toBe(false);
+      });
+
+      it("should error if top-level config and a named environment both contain a `name` field", () => {
+        const rawConfig: RawConfig = {
+          name: "NAME",
+          legacy_env: false,
+          env: {
+            DEV: {
+              name: "ENV_NAME",
+            },
+          },
+        };
+
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          rawConfig,
+          undefined
+        );
+
+        expect(config.env.DEV.name).toEqual("NAME");
+        expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+          "Processing wrangler configuration:
+
+            - \\"env.DEV\\" environment configuration
+              - The \\"name\\" field is not allowed in named service environments.
+                Please remove the field from this environment."
+        `);
+        expect(diagnostics.hasWarnings()).toBe(false);
+      });
+    });
+
     it("should warn for non-inherited fields that are missing in environments", () => {
       const vars: RawConfig["vars"] = {
         FOO: "foo",
@@ -1383,6 +1522,7 @@ describe("normalizeAndValidateConfig()", () => {
           - \\"env.ENV1\\" environment configuration
             - Expected \\"route\\" field to be a string but got 888.
             - Expected \\"routes\\" field to be an array of strings but got [666,777].
+            - Expected exactly one of [\\"routes\\",\\"route\\"].
             - Expected \\"workers_dev\\" field to be a boolean but got \\"BAD\\".
             - Expected \\"account_id\\" field to be a string but got 222.
             - Expected \\"compatibility_date\\" field to be a string but got 333.
@@ -1390,7 +1530,7 @@ describe("normalizeAndValidateConfig()", () => {
             - Expected \\"jsx_factory\\" field to be a string but got 999.
             - Expected \\"jsx_fragment\\" field to be a string but got 1000.
             - Expected \\"name\\" field to be a string but got 111.
-            - Expected \\"usage_model\\" field to be on of bundled,unbound but got \\"INVALID\\"."
+            - Expected \\"usage_model\\" field to be one of [\\"bundled\\",\\"unbound\\"] but got \\"INVALID\\"."
       `);
     });
 
@@ -1608,20 +1748,20 @@ describe("normalizeAndValidateConfig()", () => {
             - \\"env.ENV1\\" environment configuration
 
               - \\"env.ENV1.durable_objects.bindings[0]\\": {}
-                - ERROR: bindings should have a string \\"name\\" field.
-                - ERROR: should have a string \\"class_name\\" field.
+                - binding should have a string \\"name\\" field.
+                - binding should have a string \\"class_name\\" field.
 
               - \\"env.ENV1.durable_objects.bindings[1]\\": {\\"name\\":\\"VALID\\"}
-                - ERROR: should have a string \\"class_name\\" field.
+                - binding should have a string \\"class_name\\" field.
 
               - \\"env.ENV1.durable_objects.bindings[2]\\": {\\"name\\":1555,\\"class_name\\":1666}
-                - ERROR: bindings should have a string \\"name\\" field.
-                - ERROR: should have a string \\"class_name\\" field.
+                - binding should have a string \\"name\\" field.
+                - binding should have a string \\"class_name\\" field.
 
               - \\"env.ENV1.durable_objects.bindings[3]\\": {\\"name\\":1777,\\"class_name\\":1888,\\"script_name\\":1999}
-                - ERROR: bindings should have a string \\"name\\" field.
-                - ERROR: should have a string \\"class_name\\" field.
-                - ERROR: should, optionally, have a string \\"script_name\\" field."
+                - binding should have a string \\"name\\" field.
+                - binding should have a string \\"class_name\\" field.
+                - binding should, optionally, have a string \\"script_name\\" field."
         `);
       });
     });
@@ -2118,15 +2258,15 @@ describe("normalizeAndValidateConfig()", () => {
             - \\"env.ENV1\\" environment configuration
 
               - \\"env.ENV1.unsafe.bindings[0]\\": {}
-                - ERROR: should have a string \\"name\\" field.
-                - ERROR: should have a string \\"type\\" field.
+                - binding should have a string \\"name\\" field.
+                - binding should have a string \\"type\\" field.
 
               - \\"env.ENV1.unsafe.bindings[1]\\": {\\"name\\":\\"UNSAFE_BINDING_1\\"}
-                - ERROR: should have a string \\"type\\" field.
+                - binding should have a string \\"type\\" field.
 
               - \\"env.ENV1.unsafe.bindings[2]\\": {\\"name\\":2666,\\"type\\":2777}
-                - ERROR: should have a string \\"name\\" field.
-                - ERROR: should have a string \\"type\\" field."
+                - binding should have a string \\"name\\" field.
+                - binding should have a string \\"type\\" field."
         `);
       });
     });
@@ -2176,6 +2316,231 @@ describe("normalizeAndValidateConfig()", () => {
         `);
       });
     });
+
+    describe("route & routes fields", () => {
+      it("should error if both route and routes are specified in the same environment", () => {
+        const environment: RawEnvironment = {
+          name: "ENV_NAME",
+          account_id: "ENV_ACCOUNT_ID",
+          compatibility_date: "2022-02-02",
+          compatibility_flags: ["ENV_FLAG1", "ENV_FLAG2"],
+          workers_dev: true,
+          route: "ENV_ROUTE_1",
+          routes: ["ENV_ROUTE_2", "ENV_ROUTE_3"],
+          jsx_factory: "ENV_JSX_FACTORY",
+          jsx_fragment: "ENV_JSX_FRAGMENT",
+          triggers: { crons: ["ENV_CRON_1", "ENV_CRON_2"] },
+          usage_model: "unbound",
+        };
+        const expectedConfig: RawConfig = {
+          name: "NAME",
+          account_id: "ACCOUNT_ID",
+          compatibility_date: "2022-01-01",
+          compatibility_flags: ["FLAG1", "FLAG2"],
+          workers_dev: false,
+          routes: ["ROUTE_1", "ROUTE_2"],
+          jsx_factory: "JSX_FACTORY",
+          jsx_fragment: "JSX_FRAGMENT",
+          triggers: { crons: ["CRON_1", "CRON_2"] },
+          usage_model: "bundled",
+          env: {
+            ENV1: environment,
+          },
+        };
+
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          expectedConfig,
+          undefined
+        );
+
+        expect(config.env.ENV1).toEqual(expect.objectContaining(environment));
+        expect(diagnostics.hasErrors()).toBe(true);
+        expect(diagnostics.hasWarnings()).toBe(false);
+
+        expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+          "Processing wrangler configuration:
+
+            - \\"env.ENV1\\" environment configuration
+              - Expected exactly one of [\\"routes\\",\\"route\\"]."
+        `);
+      });
+
+      it("should error if both route and routes are specified in the top-level environment", () => {
+        const environment: RawEnvironment = {
+          name: "ENV_NAME",
+          account_id: "ENV_ACCOUNT_ID",
+          compatibility_date: "2022-02-02",
+          compatibility_flags: ["ENV_FLAG1", "ENV_FLAG2"],
+          workers_dev: true,
+          routes: ["ENV_ROUTE_1", "ENV_ROUTE_2"],
+          jsx_factory: "ENV_JSX_FACTORY",
+          jsx_fragment: "ENV_JSX_FRAGMENT",
+          triggers: { crons: ["ENV_CRON_1", "ENV_CRON_2"] },
+          usage_model: "unbound",
+        };
+        const expectedConfig: RawConfig = {
+          name: "NAME",
+          account_id: "ACCOUNT_ID",
+          compatibility_date: "2022-01-01",
+          compatibility_flags: ["FLAG1", "FLAG2"],
+          workers_dev: false,
+          route: "ROUTE_1",
+          routes: ["ROUTE_2", "ROUTE_3"],
+          jsx_factory: "JSX_FACTORY",
+          jsx_fragment: "JSX_FRAGMENT",
+          triggers: { crons: ["CRON_1", "CRON_2"] },
+          usage_model: "bundled",
+          env: {
+            ENV1: environment,
+          },
+        };
+
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          expectedConfig,
+          undefined
+        );
+
+        expect(config.env.ENV1).toEqual(expect.objectContaining(environment));
+        expect(diagnostics.hasErrors()).toBe(true);
+        expect(diagnostics.hasWarnings()).toBe(false);
+
+        expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
+          "Processing wrangler configuration:
+            - Expected exactly one of [\\"routes\\",\\"route\\"]."
+        `);
+      });
+
+      it("should not error if <env>.route and <top-level>.routes are specified", () => {
+        const environment: RawEnvironment = {
+          name: "ENV_NAME",
+          account_id: "ENV_ACCOUNT_ID",
+          compatibility_date: "2022-02-02",
+          compatibility_flags: ["ENV_FLAG1", "ENV_FLAG2"],
+          workers_dev: true,
+          route: "ENV_ROUTE_1",
+          jsx_factory: "ENV_JSX_FACTORY",
+          jsx_fragment: "ENV_JSX_FRAGMENT",
+          triggers: { crons: ["ENV_CRON_1", "ENV_CRON_2"] },
+          usage_model: "unbound",
+        };
+        const expectedConfig: RawConfig = {
+          name: "NAME",
+          account_id: "ACCOUNT_ID",
+          compatibility_date: "2022-01-01",
+          compatibility_flags: ["FLAG1", "FLAG2"],
+          workers_dev: false,
+          routes: ["ROUTE_1", "ROUTE_2"],
+          jsx_factory: "JSX_FACTORY",
+          jsx_fragment: "JSX_FRAGMENT",
+          triggers: { crons: ["CRON_1", "CRON_2"] },
+          usage_model: "bundled",
+          env: {
+            ENV1: environment,
+          },
+        };
+
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          expectedConfig,
+          undefined
+        );
+
+        expect(config.env.ENV1).toEqual(expect.objectContaining(environment));
+        expect(diagnostics.hasErrors()).toBe(false);
+        expect(diagnostics.hasWarnings()).toBe(false);
+      });
+
+      it("should not error if <env>.routes and <top-level>.route are specified", () => {
+        const environment: RawEnvironment = {
+          name: "ENV_NAME",
+          account_id: "ENV_ACCOUNT_ID",
+          compatibility_date: "2022-02-02",
+          compatibility_flags: ["ENV_FLAG1", "ENV_FLAG2"],
+          workers_dev: true,
+          routes: ["ENV_ROUTE_1", "ENV_ROUTE_2"],
+          jsx_factory: "ENV_JSX_FACTORY",
+          jsx_fragment: "ENV_JSX_FRAGMENT",
+          triggers: { crons: ["ENV_CRON_1", "ENV_CRON_2"] },
+          usage_model: "unbound",
+        };
+        const expectedConfig: RawConfig = {
+          name: "NAME",
+          account_id: "ACCOUNT_ID",
+          compatibility_date: "2022-01-01",
+          compatibility_flags: ["FLAG1", "FLAG2"],
+          workers_dev: false,
+          route: "ROUTE_1",
+          jsx_factory: "JSX_FACTORY",
+          jsx_fragment: "JSX_FRAGMENT",
+          triggers: { crons: ["CRON_1", "CRON_2"] },
+          usage_model: "bundled",
+          env: {
+            ENV1: environment,
+          },
+        };
+
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          expectedConfig,
+          undefined
+        );
+
+        expect(config.env.ENV1).toEqual(expect.objectContaining(environment));
+        expect(diagnostics.hasErrors()).toBe(false);
+        expect(diagnostics.hasWarnings()).toBe(false);
+      });
+
+      it("should not error if <env1>.route and <env2>.routes are specified", () => {
+        const environment1: RawEnvironment = {
+          name: "ENV_NAME",
+          account_id: "ENV_ACCOUNT_ID",
+          compatibility_date: "2022-02-02",
+          compatibility_flags: ["ENV_FLAG1", "ENV_FLAG2"],
+          workers_dev: true,
+          routes: ["ENV1_ROUTE_1", "ENV2_ROUTE_2"],
+          jsx_factory: "ENV_JSX_FACTORY",
+          jsx_fragment: "ENV_JSX_FRAGMENT",
+          triggers: { crons: ["ENV_CRON_1", "ENV_CRON_2"] },
+          usage_model: "unbound",
+        };
+        const environment2: RawEnvironment = {
+          name: "ENV_NAME",
+          account_id: "ENV_ACCOUNT_ID",
+          compatibility_date: "2022-02-02",
+          compatibility_flags: ["ENV_FLAG1", "ENV_FLAG2"],
+          workers_dev: true,
+          route: "ENV2_ROUTE_1",
+          jsx_factory: "ENV_JSX_FACTORY",
+          jsx_fragment: "ENV_JSX_FRAGMENT",
+          triggers: { crons: ["ENV_CRON_1", "ENV_CRON_2"] },
+          usage_model: "unbound",
+        };
+        const expectedConfig: RawConfig = {
+          name: "NAME",
+          account_id: "ACCOUNT_ID",
+          compatibility_date: "2022-01-01",
+          compatibility_flags: ["FLAG1", "FLAG2"],
+          workers_dev: false,
+          route: "ROUTE_1",
+          jsx_factory: "JSX_FACTORY",
+          jsx_fragment: "JSX_FRAGMENT",
+          triggers: { crons: ["CRON_1", "CRON_2"] },
+          usage_model: "bundled",
+          env: {
+            ENV1: environment1,
+            ENV2: environment2,
+          },
+        };
+
+        const { config, diagnostics } = normalizeAndValidateConfig(
+          expectedConfig,
+          undefined
+        );
+
+        expect(config.env.ENV1).toEqual(expect.objectContaining(environment1));
+        expect(config.env.ENV2).toEqual(expect.objectContaining(environment2));
+        expect(diagnostics.hasErrors()).toBe(false);
+        expect(diagnostics.hasWarnings()).toBe(false);
+      });
+    });
   });
 });