@akiojin/gwt 2.2.0 → 2.4.0

package/src/cli/ui/__tests__/components/common/Confirm.test.tsx

@@ -1,13 +1,13 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { Confirm } from '../../../components/common/Confirm.js';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { Confirm } from "../../../components/common/Confirm.js";
+import { Window } from "happy-dom";
 
-describe('Confirm', () => {
+describe("Confirm", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -15,47 +15,60 @@ describe('Confirm', () => {
     globalThis.document = window.document as any;
   });
 
-  it('should render the message', () => {
+  it("should render the message", () => {
     const onConfirm = vi.fn();
-    const { getByText } = render(<Confirm message="Are you sure?" onConfirm={onConfirm} />);
+    const { getByText } = render(
+      <Confirm message="Are you sure?" onConfirm={onConfirm} />,
+    );
 
-    expect(getByText('Are you sure?')).toBeDefined();
+    expect(getByText("Are you sure?")).toBeDefined();
   });
 
-  it('should render Yes and No options', () => {
+  it("should render Yes and No options", () => {
     const onConfirm = vi.fn();
-    const { getByText } = render(<Confirm message="Continue?" onConfirm={onConfirm} />);
+    const { getByText } = render(
+      <Confirm message="Continue?" onConfirm={onConfirm} />,
+    );
 
-    expect(getByText('Yes')).toBeDefined();
-    expect(getByText('No')).toBeDefined();
+    expect(getByText("Yes")).toBeDefined();
+    expect(getByText("No")).toBeDefined();
   });
 
-  it('should render custom Yes and No labels', () => {
+  it("should render custom Yes and No labels", () => {
     const onConfirm = vi.fn();
     const { getByText } = render(
-      <Confirm message="Delete?" onConfirm={onConfirm} yesLabel="Confirm" noLabel="Cancel" />
+      <Confirm
+        message="Delete?"
+        onConfirm={onConfirm}
+        yesLabel="Confirm"
+        noLabel="Cancel"
+      />,
     );
 
-    expect(getByText('Confirm')).toBeDefined();
-    expect(getByText('Cancel')).toBeDefined();
+    expect(getByText("Confirm")).toBeDefined();
+    expect(getByText("Cancel")).toBeDefined();
   });
 
-  it('should default to Yes option', () => {
+  it("should default to Yes option", () => {
     const onConfirm = vi.fn();
-    const { container } = render(<Confirm message="Continue?" onConfirm={onConfirm} />);
+    const { container } = render(
+      <Confirm message="Continue?" onConfirm={onConfirm} />,
+    );
 
     // Verify component renders without error
     expect(container).toBeDefined();
   });
 
-  it('should accept defaultNo prop to default to No', () => {
+  it("should accept defaultNo prop to default to No", () => {
     const onConfirm = vi.fn();
-    const { container } = render(<Confirm message="Continue?" onConfirm={onConfirm} defaultNo />);
+    const { container } = render(
+      <Confirm message="Continue?" onConfirm={onConfirm} defaultNo />,
+    );
 
     expect(container).toBeDefined();
   });
 
-  it('should call onConfirm with false by default when rendered', () => {
+  it("should call onConfirm with false by default when rendered", () => {
     const onConfirm = vi.fn();
     render(<Confirm message="Continue?" onConfirm={onConfirm} />);
 

package/src/cli/ui/__tests__/components/common/ErrorBoundary.test.tsx

@@ -1,22 +1,22 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { ErrorBoundary } from '../../../components/common/ErrorBoundary.js';
-import { Text, Box } from 'ink';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { ErrorBoundary } from "../../../components/common/ErrorBoundary.js";
+import { Text, Box } from "ink";
+import { Window } from "happy-dom";
 
 // Component that throws an error
 const ThrowError = ({ shouldThrow }: { shouldThrow: boolean }) => {
   if (shouldThrow) {
-    throw new Error('Test error message');
+    throw new Error("Test error message");
   }
   return <Text>No error</Text>;
 };
 
-describe('ErrorBoundary', () => {
+describe("ErrorBoundary", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -24,31 +24,31 @@ describe('ErrorBoundary', () => {
     globalThis.document = window.document as any;
 
     // Suppress console.error for expected errors in tests
-    vi.spyOn(console, 'error').mockImplementation(() => {});
+    vi.spyOn(console, "error").mockImplementation(() => {});
   });
 
-  it('should render children when no error occurs', () => {
+  it("should render children when no error occurs", () => {
     const { getByText } = render(
       <ErrorBoundary>
         <ThrowError shouldThrow={false} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
-    expect(getByText('No error')).toBeDefined();
+    expect(getByText("No error")).toBeDefined();
   });
 
-  it('should catch errors and display error message', () => {
+  it("should catch errors and display error message", () => {
     const { getByText } = render(
       <ErrorBoundary>
         <ThrowError shouldThrow={true} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     expect(getByText(/Error:/)).toBeDefined();
     expect(getByText(/Test error message/)).toBeDefined();
   });
 
-  it('should display custom fallback when provided', () => {
+  it("should display custom fallback when provided", () => {
     const CustomFallback = ({ error }: { error: Error }) => (
       <Box>
         <Text color="red">Custom Error: {error.message}</Text>
@@ -58,18 +58,18 @@ describe('ErrorBoundary', () => {
     const { getByText } = render(
       <ErrorBoundary fallback={CustomFallback}>
         <ThrowError shouldThrow={true} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     expect(getByText(/Custom Error:/)).toBeDefined();
     expect(getByText(/Test error message/)).toBeDefined();
   });
 
-  it('should reset error state when children change', () => {
+  it("should reset error state when children change", () => {
     const { rerender, getByText } = render(
       <ErrorBoundary>
         <ThrowError shouldThrow={true} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     // Error is shown
@@ -79,14 +79,14 @@ describe('ErrorBoundary', () => {
     rerender(
       <ErrorBoundary>
         <ThrowError shouldThrow={false} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     // Original children should be rendered
-    expect(getByText('No error')).toBeDefined();
+    expect(getByText("No error")).toBeDefined();
   });
 
-  it('should handle errors with no message', () => {
+  it("should handle errors with no message", () => {
     const ThrowNoMessage = () => {
       throw new Error();
     };
@@ -94,7 +94,7 @@ describe('ErrorBoundary', () => {
     const { getByText } = render(
       <ErrorBoundary>
         <ThrowNoMessage />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     expect(getByText(/Error:/)).toBeDefined();

package/src/cli/ui/__tests__/components/common/Input.test.tsx

@@ -1,13 +1,13 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { Input } from '../../../components/common/Input.js';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { Input } from "../../../components/common/Input.js";
+import { Window } from "happy-dom";
 
-describe('Input', () => {
+describe("Input", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -15,35 +15,42 @@ describe('Input', () => {
     globalThis.document = window.document as any;
   });
 
-  it('should render with value', () => {
+  it("should render with value", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
-    const { container } = render(<Input value="test" onChange={onChange} onSubmit={onSubmit} />);
+    const { container } = render(
+      <Input value="test" onChange={onChange} onSubmit={onSubmit} />,
+    );
 
     expect(container).toBeDefined();
   });
 
-  it('should render with placeholder', () => {
+  it("should render with placeholder", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     const { getByText } = render(
-      <Input value="" onChange={onChange} onSubmit={onSubmit} placeholder="Enter text..." />
+      <Input
+        value=""
+        onChange={onChange}
+        onSubmit={onSubmit}
+        placeholder="Enter text..."
+      />,
     );
 
-    expect(getByText('Enter text...')).toBeDefined();
+    expect(getByText("Enter text...")).toBeDefined();
   });
 
-  it('should render with label', () => {
+  it("should render with label", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     const { getByText } = render(
-      <Input value="" onChange={onChange} onSubmit={onSubmit} label="Name:" />
+      <Input value="" onChange={onChange} onSubmit={onSubmit} label="Name:" />,
     );
 
-    expect(getByText('Name:')).toBeDefined();
+    expect(getByText("Name:")).toBeDefined();
   });
 
-  it('should render label and placeholder together', () => {
+  it("should render label and placeholder together", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     const { getByText } = render(
@@ -53,24 +60,24 @@ describe('Input', () => {
         onSubmit={onSubmit}
         label="Branch name:"
         placeholder="feature/..."
-      />
+      />,
     );
 
-    expect(getByText('Branch name:')).toBeDefined();
-    expect(getByText('feature/...')).toBeDefined();
+    expect(getByText("Branch name:")).toBeDefined();
+    expect(getByText("feature/...")).toBeDefined();
   });
 
-  it('should accept mask prop for password input', () => {
+  it("should accept mask prop for password input", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     const { container } = render(
-      <Input value="secret" onChange={onChange} onSubmit={onSubmit} mask="*" />
+      <Input value="secret" onChange={onChange} onSubmit={onSubmit} mask="*" />,
     );
 
     expect(container).toBeDefined();
   });
 
-  it('should call onChange when value changes', () => {
+  it("should call onChange when value changes", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     render(<Input value="" onChange={onChange} onSubmit={onSubmit} />);
@@ -80,7 +87,7 @@ describe('Input', () => {
     expect(onChange).not.toHaveBeenCalled();
   });
 
-  it('should call onSubmit when submitted', () => {
+  it("should call onSubmit when submitted", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     render(<Input value="test" onChange={onChange} onSubmit={onSubmit} />);

package/src/cli/ui/__tests__/components/common/LoadingIndicator.test.tsx

@@ -1,81 +1,101 @@
 /**
  * @vitest-environment happy-dom
  */
-import React from 'react';
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { act, render } from '@testing-library/react';
-import { LoadingIndicator } from '../../../components/common/LoadingIndicator.js';
-import { Window } from 'happy-dom';
+import React from "react";
+import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
+import { act, render } from "@testing-library/react";
+import { LoadingIndicator } from "../../../components/common/LoadingIndicator.js";
+import { Window } from "happy-dom";
 
 const advanceTimersBy = async (ms: number) => {
   await act(async () => {
-    await vi.advanceTimersByTimeAsync(ms);
+    if (typeof vi.advanceTimersByTimeAsync === "function") {
+      await vi.advanceTimersByTimeAsync(ms);
+    } else if (typeof vi.advanceTimersByTime === "function") {
+      vi.advanceTimersByTime(ms);
+    } else {
+      await new Promise((resolve) => setTimeout(resolve, ms));
+    }
   });
 };
 
 beforeEach(() => {
-  vi.useFakeTimers();
+  if (typeof vi.useFakeTimers === "function") {
+    vi.useFakeTimers();
+  }
   const window = new Window();
   globalThis.window = window as any;
   globalThis.document = window.document as any;
 });
 
 afterEach(() => {
-  vi.clearAllTimers();
-  vi.useRealTimers();
+  if (typeof vi.clearAllTimers === "function") {
+    vi.clearAllTimers();
+  }
+  if (typeof vi.useRealTimers === "function") {
+    vi.useRealTimers();
+  }
 });
 
-describe('LoadingIndicator', () => {
+describe("LoadingIndicator", () => {
   const getSpinnerText = (container: HTMLElement) => {
-    return container.querySelector('ink-text')?.textContent ?? '';
+    return container.querySelector("ink-text")?.textContent ?? "";
   };
 
   const getMessageText = (container: HTMLElement) => {
-    const texts = container.querySelectorAll('ink-text');
-    return texts.length > 1 ? texts[1]?.textContent ?? '' : '';
+    const texts = container.querySelectorAll("ink-text");
+    return texts.length > 1 ? (texts[1]?.textContent ?? "") : "";
   };
 
-  it('does not render before the delay elapses', async () => {
+  it("does not render before the delay elapses", async () => {
     const { container } = render(
-      <LoadingIndicator isLoading={true} message="Loading data" delay={50} />
+      <LoadingIndicator isLoading={true} message="Loading data" delay={50} />,
     );
 
-    expect(container.textContent).toBe('');
+    expect(container.textContent).toBe("");
 
     await advanceTimersBy(20);
 
-    expect(container.textContent).toBe('');
+    expect(container.textContent).toBe("");
   });
 
-  it('renders after the delay elapses', async () => {
+  it("renders after the delay elapses", async () => {
     const { container } = render(
-      <LoadingIndicator isLoading={true} message="Loading data" delay={30} />
+      <LoadingIndicator isLoading={true} message="Loading data" delay={30} />,
     );
 
     await advanceTimersBy(30);
 
-    expect(getMessageText(container)).toContain('Loading data');
+    expect(getMessageText(container)).toContain("Loading data");
   });
 
-  it('stops rendering when loading becomes false', async () => {
+  it("stops rendering when loading becomes false", async () => {
     const { container, rerender } = render(
-      <LoadingIndicator isLoading={true} message="Loading data" delay={10} />
+      <LoadingIndicator isLoading={true} message="Loading data" delay={10} />,
     );
 
     await advanceTimersBy(10);
 
-    expect(getMessageText(container)).toContain('Loading data');
-
-    await act(async () => {
-      rerender(<LoadingIndicator isLoading={false} message="Loading data" delay={10} />);
-      await vi.advanceTimersByTimeAsync(0);
-    });
-
-    expect(container.textContent).toBe('');
+    expect(getMessageText(container)).toContain("Loading data");
+
+      await act(async () => {
+        rerender(
+          <LoadingIndicator
+            isLoading={false}
+            message="Loading data"
+            delay={10}
+          />,
+        );
+        if (typeof vi.advanceTimersByTimeAsync === "function") {
+          await vi.advanceTimersByTimeAsync(0);
+        }
+      });
+
+    expect(container.textContent).toBe("");
   });
 
-  it('cycles through spinner frames over time', async () => {
-    const customFrames = ['.', '..', '...'];
+  it("cycles through spinner frames over time", async () => {
+    const customFrames = [".", "..", "..."];
     const { container } = render(
       <LoadingIndicator
         isLoading={true}
@@ -83,7 +103,7 @@ describe('LoadingIndicator', () => {
         delay={0}
         interval={5}
         frames={customFrames}
-      />
+      />,
     );
 
     await advanceTimersBy(0);
@@ -100,28 +120,28 @@ describe('LoadingIndicator', () => {
 
     expect(secondFrame).not.toEqual(firstFrame);
     expect(thirdFrame).not.toEqual(secondFrame);
-    expect(customFrames).toContain(firstFrame ?? '');
-    expect(customFrames).toContain(secondFrame ?? '');
-    expect(customFrames).toContain(thirdFrame ?? '');
-    expect(getMessageText(container)).toContain('Loading data');
+    expect(customFrames).toContain(firstFrame ?? "");
+    expect(customFrames).toContain(secondFrame ?? "");
+    expect(customFrames).toContain(thirdFrame ?? "");
+    expect(getMessageText(container)).toContain("Loading data");
   });
 
-  it('keeps rendering even when only a single frame is provided', async () => {
+  it("keeps rendering even when only a single frame is provided", async () => {
     const { container } = render(
       <LoadingIndicator
         isLoading={true}
         message="Loading data"
         delay={0}
         interval={10}
-        frames={['*']}
-      />
+        frames={["*"]}
+      />,
     );
 
     await advanceTimersBy(0);
-    expect(getSpinnerText(container)).toBe('*');
+    expect(getSpinnerText(container)).toBe("*");
 
     await advanceTimersBy(30);
-    expect(getSpinnerText(container)).toBe('*');
-    expect(getMessageText(container)).toContain('Loading data');
+    expect(getSpinnerText(container)).toBe("*");
+    expect(getMessageText(container)).toContain("Loading data");
   });
 });