wrangler 0.0.0-e88a54e → 0.0.0-e89ee2a5

package/src/__tests__/sentry.test.ts

@@ -1,146 +0,0 @@
-import * as fs from "node:fs";
-import * as fsp from "node:fs/promises";
-import os from "node:os";
-import path from "node:path";
-import * as TOML from "@iarna/toml";
-import * as Sentry from "@sentry/node";
-import prompts from "prompts";
-
-import { mockConsoleMethods } from "./helpers/mock-console";
-import { runInTempDir } from "./helpers/run-in-tmp";
-import { runWrangler } from "./helpers/run-wrangler";
-const { reportError } = jest.requireActual("../reporting");
-
-describe("Error Reporting", () => {
-  runInTempDir({ homedir: "./home" });
-  mockConsoleMethods();
-  const reportingTOMLPath = ".wrangler/config/reporting.toml";
-
-  const originalTTY = process.stdout.isTTY;
-  beforeEach(() => {
-    jest.mock("@sentry/node");
-    jest.spyOn(Sentry, "captureException");
-    process.stdout.isTTY = true;
-  });
-
-  afterEach(() => {
-    jest.unmock("@sentry/node");
-    jest.clearAllMocks();
-    process.stdout.isTTY = originalTTY;
-  });
-
-  it("should confirm user will allow error reporting usage", async () => {
-    jest.spyOn(prompts, "prompt").mockResolvedValue({ sentryDecision: true });
-
-    await reportError(new Error("test error"), "testFalse");
-
-    const { error_tracking_opt, error_tracking_opt_date } = TOML.parse(
-      await fsp.readFile(path.join(os.homedir(), reportingTOMLPath), "utf-8")
-    );
-
-    expect(error_tracking_opt).toBe(true);
-    expect(error_tracking_opt_date).toBeTruthy();
-
-    expect(Sentry.captureException).toHaveBeenCalledWith(
-      new Error("test error")
-    );
-  });
-
-  it("should confirm user will disallow error reporting usage", async () => {
-    jest.spyOn(prompts, "prompt").mockResolvedValue({ sentryDecision: false });
-    await reportError(new Error("test error"), "testFalse");
-
-    const { error_tracking_opt, error_tracking_opt_date } = TOML.parse(
-      await fsp.readFile(path.join(os.homedir(), reportingTOMLPath), "utf-8")
-    );
-
-    expect(error_tracking_opt).toBe(false);
-    expect(error_tracking_opt_date).toBeTruthy();
-
-    expect(Sentry.captureException).not.toHaveBeenCalledWith(
-      new Error("test error"),
-      "testFalse"
-    );
-  });
-
-  it("should confirm user will allow 'once' error reporting usage", async () => {
-    jest.spyOn(prompts, "prompt").mockResolvedValue({ sentryDecision: "once" });
-    await runWrangler();
-    await reportError(new Error("test error"), "testFalse");
-
-    expect(fs.existsSync(path.join(os.homedir(), reportingTOMLPath))).toBe(
-      false
-    );
-
-    expect(Sentry.captureException).not.toHaveBeenCalledWith(
-      new Error("test error"),
-      "testFalse"
-    );
-  });
-
-  it("should not prompt w/ subsequent Errors after user disallows error reporting", async () => {
-    jest.spyOn(prompts, "prompt").mockResolvedValue({ sentryDecision: false });
-    await reportError(new Error("test error"), "testFalse");
-
-    expect(Sentry.captureException).not.toHaveBeenCalledWith(
-      new Error("test error"),
-      "testFalse"
-    );
-
-    await reportError(new Error("second test error"), "testFalse");
-    expect(Sentry.captureException).not.toHaveBeenCalledWith(
-      new Error("second test error"),
-      "testFalse"
-    );
-  });
-
-  it("should not prompt w/ subsequent Errors after user Always allows error reporting", async () => {
-    jest.spyOn(prompts, "prompt").mockResolvedValue({ sentryDecision: true });
-    await reportError(new Error("test error"), "testFalse");
-
-    expect(Sentry.captureException).toHaveBeenCalledWith(
-      new Error("test error")
-    );
-
-    await reportError(new Error("second test error"), "testFalse");
-    expect(Sentry.captureException).toHaveBeenCalledWith(
-      new Error("second test error")
-    );
-  });
-
-  it("should prompt w/ subsequent Errors after user allows error reporting once", async () => {
-    const promptSpy = jest
-      .spyOn(prompts, "prompt")
-      .mockResolvedValue({ sentryDecision: "once" });
-    await reportError(new Error("test error"), "testFalse");
-
-    expect(Sentry.captureException).toHaveBeenCalledWith(
-      new Error("test error")
-    );
-
-    await reportError(new Error("second test error"), "testFalse");
-    expect(Sentry.captureException).toHaveBeenCalledWith(
-      new Error("second test error")
-    );
-
-    expect(promptSpy).toBeCalledTimes(2);
-  });
-
-  it("should not prompt in non-TTY environment", async () => {
-    process.stdout.isTTY = false;
-
-    await reportError(new Error("test error"), "testFalse");
-
-    const { error_tracking_opt, error_tracking_opt_date } = TOML.parse(
-      await fsp.readFile(path.join(os.homedir(), reportingTOMLPath), "utf-8")
-    );
-
-    expect(error_tracking_opt).toBe(false);
-    expect(error_tracking_opt_date).toBeTruthy();
-
-    expect(Sentry.captureException).not.toHaveBeenCalledWith(
-      new Error("test error")
-    );
-    process.stdout.isTTY = originalTTY;
-  });
-});

