fix: update help.test.ts for printCustomHelp return-based API

printCustomHelp now returns a string instead of accepting a Writer.
Update tests to call without arguments and assert on return value.

Author: BYK

test/lib/help.test.ts

@@ -38,45 +38,18 @@ describe("formatBanner", () => {
 describe("printCustomHelp", () => {
   useTestConfigDir("help-test-");
 
-  test("writes output to the provided writer", async () => {
-    const chunks: string[] = [];
-    const writer = {
-      write: (s: string) => {
-        chunks.push(s);
-        return true;
-      },
-    };
-
-    await printCustomHelp(writer);
-    expect(chunks.length).toBeGreaterThan(0);
-    expect(chunks.join("").length).toBeGreaterThan(0);
+  test("returns non-empty string", async () => {
+    const output = await printCustomHelp();
+    expect(output.length).toBeGreaterThan(0);
   });
 
   test("output contains the tagline", async () => {
-    const chunks: string[] = [];
-    const writer = {
-      write: (s: string) => {
-        chunks.push(s);
-        return true;
-      },
-    };
-
-    await printCustomHelp(writer);
-    const output = stripAnsi(chunks.join(""));
+    const output = stripAnsi(await printCustomHelp());
     expect(output).toContain("The command-line interface for Sentry");
   });
 
   test("output contains registered commands", async () => {
-    const chunks: string[] = [];
-    const writer = {
-      write: (s: string) => {
-        chunks.push(s);
-        return true;
-      },
-    };
-
-    await printCustomHelp(writer);
-    const output = stripAnsi(chunks.join(""));
+    const output = stripAnsi(await printCustomHelp());
 
     // Should include at least some core commands from routes
     expect(output).toContain("sentry");
@@ -87,31 +60,13 @@ describe("printCustomHelp", () => {
   });
 
   test("output contains docs URL", async () => {
-    const chunks: string[] = [];
-    const writer = {
-      write: (s: string) => {
-        chunks.push(s);
-        return true;
-      },
-    };
-
-    await printCustomHelp(writer);
-    const output = stripAnsi(chunks.join(""));
+    const output = stripAnsi(await printCustomHelp());
     expect(output).toContain("cli.sentry.dev");
   });
 
   test("shows login example when not authenticated", async () => {
     // useTestConfigDir provides a clean env with no auth token
-    const chunks: string[] = [];
-    const writer = {
-      write: (s: string) => {
-        chunks.push(s);
-        return true;
-      },
-    };
-
-    await printCustomHelp(writer);
-    const output = stripAnsi(chunks.join(""));
+    const output = stripAnsi(await printCustomHelp());
     expect(output).toContain("sentry auth login");
   });
 });