wrangler 0.0.0-e043479 → 0.0.0-e09abbc

package/src/__tests__/tail.test.ts

@@ -3,297 +3,426 @@ import { Headers, Request } from "undici";
 import { mockAccountId, mockApiToken } from "./helpers/mock-account-id";
 import { setMockResponse, unsetAllMocks } from "./helpers/mock-cfetch";
 import { mockConsoleMethods } from "./helpers/mock-console";
+import { useMockIsTTY } from "./helpers/mock-istty";
 import { runInTempDir } from "./helpers/run-in-tmp";
 import { runWrangler } from "./helpers/run-wrangler";
 import type { TailEventMessage, RequestEvent, ScheduledEvent } from "../tail";
 import type WebSocket from "ws";
 
 describe("tail", () => {
-  runInTempDir();
-  mockAccountId();
-  mockApiToken();
-
-  const std = mockConsoleMethods();
-
-  afterEach(() => {
-    mockWebSockets.forEach((ws) => ws.close());
-    mockWebSockets.splice(0);
-    unsetAllMocks();
-  });
-
-  /**
-   * Interaction with the tailing API, including tail creation,
-   * deletion, and connection.
-   */
-  describe("API interaction", () => {
-    it("creates and then delete tails", async () => {
-      const api = mockWebsocketAPIs();
-      expect(api.requests.creation.count).toStrictEqual(0);
-
-      await runWrangler("tail test-worker");
-
-      await expect(api.ws.connected).resolves.toBeTruthy();
-      expect(api.requests.creation.count).toStrictEqual(1);
-      expect(api.requests.deletion.count).toStrictEqual(0);
-
-      api.ws.close();
-      expect(api.requests.deletion.count).toStrictEqual(1);
-    });
-
-    it("creates and then delete tails: legacy envs", async () => {
-      const api = mockWebsocketAPIs("some-env", true);
-      expect(api.requests.creation.count).toStrictEqual(0);
-
-      await runWrangler("tail test-worker --env some-env --legacy-env true");
-
-      await expect(api.ws.connected).resolves.toBeTruthy();
-      expect(api.requests.creation.count).toStrictEqual(1);
-      expect(api.requests.deletion.count).toStrictEqual(0);
-
-      api.ws.close();
-      expect(api.requests.deletion.count).toStrictEqual(1);
-    });
-
-    it("creates and then delete tails: service envs", async () => {
-      const api = mockWebsocketAPIs("some-env");
-      expect(api.requests.creation.count).toStrictEqual(0);
-
-      await runWrangler("tail test-worker --env some-env --legacy-env false");
-
-      await expect(api.ws.connected).resolves.toBeTruthy();
-      expect(api.requests.creation.count).toStrictEqual(1);
-      expect(api.requests.deletion.count).toStrictEqual(0);
-
-      api.ws.close();
-      expect(api.requests.deletion.count).toStrictEqual(1);
-    });
-
-    it("errors when the websocket closes unexpectedly", async () => {
-      const api = mockWebsocketAPIs();
-      api.ws.close();
-
-      await expect(runWrangler("tail test-worker")).rejects.toThrow();
-    });
-
-    it("activates debug mode when the cli arg is passed in", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --debug");
-      await expect(api.nextMessageJson()).resolves.toHaveProperty(
-        "debug",
-        true
-      );
-    });
-  });
-
-  describe("filtering", () => {
-    it("sends sampling rate filters", async () => {
-      const api = mockWebsocketAPIs();
-      const tooHigh = runWrangler("tail test-worker --sampling-rate 10");
-      await expect(tooHigh).rejects.toThrow();
-
-      const tooLow = runWrangler("tail test-worker --sampling-rate -5");
-      await expect(tooLow).rejects.toThrow();
-
-      await runWrangler("tail test-worker --sampling-rate 0.25");
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { sampling_rate: 0.25 },
-      ]);
-    });
-
-    it("sends single status filters", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --status error");
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { outcome: ["exception", "exceededCpu", "unknown"] },
-      ]);
-    });
-
-    it("sends multiple status filters", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --status error --status canceled");
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { outcome: ["exception", "exceededCpu", "unknown", "canceled"] },
-      ]);
-    });
-
-    it("sends single HTTP method filters", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --method POST");
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { method: ["POST"] },
-      ]);
-    });
-
-    it("sends multiple HTTP method filters", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --method POST --method GET");
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { method: ["POST", "GET"] },
-      ]);
-    });
-
-    it("sends header filters without a query", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --header X-CUSTOM-HEADER ");
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { header: { key: "X-CUSTOM-HEADER" } },
-      ]);
-    });
-
-    it("sends header filters with a query", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler(
-        "tail test-worker --header X-CUSTOM-HEADER:some-value "
-      );
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { header: { key: "X-CUSTOM-HEADER", query: "some-value" } },
-      ]);
-    });
-
-    it("sends single IP filters", async () => {
-      const api = mockWebsocketAPIs();
-      const fakeIp = "192.0.2.1";
-
-      await runWrangler(`tail test-worker --ip ${fakeIp}`);
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { client_ip: [fakeIp] },
-      ]);
-    });
-
-    it("sends multiple IP filters", async () => {
-      const api = mockWebsocketAPIs();
-      const fakeIp = "192.0.2.1";
-
-      await runWrangler(`tail test-worker --ip ${fakeIp} --ip self`);
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { client_ip: [fakeIp, "self"] },
-      ]);
-    });
-
-    it("sends search filters", async () => {
-      const api = mockWebsocketAPIs();
-      const search = "filterMe";
-
-      await runWrangler(`tail test-worker --search ${search}`);
-      await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
-        { query: search },
-      ]);
-    });
-
-    it("sends everything but the kitchen sink", async () => {
-      const api = mockWebsocketAPIs();
-      const sampling_rate = 0.69;
-      const status = ["ok", "error"];
-      const method = ["GET", "POST", "PUT"];
-      const header = "X-HELLO:world";
-      const client_ip = ["192.0.2.1", "self"];
-      const query = "onlyTheseMessagesPlease";
-
-      const cliFilters =
-        `--sampling-rate ${sampling_rate} ` +
-        status.map((s) => `--status ${s} `).join("") +
-        method.map((m) => `--method ${m} `).join("") +
-        `--header ${header} ` +
-        client_ip.map((c) => `--ip ${c} `).join("") +
-        `--search ${query} ` +
-        `--debug`;
-
-      const expectedWebsocketMessage = {
-        filters: [
-          { sampling_rate },
-          { outcome: ["ok", "exception", "exceededCpu", "unknown"] },
-          { method },
-          { header: { key: "X-HELLO", query: "world" } },
-          { client_ip },
-          { query },
-        ],
-        debug: true,
-      };
-
-      await runWrangler(`tail test-worker ${cliFilters}`);
-      await expect(api.nextMessageJson()).resolves.toEqual(
-        expectedWebsocketMessage
-      );
-    });
-  });
-
-  describe("printing", () => {
-    it("logs request messages in JSON format", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --format json");
-
-      const event = generateMockRequestEvent();
-      const message = generateMockEventMessage({ event });
-      const serializedMessage = serialize(message);
-
-      api.ws.send(serializedMessage);
-      expect(std.out).toMatch(deserializeToJson(serializedMessage));
-    });
-
-    it("logs scheduled messages in JSON format", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --format json");
-
-      const event = generateMockScheduledEvent();
-      const message = generateMockEventMessage({ event });
-      const serializedMessage = serialize(message);
-
-      api.ws.send(serializedMessage);
-      expect(std.out).toMatch(deserializeToJson(serializedMessage));
-    });
-
-    it("logs request messages in pretty format", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --format pretty");
-
-      const event = generateMockRequestEvent();
-      const message = generateMockEventMessage({ event });
-      const serializedMessage = serialize(message);
-
-      api.ws.send(serializedMessage);
-      expect(
-        std.out
-          .replace(
-            new Date(mockEventTimestamp).toLocaleString(),
-            "[mock event timestamp]"
-          )
-          .replace(
-            mockTailExpiration.toLocaleString(),
-            "[mock expiration date]"
-          )
-      ).toMatchInlineSnapshot(`
-        "successfully created tail, expires at [mock expiration date]
+	runInTempDir();
+	mockAccountId();
+	mockApiToken();
+
+	const std = mockConsoleMethods();
+
+	afterEach(() => {
+		mockWebSockets.forEach((ws) => ws.close());
+		mockWebSockets.splice(0);
+		unsetAllMocks();
+	});
+
+	/**
+	 * Interaction with the tailing API, including tail creation,
+	 * deletion, and connection.
+	 */
+	describe("API interaction", () => {
+		it("should throw an error if name isn't provided", async () => {
+			await expect(
+				runWrangler("tail")
+			).rejects.toThrowErrorMatchingInlineSnapshot(
+				`"Required Worker name missing. Please specify the Worker name in wrangler.toml, or pass it as an argument with \`wrangler tail <worker-name>\`"`
+			);
+		});
+
+		it("creates and then delete tails", async () => {
+			const api = mockWebsocketAPIs();
+			expect(api.requests.creation.count).toStrictEqual(0);
+
+			await runWrangler("tail test-worker");
+
+			await expect(api.ws.connected).resolves.toBeTruthy();
+			expect(api.requests.creation.count).toStrictEqual(1);
+			expect(api.requests.deletion.count).toStrictEqual(0);
+
+			api.ws.close();
+			expect(api.requests.deletion.count).toStrictEqual(1);
+		});
+
+		it("creates and then delete tails: legacy envs", async () => {
+			const api = mockWebsocketAPIs("some-env", true);
+			expect(api.requests.creation.count).toStrictEqual(0);
+
+			await runWrangler("tail test-worker --env some-env --legacy-env true");
+
+			await expect(api.ws.connected).resolves.toBeTruthy();
+			expect(api.requests.creation.count).toStrictEqual(1);
+			expect(api.requests.deletion.count).toStrictEqual(0);
+
+			api.ws.close();
+			expect(api.requests.deletion.count).toStrictEqual(1);
+		});
+
+		it("creates and then delete tails: service envs", async () => {
+			const api = mockWebsocketAPIs("some-env");
+			expect(api.requests.creation.count).toStrictEqual(0);
+
+			await runWrangler("tail test-worker --env some-env --legacy-env false");
+
+			await expect(api.ws.connected).resolves.toBeTruthy();
+			expect(api.requests.creation.count).toStrictEqual(1);
+			expect(api.requests.deletion.count).toStrictEqual(0);
+
+			api.ws.close();
+			expect(api.requests.deletion.count).toStrictEqual(1);
+		});
+
+		it("errors when the websocket closes unexpectedly", async () => {
+			const api = mockWebsocketAPIs();
+			api.ws.close();
+
+			await expect(runWrangler("tail test-worker")).rejects.toThrow();
+		});
+
+		it("activates debug mode when the cli arg is passed in", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --debug");
+			await expect(api.nextMessageJson()).resolves.toHaveProperty(
+				"debug",
+				true
+			);
+		});
+	});
+
+	describe("filtering", () => {
+		it("sends sampling rate filters", async () => {
+			const api = mockWebsocketAPIs();
+			const tooHigh = runWrangler("tail test-worker --sampling-rate 10");
+			await expect(tooHigh).rejects.toThrow();
+
+			const tooLow = runWrangler("tail test-worker --sampling-rate -5");
+			await expect(tooLow).rejects.toThrow();
+
+			await runWrangler("tail test-worker --sampling-rate 0.25");
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ sampling_rate: 0.25 },
+			]);
+		});
+
+		it("sends single status filters", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --status error");
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ outcome: ["exception", "exceededCpu", "unknown"] },
+			]);
+		});
+
+		it("sends multiple status filters", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --status error --status canceled");
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ outcome: ["exception", "exceededCpu", "unknown", "canceled"] },
+			]);
+		});
+
+		it("sends single HTTP method filters", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --method POST");
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ method: ["POST"] },
+			]);
+		});
+
+		it("sends multiple HTTP method filters", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --method POST --method GET");
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ method: ["POST", "GET"] },
+			]);
+		});
+
+		it("sends header filters without a query", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --header X-CUSTOM-HEADER");
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ header: { key: "X-CUSTOM-HEADER" } },
+			]);
+		});
+
+		it("sends header filters with a query", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --header X-CUSTOM-HEADER:some-value");
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ header: { key: "X-CUSTOM-HEADER", query: "some-value" } },
+			]);
+		});
+
+		it("sends single IP filters", async () => {
+			const api = mockWebsocketAPIs();
+			const fakeIp = "192.0.2.1";
+
+			await runWrangler(`tail test-worker --ip ${fakeIp}`);
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ client_ip: [fakeIp] },
+			]);
+		});
+
+		it("sends multiple IP filters", async () => {
+			const api = mockWebsocketAPIs();
+			const fakeIp = "192.0.2.1";
+
+			await runWrangler(`tail test-worker --ip ${fakeIp} --ip self`);
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ client_ip: [fakeIp, "self"] },
+			]);
+		});
+
+		it("sends search filters", async () => {
+			const api = mockWebsocketAPIs();
+			const search = "filterMe";
+
+			await runWrangler(`tail test-worker --search ${search}`);
+			await expect(api.nextMessageJson()).resolves.toHaveProperty("filters", [
+				{ query: search },
+			]);
+		});
+
+		it("sends everything but the kitchen sink", async () => {
+			const api = mockWebsocketAPIs();
+			const sampling_rate = 0.69;
+			const status = ["ok", "error"];
+			const method = ["GET", "POST", "PUT"];
+			const header = "X-HELLO:world";
+			const client_ip = ["192.0.2.1", "self"];
+			const query = "onlyTheseMessagesPlease";
+
+			const cliFilters =
+				`--sampling-rate ${sampling_rate} ` +
+				status.map((s) => `--status ${s} `).join("") +
+				method.map((m) => `--method ${m} `).join("") +
+				`--header ${header} ` +
+				client_ip.map((c) => `--ip ${c} `).join("") +
+				`--search ${query} ` +
+				`--debug`;
+
+			const expectedWebsocketMessage = {
+				filters: [
+					{ sampling_rate },
+					{ outcome: ["ok", "exception", "exceededCpu", "unknown"] },
+					{ method },
+					{ header: { key: "X-HELLO", query: "world" } },
+					{ client_ip },
+					{ query },
+				],
+				debug: true,
+			};
+
+			await runWrangler(`tail test-worker ${cliFilters}`);
+			await expect(api.nextMessageJson()).resolves.toEqual(
+				expectedWebsocketMessage
+			);
+		});
+	});
+
+	describe("printing", () => {
+		const { setIsTTY } = useMockIsTTY();
+
+		it("logs request messages in JSON format", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --format json");
+
+			const event = generateMockRequestEvent();
+			const message = generateMockEventMessage({ event });
+			const serializedMessage = serialize(message);
+
+			api.ws.send(serializedMessage);
+			expect(std.out).toMatch(deserializeToJson(serializedMessage));
+		});
+
+		it("logs scheduled messages in JSON format", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --format json");
+
+			const event = generateMockScheduledEvent();
+			const message = generateMockEventMessage({ event });
+			const serializedMessage = serialize(message);
+
+			api.ws.send(serializedMessage);
+			expect(std.out).toMatch(deserializeToJson(serializedMessage));
+		});
+
+		it("logs request messages in pretty format", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --format pretty");
+
+			const event = generateMockRequestEvent();
+			const message = generateMockEventMessage({ event });
+			const serializedMessage = serialize(message);
+
+			api.ws.send(serializedMessage);
+			expect(
+				std.out
+					.replace(
+						new Date(mockEventTimestamp).toLocaleString(),
+						"[mock event timestamp]"
+					)
+					.replace(
+						mockTailExpiration.toLocaleString(),
+						"[mock expiration date]"
+					)
+			).toMatchInlineSnapshot(`
+        "Successfully created tail, expires at [mock expiration date]
         Connected to test-worker, waiting for logs...
         GET https://example.org/ - Ok @ [mock event timestamp]"
       `);
-    });
-
-    it("logs scheduled messages in pretty format", async () => {
-      const api = mockWebsocketAPIs();
-      await runWrangler("tail test-worker --format pretty");
-
-      const event = generateMockScheduledEvent();
-      const message = generateMockEventMessage({ event });
-      const serializedMessage = serialize(message);
-
-      api.ws.send(serializedMessage);
-      expect(
-        std.out
-          .replace(
-            new Date(mockEventTimestamp).toLocaleString(),
-            "[mock timestamp string]"
-          )
-          .replace(
-            mockTailExpiration.toLocaleString(),
-            "[mock expiration date]"
-          )
-      ).toMatchInlineSnapshot(`
-        "successfully created tail, expires at [mock expiration date]
+		});
+
+		it("logs scheduled messages in pretty format", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --format pretty");
+
+			const event = generateMockScheduledEvent();
+			const message = generateMockEventMessage({ event });
+			const serializedMessage = serialize(message);
+
+			api.ws.send(serializedMessage);
+			expect(
+				std.out
+					.replace(
+						new Date(mockEventTimestamp).toLocaleString(),
+						"[mock timestamp string]"
+					)
+					.replace(
+						mockTailExpiration.toLocaleString(),
+						"[mock expiration date]"
+					)
+			).toMatchInlineSnapshot(`
+        "Successfully created tail, expires at [mock expiration date]
         Connected to test-worker, waiting for logs...
         \\"* * * * *\\" @ [mock timestamp string] - Ok"
       `);
-    });
-  });
+		});
+
+		it("should not crash when the tail message has a void event", async () => {
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker --format pretty");
+
+			const message = generateMockEventMessage({ event: null });
+			const serializedMessage = serialize(message);
+
+			api.ws.send(serializedMessage);
+			expect(
+				std.out
+					.replace(
+						new Date(mockEventTimestamp).toLocaleString(),
+						"[mock timestamp string]"
+					)
+					.replace(
+						mockTailExpiration.toLocaleString(),
+						"[mock expiration date]"
+					)
+			).toMatchInlineSnapshot(`
+        "Successfully created tail, expires at [mock expiration date]
+        Connected to test-worker, waiting for logs...
+        [missing request] - Ok @ [mock timestamp string]"
+      `);
+		});
+
+		it("defaults to logging in pretty format when the output is a TTY", async () => {
+			setIsTTY(true);
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker");
+
+			const event = generateMockRequestEvent();
+			const message = generateMockEventMessage({ event });
+			const serializedMessage = serialize(message);
+
+			api.ws.send(serializedMessage);
+			expect(
+				std.out
+					.replace(
+						new Date(mockEventTimestamp).toLocaleString(),
+						"[mock event timestamp]"
+					)
+					.replace(
+						mockTailExpiration.toLocaleString(),
+						"[mock expiration date]"
+					)
+			).toMatchInlineSnapshot(`
+        "Successfully created tail, expires at [mock expiration date]
+        Connected to test-worker, waiting for logs...
+        GET https://example.org/ - Ok @ [mock event timestamp]"
+      `);
+		});
+
+		it("defaults to logging in json format when the output is not a TTY", async () => {
+			setIsTTY(false);
+
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker");
+
+			const event = generateMockRequestEvent();
+			const message = generateMockEventMessage({ event });
+			const serializedMessage = serialize(message);
+
+			api.ws.send(serializedMessage);
+			expect(std.out).toMatch(deserializeToJson(serializedMessage));
+		});
+
+		it("logs console messages and exceptions", async () => {
+			setIsTTY(true);
+			const api = mockWebsocketAPIs();
+			await runWrangler("tail test-worker");
+
+			const event = generateMockRequestEvent();
+			const message = generateMockEventMessage({
+				event,
+				logs: [
+					{ message: ["some string"], level: "log", timestamp: 1234561 },
+					{
+						message: [{ complex: "object" }],
+						level: "log",
+						timestamp: 1234562,
+					},
+					{ message: [1234], level: "error", timestamp: 1234563 },
+				],
+				exceptions: [
+					{ name: "Error", message: "some error", timestamp: 1234564 },
+					{ name: "Error", message: { complex: "error" }, timestamp: 1234564 },
+				],
+			});
+			const serializedMessage = serialize(message);
+
+			api.ws.send(serializedMessage);
+			expect(
+				std.out
+					.replace(
+						new Date(mockEventTimestamp).toLocaleString(),
+						"[mock event timestamp]"
+					)
+					.replace(
+						mockTailExpiration.toLocaleString(),
+						"[mock expiration date]"
+					)
+			).toMatchInlineSnapshot(`
+        "Successfully created tail, expires at [mock expiration date]
+        Connected to test-worker, waiting for logs...
+        GET https://example.org/ - Ok @ [mock event timestamp]
+          (log) some string
+          (log) { complex: 'object' }
+          (error) 1234"
+      `);
+			expect(std.err).toMatchInlineSnapshot(`
+        "X [ERROR]   Error: some error
+
+
+        X [ERROR]   Error: { complex: 'error' }
+
+        "
+      `);
+			expect(std.warn).toMatchInlineSnapshot(`""`);
+		});
+	});
 });
 
 /* helpers */
