test: add unit tests for event/profile view positional arg parsing

Export parsePositionalArgs from event/view.ts and profile/view.ts to enable
unit testing. Add comprehensive tests covering:
- Single argument (event ID or transaction only)
- Two arguments (target + required arg)
- Error cases (empty args)
- Edge cases (extra args, empty strings)

Author: BYK

src/commands/event/view.ts

@@ -72,7 +72,7 @@ const USAGE_HINT = "sentry event view <org>/<project> <event-id>";
  *
  * @returns Parsed event ID and optional target arg
  */
-function parsePositionalArgs(args: string[]): {
+export function parsePositionalArgs(args: string[]): {
   eventId: string;
   targetArg: string | undefined;
 } {

src/commands/profile/view.ts

@@ -63,7 +63,7 @@ function parsePeriod(value: string): string {
  *
  * @returns Parsed transaction and optional target arg
  */
-function parsePositionalArgs(args: string[]): {
+export function parsePositionalArgs(args: string[]): {
   transactionRef: string;
   targetArg: string | undefined;
 } {

test/commands/event/view.test.ts

@@ -0,0 +1,87 @@
+/**
+ * Event View Command Tests
+ *
+ * Tests for positional argument parsing in src/commands/event/view.ts
+ */
+
+import { describe, expect, test } from "bun:test";
+import { parsePositionalArgs } from "../../../src/commands/event/view.js";
+import { ContextError } from "../../../src/lib/errors.js";
+
+describe("parsePositionalArgs", () => {
+  describe("single argument (event ID only)", () => {
+    test("parses single arg as event ID", () => {
+      const result = parsePositionalArgs(["abc123def456"]);
+      expect(result.eventId).toBe("abc123def456");
+      expect(result.targetArg).toBeUndefined();
+    });
+
+    test("parses UUID-like event ID", () => {
+      const result = parsePositionalArgs([
+        "a1b2c3d4-e5f6-7890-abcd-ef1234567890",
+      ]);
+      expect(result.eventId).toBe("a1b2c3d4-e5f6-7890-abcd-ef1234567890");
+      expect(result.targetArg).toBeUndefined();
+    });
+
+    test("parses short event ID", () => {
+      const result = parsePositionalArgs(["abc"]);
+      expect(result.eventId).toBe("abc");
+      expect(result.targetArg).toBeUndefined();
+    });
+  });
+
+  describe("two arguments (target + event ID)", () => {
+    test("parses org/project target and event ID", () => {
+      const result = parsePositionalArgs(["my-org/frontend", "abc123def456"]);
+      expect(result.targetArg).toBe("my-org/frontend");
+      expect(result.eventId).toBe("abc123def456");
+    });
+
+    test("parses project-only target and event ID", () => {
+      const result = parsePositionalArgs(["frontend", "abc123def456"]);
+      expect(result.targetArg).toBe("frontend");
+      expect(result.eventId).toBe("abc123def456");
+    });
+
+    test("parses org/ target (all projects) and event ID", () => {
+      const result = parsePositionalArgs(["my-org/", "abc123def456"]);
+      expect(result.targetArg).toBe("my-org/");
+      expect(result.eventId).toBe("abc123def456");
+    });
+  });
+
+  describe("error cases", () => {
+    test("throws ContextError for empty args", () => {
+      expect(() => parsePositionalArgs([])).toThrow(ContextError);
+    });
+
+    test("throws ContextError with usage hint", () => {
+      try {
+        parsePositionalArgs([]);
+        expect.unreachable("Should have thrown");
+      } catch (error) {
+        expect(error).toBeInstanceOf(ContextError);
+        expect((error as ContextError).message).toContain("Event ID");
+      }
+    });
+  });
+
+  describe("edge cases", () => {
+    test("handles more than two args (ignores extras)", () => {
+      const result = parsePositionalArgs([
+        "my-org/frontend",
+        "abc123",
+        "extra-arg",
+      ]);
+      expect(result.targetArg).toBe("my-org/frontend");
+      expect(result.eventId).toBe("abc123");
+    });
+
+    test("handles empty string event ID in two-arg case", () => {
+      const result = parsePositionalArgs(["my-org/frontend", ""]);
+      expect(result.targetArg).toBe("my-org/frontend");
+      expect(result.eventId).toBe("");
+    });
+  });
+});

test/commands/profile/view.test.ts

@@ -0,0 +1,103 @@
+/**
+ * Profile View Command Tests
+ *
+ * Tests for positional argument parsing in src/commands/profile/view.ts
+ */
+
+import { describe, expect, test } from "bun:test";
+import { parsePositionalArgs } from "../../../src/commands/profile/view.js";
+import { ContextError } from "../../../src/lib/errors.js";
+
+describe("parsePositionalArgs", () => {
+  describe("single argument (transaction only)", () => {
+    test("parses single arg as transaction name", () => {
+      const result = parsePositionalArgs(["/api/users"]);
+      expect(result.transactionRef).toBe("/api/users");
+      expect(result.targetArg).toBeUndefined();
+    });
+
+    test("parses transaction index", () => {
+      const result = parsePositionalArgs(["1"]);
+      expect(result.transactionRef).toBe("1");
+      expect(result.targetArg).toBeUndefined();
+    });
+
+    test("parses transaction alias", () => {
+      const result = parsePositionalArgs(["a"]);
+      expect(result.transactionRef).toBe("a");
+      expect(result.targetArg).toBeUndefined();
+    });
+
+    test("parses complex transaction name", () => {
+      const result = parsePositionalArgs(["POST /api/v2/users/:id/settings"]);
+      expect(result.transactionRef).toBe("POST /api/v2/users/:id/settings");
+      expect(result.targetArg).toBeUndefined();
+    });
+  });
+
+  describe("two arguments (target + transaction)", () => {
+    test("parses org/project target and transaction name", () => {
+      const result = parsePositionalArgs(["my-org/backend", "/api/users"]);
+      expect(result.targetArg).toBe("my-org/backend");
+      expect(result.transactionRef).toBe("/api/users");
+    });
+
+    test("parses project-only target and transaction", () => {
+      const result = parsePositionalArgs(["backend", "/api/users"]);
+      expect(result.targetArg).toBe("backend");
+      expect(result.transactionRef).toBe("/api/users");
+    });
+
+    test("parses org/ target (all projects) and transaction", () => {
+      const result = parsePositionalArgs(["my-org/", "/api/users"]);
+      expect(result.targetArg).toBe("my-org/");
+      expect(result.transactionRef).toBe("/api/users");
+    });
+
+    test("parses target and transaction index", () => {
+      const result = parsePositionalArgs(["my-org/backend", "1"]);
+      expect(result.targetArg).toBe("my-org/backend");
+      expect(result.transactionRef).toBe("1");
+    });
+
+    test("parses target and transaction alias", () => {
+      const result = parsePositionalArgs(["my-org/backend", "a"]);
+      expect(result.targetArg).toBe("my-org/backend");
+      expect(result.transactionRef).toBe("a");
+    });
+  });
+
+  describe("error cases", () => {
+    test("throws ContextError for empty args", () => {
+      expect(() => parsePositionalArgs([])).toThrow(ContextError);
+    });
+
+    test("throws ContextError with usage hint", () => {
+      try {
+        parsePositionalArgs([]);
+        expect.unreachable("Should have thrown");
+      } catch (error) {
+        expect(error).toBeInstanceOf(ContextError);
+        expect((error as ContextError).message).toContain("Transaction");
+      }
+    });
+  });
+
+  describe("edge cases", () => {
+    test("handles more than two args (ignores extras)", () => {
+      const result = parsePositionalArgs([
+        "my-org/backend",
+        "/api/users",
+        "extra-arg",
+      ]);
+      expect(result.targetArg).toBe("my-org/backend");
+      expect(result.transactionRef).toBe("/api/users");
+    });
+
+    test("handles empty string transaction in two-arg case", () => {
+      const result = parsePositionalArgs(["my-org/backend", ""]);
+      expect(result.targetArg).toBe("my-org/backend");
+      expect(result.transactionRef).toBe("");
+    });
+  });
+});