Merge pull request #97 from hqwlkj/refactor/extract-dropdown-components

refactor: Extract dropdown components

Author: qorzj

src/tests/sessionList.test.ts

@@ -1,6 +1,7 @@
 import { test } from "node:test";
 import assert from "node:assert/strict";
-import { formatSessionTitle } from "../ui";
+import { formatSessionTitle, filterSessions, formatSessionStatus } from "../ui";
+import type { SessionEntry } from "../session";
 
 test("formatSessionTitle replaces newlines with spaces", () => {
   assert.equal(formatSessionTitle("first line\nsecond line\r\nthird"), "first line second line third");
@@ -9,3 +10,108 @@ test("formatSessionTitle replaces newlines with spaces", () => {
 test("formatSessionTitle truncates after normalizing whitespace", () => {
   assert.equal(formatSessionTitle("one\n two   three", 10), "one two th…");
 });
+
+test("formatSessionStatus maps status values to display labels", () => {
+  assert.equal(formatSessionStatus("completed"), "done");
+  assert.equal(formatSessionStatus("processing"), "running");
+  assert.equal(formatSessionStatus("pending"), "pending");
+  assert.equal(formatSessionStatus("waiting_for_user"), "waiting");
+  assert.equal(formatSessionStatus("failed"), "failed");
+  assert.equal(formatSessionStatus("interrupted"), "stopped");
+  assert.equal(formatSessionStatus("unknown_status" as any), "unknown_status");
+});
+
+test("filterSessions returns all sessions when query is empty", () => {
+  const sessions = buildSessions([{ summary: "Fix login bug" }, { summary: "Add dark mode" }]);
+  assert.equal(filterSessions(sessions, "").length, 2);
+  assert.equal(filterSessions(sessions, "   ").length, 2);
+});
+
+test("filterSessions matches by summary (case-insensitive)", () => {
+  const sessions = buildSessions([
+    { summary: "Fix login bug" },
+    { summary: "Add dark mode" },
+    { summary: "Refactor auth module" },
+  ]);
+
+  assert.equal(filterSessions(sessions, "login").length, 1);
+  assert.equal(filterSessions(sessions, "LOGIN").length, 1);
+  assert.equal(filterSessions(sessions, "Login").length, 1);
+});
+
+test("filterSessions matches by status (case-insensitive)", () => {
+  const sessions = buildSessions([
+    { summary: "Task 1", status: "completed" },
+    { summary: "Task 2", status: "failed" },
+    { summary: "Task 3", status: "completed" },
+  ]);
+
+  assert.equal(filterSessions(sessions, "failed").length, 1);
+  assert.equal(filterSessions(sessions, "completed").length, 2);
+});
+
+test("filterSessions matches by failReason", () => {
+  const sessions = buildSessions([
+    { summary: "Task 1", status: "failed", failReason: "API key not found" },
+    { summary: "Task 2", status: "completed" },
+  ]);
+
+  assert.equal(filterSessions(sessions, "API key").length, 1);
+  assert.equal(filterSessions(sessions, "not found").length, 1);
+});
+
+test("filterSessions matches by assistantReply", () => {
+  const sessions = buildSessions([
+    { summary: "Task 1", assistantReply: "The bug was fixed by updating the config." },
+    { summary: "Task 2", assistantReply: "Dark mode has been added successfully." },
+  ]);
+
+  assert.equal(filterSessions(sessions, "dark mode").length, 1);
+  assert.equal(filterSessions(sessions, "config").length, 1);
+});
+
+test("filterSessions returns empty array when no match", () => {
+  const sessions = buildSessions([{ summary: "Fix login bug" }, { summary: "Add dark mode" }]);
+
+  assert.equal(filterSessions(sessions, "nonexistent").length, 0);
+});
+
+test("filterSessions matches across multiple fields on same session", () => {
+  const sessions = buildSessions([
+    { summary: "Fix login bug", status: "failed", failReason: "Timeout error" },
+    { summary: "Add dark mode", status: "completed" },
+  ]);
+
+  // Should match the first session via status
+  assert.equal(filterSessions(sessions, "failed").length, 1);
+  // Should match the first session via failReason
+  assert.equal(filterSessions(sessions, "timeout").length, 1);
+  // Partial summary match
+  assert.equal(filterSessions(sessions, "login").length, 1);
+});
+
+test("filterSessions handles sessions with null fields", () => {
+  const sessions = buildSessions([{ summary: null }, { summary: "Valid summary" }]);
+
+  assert.equal(filterSessions(sessions, "valid").length, 1);
+  assert.equal(filterSessions(sessions, "summary").length, 1);
+});
+
+function buildSessions(overrides: Array<Partial<SessionEntry>>): SessionEntry[] {
+  return overrides.map((override, i) => ({
+    id: `session-${i}`,
+    summary: override.summary ?? null,
+    assistantReply: override.assistantReply ?? null,
+    assistantThinking: null,
+    assistantRefusal: null,
+    toolCalls: null,
+    status: override.status ?? "completed",
+    failReason: override.failReason ?? null,
+    usage: null,
+    usagePerModel: null,
+    activeTokens: 0,
+    createTime: new Date().toISOString(),
+    updateTime: new Date().toISOString(),
+    processes: null,
+  }));
+}