@@ -308,32 +437,33 @@ describe("tail", () => {
  * @returns the same type we expect when deserializing in wrangler
  */
 function serialize(message: TailEventMessage): WebSocket.RawData {
-  if (isScheduled(message.event)) {
-    // `ScheduledEvent`s work just fine
-    const stringified = JSON.stringify(message);
-    return Buffer.from(stringified, "utf-8");
-  } else {
-    // Since the "properties" of an `undici.Request` are actually getters,
-    // which don't serialize properly, we need to hydrate them manually.
-    // This isn't a problem outside of testing since deserialization
-    // works just fine and wrangler never _sends_ any event messages,
-    // it only receives them.
-    const request = (message.event as RequestEvent).request;
-    const stringified = JSON.stringify(message, (key, value) => {
-      if (key !== "request") {
-        return value;
-      }
-
-      return {
-        ...request,
-        url: request.url,
-        headers: request.headers,
-        method: request.method,
-      };
-    });
-
-    return Buffer.from(stringified, "utf-8");
-  }
+	if (isScheduled(message.event)) {
+		// `ScheduledEvent`s work just fine
+		const stringified = JSON.stringify(message);
+		return Buffer.from(stringified, "utf-8");
+	} else {
+		// Since the "properties" of an `undici.Request` are actually getters,
+		// which don't serialize properly, we need to hydrate them manually.
+		// This isn't a problem outside of testing since deserialization
+		// works just fine and wrangler never _sends_ any event messages,
+		// it only receives them.
+		const request = ((message.event as RequestEvent | undefined | null) || {})
+			.request;
+		const stringified = JSON.stringify(message, (key, value) => {
+			if (key !== "request") {
+				return value;
+			}
+
+			return {
+				...request,
+				url: request?.url,
+				headers: request?.headers,
+				method: request?.method,
+			};
+		});
+
+		return Buffer.from(stringified, "utf-8");
+	}
 }
 
 /**
@@ -343,9 +473,9 @@ function serialize(message: TailEventMessage): WebSocket.RawData {
  * @returns whether event is a ScheduledEvent (true) or a RequestEvent
  */
 function isScheduled(
-  event: ScheduledEvent | RequestEvent
+	event: ScheduledEvent | RequestEvent | undefined | null
 ): event is ScheduledEvent {
-  return "cron" in event;
+	return Boolean(event && "cron" in event);
 }
 
 /**
@@ -357,7 +487,7 @@ function isScheduled(
  * @returns a string ready to be printed to the terminal or compared against
  */
 function deserializeToJson(message: WebSocket.RawData): string {
-  return JSON.stringify(JSON.parse(message.toString()), null, 2);
+	return JSON.stringify(JSON.parse(message.toString()), null, 2);
 }
 
 /**
@@ -365,12 +495,12 @@ function deserializeToJson(message: WebSocket.RawData): string {
  * when running `wrangler tail`
  */
 type MockAPI = {
-  requests: {
-    creation: RequestCounter;
-    deletion: RequestCounter;
-  };
-  ws: MockWebSocket;
-  nextMessageJson(): Promise<unknown>;
+	requests: {
+		creation: RequestCounter;
+		deletion: RequestCounter;
+	};
+	ws: MockWebSocket;
+	nextMessageJson(): Promise<unknown>;
 };
 
 /**
@@ -379,7 +509,7 @@ type MockAPI = {
  * the correct API calls without actually sending any web traffic
  */
 type RequestCounter = {
-  count: number;
+	count: number;
 };
 
 /**
@@ -389,34 +519,34 @@ type RequestCounter = {
  * @returns a `RequestCounter` for counting how many times the API is hit
  */
 function mockCreateTailRequest(
-  websocketURL: string,
-  env?: string,
-  legacyEnv = false
+	websocketURL: string,
+	env?: string,
+	legacyEnv = false
 ): RequestCounter {
-  const requests = { count: 0 };
-  const servicesOrScripts = env && !legacyEnv ? "services" : "scripts";
-  const environment = env && !legacyEnv ? "/environments/:envName" : "";
-  setMockResponse(
-    `/accounts/:accountId/workers/${servicesOrScripts}/:scriptName${environment}/tails`,
-    "POST",
-    ([_url, accountId, scriptName, envName]) => {
-      requests.count++;
-      expect(accountId).toEqual("some-account-id");
-      expect(scriptName).toEqual(
-        legacyEnv && env ? `test-worker-${env}` : "test-worker"
-      );
-      if (!legacyEnv) {
-        expect(envName).toEqual(env);
-      }
-      return {
-        id: "tail-id",
-        url: websocketURL,
-        expires_at: mockTailExpiration,
-      };
-    }
-  );
-
-  return requests;
+	const requests = { count: 0 };
+	const servicesOrScripts = env && !legacyEnv ? "services" : "scripts";
+	const environment = env && !legacyEnv ? "/environments/:envName" : "";
+	setMockResponse(
+		`/accounts/:accountId/workers/${servicesOrScripts}/:scriptName${environment}/tails`,
+		"POST",
+		([_url, accountId, scriptName, envName]) => {
+			requests.count++;
+			expect(accountId).toEqual("some-account-id");
+			expect(scriptName).toEqual(
+				legacyEnv && env ? `test-worker-${env}` : "test-worker"
+			);
+			if (!legacyEnv) {
+				expect(envName).toEqual(env);
+			}
+			return {
+				id: "tail-id",
+				url: websocketURL,
+				expires_at: mockTailExpiration,
+			};
+		}
+	);
+
+	return requests;
 }
 
 /**
@@ -435,37 +565,37 @@ const mockEventTimestamp = 1645454470467;
  * @returns a `RequestCounter` for counting how many times the API is hit
  */
 function mockDeleteTailRequest(
-  env?: string,
-  legacyEnv = false
+	env?: string,
+	legacyEnv = false
 ): RequestCounter {
-  const requests = { count: 0 };
-  const servicesOrScripts = env && !legacyEnv ? "services" : "scripts";
-  const environment = env && !legacyEnv ? "/environments/:envName" : "";
-  setMockResponse(
-    // "/accounts/:accountId/workers/scripts/:worker/tails",
-    `/accounts/:accountId/workers/${servicesOrScripts}/:scriptName${environment}/tails/:tailId`,
-    "DELETE",
-    ([_url, accountId, scriptName, envNameOrTailId, tailId]) => {
-      requests.count++;
-      expect(accountId).toEqual("some-account-id");
-      expect(scriptName).toEqual(
-        legacyEnv && env ? `test-worker-${env}` : "test-worker"
-      );
-      if (!legacyEnv) {
-        if (env) {
-          expect(envNameOrTailId).toEqual(env);
-          expect(tailId).toEqual("tail-id");
-        } else {
-          expect(envNameOrTailId).toEqual("tail-id");
-        }
-      } else {
-        expect(envNameOrTailId).toEqual("tail-id");
-      }
-      return null;
-    }
-  );
-
-  return requests;
+	const requests = { count: 0 };
+	const servicesOrScripts = env && !legacyEnv ? "services" : "scripts";
+	const environment = env && !legacyEnv ? "/environments/:envName" : "";
+	setMockResponse(
+		// "/accounts/:accountId/workers/scripts/:worker/tails",
+		`/accounts/:accountId/workers/${servicesOrScripts}/:scriptName${environment}/tails/:tailId`,
+		"DELETE",
+		([_url, accountId, scriptName, envNameOrTailId, tailId]) => {
+			requests.count++;
+			expect(accountId).toEqual("some-account-id");
+			expect(scriptName).toEqual(
+				legacyEnv && env ? `test-worker-${env}` : "test-worker"
+			);
+			if (!legacyEnv) {
+				if (env) {
+					expect(envNameOrTailId).toEqual(env);
+					expect(tailId).toEqual("tail-id");
+				} else {
+					expect(envNameOrTailId).toEqual("tail-id");
+				}
+			} else {
+				expect(envNameOrTailId).toEqual("tail-id");
+			}
+			return null;
+		}
+	);
+
+	return requests;
 }
 
 const mockWebSockets: MockWebSocket[] = [];
@@ -478,30 +608,30 @@ const mockWebSockets: MockWebSocket[] = [];
  * @returns a mocked-out version of the API
  */
 function mockWebsocketAPIs(env?: string, legacyEnv = false): MockAPI {
-  const websocketURL = "ws://localhost:1234";
-  const api: MockAPI = {
-    requests: {
-      deletion: { count: 0 },
-      creation: { count: 0 },
-    },
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-    ws: null!, // will be set in the `beforeEach()` below.
-
-    /**
-     * Parse the next message received by the mock websocket as JSON
-     * @returns JSON.parse of the next message received by the websocket
-     */
-    async nextMessageJson() {
-      const message = await api.ws.nextMessage;
-      return JSON.parse(message as string);
-    },
-  };
-  api.requests.creation = mockCreateTailRequest(websocketURL, env, legacyEnv);
-  api.requests.deletion = mockDeleteTailRequest(env, legacyEnv);
-  api.ws = new MockWebSocket(websocketURL);
-  mockWebSockets.push(api.ws);
-
-  return api;
+	const websocketURL = "ws://localhost:1234";
+	const api: MockAPI = {
+		requests: {
+			deletion: { count: 0 },
+			creation: { count: 0 },
+		},
+		// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+		ws: null!, // will be set in the `beforeEach()` below.
+
+		/**
+		 * Parse the next message received by the mock websocket as JSON
+		 * @returns JSON.parse of the next message received by the websocket
+		 */
+		async nextMessageJson() {
+			const message = await api.ws.nextMessage;
+			return JSON.parse(message as string);
+		},
+	};
+	api.requests.creation = mockCreateTailRequest(websocketURL, env, legacyEnv);
+	api.requests.deletion = mockDeleteTailRequest(env, legacyEnv);
+	api.ws = new MockWebSocket(websocketURL);
+	mockWebSockets.push(api.ws);
+
+	return api;
 }
 
 /**
@@ -511,16 +641,20 @@ function mockWebsocketAPIs(env?: string, legacyEnv = false): MockAPI {
  * @param opts Any specific parts of the message to use instead of defaults
  * @returns a `TailEventMessage` that wrangler can process and display
  */
-function generateMockEventMessage(
-  opts?: Partial<TailEventMessage>
-): TailEventMessage {
-  return {
-    outcome: opts?.outcome || "ok",
-    exceptions: opts?.exceptions || [],
-    logs: opts?.logs || [],
-    eventTimestamp: opts?.eventTimestamp || mockEventTimestamp,
-    event: opts?.event || generateMockRequestEvent(),
-  };
+function generateMockEventMessage({
+	outcome = "ok",
+	exceptions = [],
+	logs = [],
+	eventTimestamp = mockEventTimestamp,
+	event = generateMockRequestEvent(),
+}: Partial<TailEventMessage>): TailEventMessage {
+	return {
+		outcome,
+		exceptions,
+		logs,
+		eventTimestamp,
+		event,
+	};
 }
 
 /**
@@ -531,34 +665,34 @@ function generateMockEventMessage(
  * @returns a `RequestEvent` that can be used within an `EventMessage`
  */
 function generateMockRequestEvent(
-  opts?: Partial<RequestEvent["request"]>
+	opts?: Partial<RequestEvent["request"]>
 ): RequestEvent {
-  return {
-    request: Object.assign(
-      new Request(opts?.url || "https://example.org/", {
-        method: opts?.method || "GET",
-        headers:
-          opts?.headers || new Headers({ "X-EXAMPLE-HEADER": "some_value" }),
-      }),
-      {
-        cf: opts?.cf || {
-          tlsCipher: "AEAD-ENCRYPT-O-MATIC-SHA",
-          tlsVersion: "TLSv2.0",
-          asn: 42069,
-          colo: "ATL",
-          httpProtocol: "HTTP/4",
-          asOrganization: "Cloudflare",
-        },
-      }
-    ),
-  };
+	return {
+		request: Object.assign(
+			new Request(opts?.url || "https://example.org/", {
+				method: opts?.method || "GET",
+				headers:
+					opts?.headers || new Headers({ "X-EXAMPLE-HEADER": "some_value" }),
+			}),
+			{
+				cf: opts?.cf || {
+					tlsCipher: "AEAD-ENCRYPT-O-MATIC-SHA",
+					tlsVersion: "TLSv2.0",
+					asn: 42069,
+					colo: "ATL",
+					httpProtocol: "HTTP/4",
+					asOrganization: "Cloudflare",
+				},
+			}
+		),
+	};
 }
 
 function generateMockScheduledEvent(
-  opts?: Partial<ScheduledEvent>
+	opts?: Partial<ScheduledEvent>
 ): ScheduledEvent {
-  return {
-    cron: opts?.cron || "* * * * *",
-    scheduledTime: opts?.scheduledTime || mockEventTimestamp,
-  };
+	return {
+		cron: opts?.cron || "* * * * *",
+		scheduledTime: opts?.scheduledTime || mockEventTimestamp,
+	};
 }