package/src/__tests__/tail.test.ts

@@ -1,564 +0,0 @@
-import MockWebSocket from "jest-websocket-mock";
-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 { 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]
-        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]
-        Connected to test-worker, waiting for logs...
-        \\"* * * * *\\" @ [mock timestamp string] - Ok"
-      `);
-    });
-  });
-});
-
-/* helpers */
-
-/**
- * The built in serialize-to-JSON feature of our mock websocket doesn't work
- * for our use-case since we actually expect a raw buffer,
- * not a Javascript string. Additionally, we have to do some fiddling
- * with `RequestEvent`s to get them to serialize properly.
- *
- * @param message a message to serialize to JSON
- * @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");
-  }
-}
-
-/**
- * Small helper to disambiguate the event types possible in a `TailEventMessage`
- *
- * @param event A TailEvent
- * @returns whether event is a ScheduledEvent (true) or a RequestEvent
- */
-function isScheduled(
-  event: ScheduledEvent | RequestEvent
-): event is ScheduledEvent {
-  return "cron" in event;
-}
-
-/**
- * Similarly, we need to deserialize from a raw buffer instead
- * of just JSON.parsing a raw string. This deserializer also then
- * re-stringifies with some spacing, the same way wrangler tail does.
- *
- * @param message a buffer of data received from the websocket
- * @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);
-}
-
-/**
- * A mock for all the different API resources wrangler accesses
- * when running `wrangler tail`
- */
-type MockAPI = {
-  requests: {
-    creation: RequestCounter;
-    deletion: RequestCounter;
-  };
-  ws: MockWebSocket;
-  nextMessageJson(): Promise<unknown>;
-};
-
-/**
- * A counter used to check how many times a mock API has been hit.
- * Useful as a helper in our testing to check if wrangler is making
- * the correct API calls without actually sending any web traffic
- */
-type RequestCounter = {
-  count: number;
-};
-
-/**
- * Mock out the API hit during Tail creation
- *
- * @param websocketURL a fake URL for wrangler to connect a websocket to
- * @returns a `RequestCounter` for counting how many times the API is hit
- */
-function mockCreateTailRequest(
-  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;
-}
-
-/**
- * Mock expiration datetime for tails created during testing
- */
-const mockTailExpiration = new Date(3005, 1);
-
-/**
- * Default value for event timestamps
- */
-const mockEventTimestamp = 1645454470467;
-
-/**
- * Mock out the API hit during Tail deletion
- *
- * @returns a `RequestCounter` for counting how many times the API is hit
- */
-function mockDeleteTailRequest(
-  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 mockWebSockets: MockWebSocket[] = [];
-
-/**
- * All-in-one convenience method to mock the appropriate API calls before
- * each test, and clean up afterwards.
- *
- * @param websocketURL a fake websocket URL for wrangler to connect to
- * @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;
-}
-
-/**
- * Generate a mock `TailEventMessage` of the same shape sent back by the
- * tail worker.
- *
- * @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(),
-  };
-}
-
-/**
- * Generate a mock `RequestEvent` that, in an alternate timeline, was used
- * to trigger a worker. You can't disprove this!
- *
- * @param opts Any specific parts of the event to use instead of defaults
- * @returns a `RequestEvent` that can be used within an `EventMessage`
- */
-function generateMockRequestEvent(
-  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",
-        },
-      }
-    ),
-  };
-}
-
-function generateMockScheduledEvent(
-  opts?: Partial<ScheduledEvent>
-): ScheduledEvent {
-  return {
-    cron: opts?.cron || "* * * * *",
-    scheduledTime: opts?.scheduledTime || mockEventTimestamp,
-  };
-}