@zapier/zapier-sdk-cli 0.16.1 → 0.16.3

package/src/utils/cli-generator.test.ts

@@ -1,347 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
-import chalk from "chalk";
-import { generateCliCommands } from "./cli-generator";
-import { Command } from "commander";
-import type { ZapierSdk } from "@zapier/zapier-sdk";
-import { z } from "zod";
-
-// We need to create a test for the formatItemsGeneric function that's defined in cli-generator.ts
-// Since it's not exported, we'll create a simple test that verifies the numbering logic
-
-interface TestItem {
-  name?: string;
-  key?: string;
-  id?: string;
-}
-
-describe("CLI Generator Pagination Numbering", () => {
-  let mockConsoleLog: ReturnType<typeof vi.spyOn>;
-
-  beforeEach(() => {
-    mockConsoleLog = vi.spyOn(console, "log").mockImplementation(() => {});
-  });
-
-  afterEach(() => {
-    mockConsoleLog.mockRestore();
-  });
-
-  // Since formatItemsGeneric is not exported, let's create a regression test
-  // that tests the key functionality we care about: continuous numbering
-  it("should prevent pagination numbering regression by testing chalk formatting", () => {
-    // Simulate what formatItemsGeneric does for pagination numbering
-    const simulateFormatting = (items: TestItem[], startingNumber: number) => {
-      const calls: string[] = [];
-      items.forEach((item, index) => {
-        const name = item.name || item.key || item.id || "Item";
-        const formatted = `${chalk.gray(`${startingNumber + index + 1}.`)} ${chalk.cyan(String(name))}`;
-        calls.push(formatted);
-      });
-      return calls;
-    };
-
-    // Test first page (items 1-3)
-    const page1Items = [
-      { name: "App 1", key: "app1" },
-      { name: "App 2", key: "app2" },
-      { name: "App 3", key: "app3" },
-    ];
-
-    const page2Items = [
-      { name: "App 4", key: "app4" },
-      { name: "App 5", key: "app5" },
-    ];
-
-    // Format first page starting from 0
-    const page1Output = simulateFormatting(page1Items, 0);
-
-    // Format second page starting from 3 (3 items shown already)
-    const page2Output = simulateFormatting(page2Items, 3);
-
-    // Check that page 1 contains 1., 2., 3.
-    expect(page1Output.some((line) => line.includes("1."))).toBe(true);
-    expect(page1Output.some((line) => line.includes("2."))).toBe(true);
-    expect(page1Output.some((line) => line.includes("3."))).toBe(true);
-
-    // Check that page 2 contains 4., 5. (continuing from page 1)
-    expect(page2Output.some((line) => line.includes("4."))).toBe(true);
-    expect(page2Output.some((line) => line.includes("5."))).toBe(true);
-
-    // Most importantly: page 2 should NOT restart at 1.
-    expect(page2Output.some((line) => line.includes("1."))).toBe(false);
-    expect(page2Output.some((line) => line.includes("2."))).toBe(false);
-    expect(page2Output.some((line) => line.includes("3."))).toBe(false);
-  });
-
-  it("should handle edge case of starting number 0", () => {
-    const simulateFormatting = (
-      items: TestItem[],
-      startingNumber: number = 0,
-    ) => {
-      const calls: string[] = [];
-      items.forEach((item, index) => {
-        const name = item.name || item.key || item.id || "Item";
-        const formatted = `${chalk.gray(`${startingNumber + index + 1}.`)} ${chalk.cyan(String(name))}`;
-        calls.push(formatted);
-      });
-      return calls;
-    };
-
-    const items = [{ name: "First Item" }, { name: "Second Item" }];
-
-    // Default startingNumber should be 0, resulting in 1. and 2.
-    const output = simulateFormatting(items, 0);
-
-    expect(output.some((line) => line.includes("1."))).toBe(true);
-    expect(output.some((line) => line.includes("2."))).toBe(true);
-  });
-});
-
-// Create proper Zod schema for testing
-const testSchema = z
-  .object({
-    maxItems: z.number().optional(),
-    pageSize: z.number().optional(),
-  })
-  .describe("List all apps");
-
-// Mock SDK for CLI tests
-const mockSdk = {
-  getRegistry: vi.fn(() => ({
-    functions: [
-      {
-        name: "listApps",
-        type: "list",
-        inputSchema: testSchema,
-      },
-    ],
-    categories: [],
-  })),
-  listApps: vi.fn().mockResolvedValue({ data: [] }),
-} as unknown as ZapierSdk;
-
-// Schema for request command testing
-const requestSchema = z
-  .object({
-    url: z.string().describe("The URL to request"),
-    method: z.string().optional().describe("HTTP method"),
-    authenticationId: z.number().optional(),
-  })
-  .describe("Make authenticated HTTP requests");
-
-describe("CLI Response Body Handling", () => {
-  let consoleSpy: ReturnType<typeof vi.spyOn>;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
-  });
-
-  afterEach(() => {
-    consoleSpy.mockRestore();
-  });
-
-  it("should wrap Response objects in structured envelope with statusCode, headers, body", async () => {
-    const bodyData = { ok: true, users: [{ id: 1, name: "Test User" }] };
-
-    // Create a mock Response object
-    const mockResponse = new Response(JSON.stringify(bodyData), {
-      status: 200,
-      headers: { "Content-Type": "application/json" },
-    });
-
-    const mockSdkWithRequest = {
-      getRegistry: vi.fn(() => ({
-        functions: [
-          {
-            name: "request",
-            type: "single",
-            inputSchema: requestSchema,
-          },
-        ],
-        categories: [],
-      })),
-      request: vi.fn().mockResolvedValue(mockResponse),
-    } as unknown as ZapierSdk;
-
-    const program = new Command();
-    program.exitOverride();
-
-    generateCliCommands(program, mockSdkWithRequest);
-
-    await program.parseAsync([
-      "node",
-      "test",
-      "request",
-      "https://api.example.com/users",
-      "--json",
-    ]);
-
-    // Verify the SDK method was called
-    expect(mockSdkWithRequest.request).toHaveBeenCalled();
-
-    // Verify output includes the structured envelope with statusCode, headers, and body
-    const expectedEnvelope = {
-      statusCode: 200,
-      headers: { "content-type": "application/json" },
-      body: bodyData,
-    };
-    expect(consoleSpy).toHaveBeenCalledWith(
-      JSON.stringify(expectedEnvelope, null, 2),
-    );
-  });
-
-  it("should handle invalid JSON in Response with helpful error", async () => {
-    // Create a mock Response with invalid JSON
-    const mockResponse = new Response("not valid json {", {
-      status: 200,
-      headers: { "Content-Type": "application/json" },
-    });
-
-    const mockSdkWithBadResponse = {
-      getRegistry: vi.fn(() => ({
-        functions: [
-          {
-            name: "request",
-            type: "single",
-            inputSchema: requestSchema,
-          },
-        ],
-        categories: [],
-      })),
-      request: vi.fn().mockResolvedValue(mockResponse),
-    } as unknown as ZapierSdk;
-
-    const program = new Command();
-    program.exitOverride();
-    const consoleErrorSpy = vi
-      .spyOn(console, "error")
-      .mockImplementation(() => {});
-
-    generateCliCommands(program, mockSdkWithBadResponse);
-
-    try {
-      await program.parseAsync([
-        "node",
-        "test",
-        "request",
-        "https://api.example.com/users",
-        "--json",
-      ]);
-    } catch {
-      // Expected to throw
-    }
-
-    // Verify error message includes helpful context
-    expect(consoleErrorSpy).toHaveBeenCalledWith(
-      expect.stringContaining("❌"),
-      expect.stringContaining("Failed to parse response as JSON"),
-    );
-
-    consoleErrorSpy.mockRestore();
-  });
-
-  it("should handle non-Response objects normally", async () => {
-    const regularData = { apps: [{ id: 1, name: "Test App" }] };
-
-    const mockSdkRegular = {
-      getRegistry: vi.fn(() => ({
-        functions: [
-          {
-            name: "getApp",
-            type: "single",
-            inputSchema: z.object({ appKey: z.string() }).describe("Get app"),
-          },
-        ],
-        categories: [],
-      })),
-      getApp: vi.fn().mockResolvedValue(regularData),
-    } as unknown as ZapierSdk;
-
-    const program = new Command();
-    program.exitOverride();
-
-    generateCliCommands(program, mockSdkRegular);
-
-    await program.parseAsync(["node", "test", "get-app", "test-app", "--json"]);
-
-    expect(mockSdkRegular.getApp).toHaveBeenCalled();
-    expect(consoleSpy).toHaveBeenCalledWith(
-      JSON.stringify(regularData, null, 2),
-    );
-  });
-});
-
-describe("CLI Command Generation", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it("should execute CLI commands successfully", async () => {
-    const program = new Command();
-    program.exitOverride(); // Prevent process.exit() in tests
-
-    // Generate CLI commands
-    generateCliCommands(program, mockSdk);
-
-    // Execute the command through Commander.js parsing
-    await program.parseAsync(["node", "test", "list-apps", "--json"]);
-
-    // Verify the SDK method was called
-    expect(mockSdk.listApps).toHaveBeenCalled();
-  });
-
-  it("should handle CLI command failures gracefully", async () => {
-    const program = new Command();
-    program.exitOverride(); // Prevent process.exit() in tests
-    const consoleSpy = vi.spyOn(console, "error").mockImplementation(() => {});
-
-    // Generate CLI commands
-    generateCliCommands(program, mockSdk);
-
-    // Mock the SDK method to throw an error
-    (mockSdk as unknown).listApps = vi
-      .fn()
-      .mockRejectedValue(new Error("API error"));
-
-    try {
-      // Execute the command through Commander.js parsing
-      await program.parseAsync(["node", "test", "list-apps", "--json"]);
-    } catch {
-      // Expected due to exitOverride throwing CommanderError
-    }
-
-    // Verify the SDK method was attempted
-    expect(mockSdk.listApps).toHaveBeenCalled();
-
-    // Cleanup
-    consoleSpy.mockRestore();
-  });
-
-  it("should generate commands for SDK without event emission", async () => {
-    const mockSdkWithoutEvents = {
-      getRegistry: vi.fn(() => ({
-        functions: [
-          {
-            name: "listApps",
-            type: "list",
-            inputSchema: testSchema,
-          },
-        ],
-        categories: [],
-      })),
-      listApps: vi.fn().mockResolvedValue({ data: [] }),
-    } as unknown as ZapierSdk;
-
-    const program = new Command();
-    program.exitOverride(); // Prevent process.exit() in tests
-
-    // Generate CLI commands
-    generateCliCommands(program, mockSdkWithoutEvents);
-
-    // Should not throw even without event emission capability
-    await program.parseAsync(["node", "test", "list-apps", "--json"]);
-
-    // Verify the command executed successfully
-    expect(mockSdkWithoutEvents.listApps).toHaveBeenCalled();
-  });
-});