@hasna/recordings 0.1.11 → 0.1.12

package/src/__tests__/projects.test.ts

@@ -1,109 +0,0 @@
-import { describe, test, expect, beforeEach, afterEach } from "bun:test";
-import { tmpdir } from "os";
-import { join } from "path";
-import { mkdirSync, rmSync, existsSync } from "fs";
-import {
-  getDatabase,
-  closeDatabase,
-  resetDatabase,
-} from "../db/database.js";
-import { registerProject, getProject, listProjects } from "../db/projects.js";
-import { type Database } from "bun:sqlite";
-
-let tempDir: string;
-let db: Database;
-
-beforeEach(() => {
-  resetDatabase();
-  tempDir = join(tmpdir(), `open-recordings-test-proj-${Date.now()}-${Math.random().toString(36).slice(2)}`);
-  mkdirSync(tempDir, { recursive: true });
-  const dbPath = join(tempDir, "test.db");
-  db = getDatabase(dbPath);
-});
-
-afterEach(() => {
-  closeDatabase();
-  resetDatabase();
-  if (existsSync(tempDir)) {
-    rmSync(tempDir, { recursive: true, force: true });
-  }
-});
-
-describe("registerProject", () => {
-  test("creates a new project", () => {
-    const project = registerProject("my-app", "/home/user/my-app", undefined, db);
-    expect(project).toBeDefined();
-    expect(project.name).toBe("my-app");
-    expect(project.path).toBe("/home/user/my-app");
-    expect(project.description).toBeNull();
-    expect(project.id).toBeDefined();
-    expect(project.id).toHaveLength(36); // full UUID
-    expect(project.created_at).toBeDefined();
-    expect(project.updated_at).toBeDefined();
-  });
-
-  test("creates a project with description", () => {
-    const project = registerProject("my-app", "/home/user/my-app", "A cool project", db);
-    expect(project.description).toBe("A cool project");
-  });
-
-  test("returns existing project by path (idempotent)", () => {
-    const first = registerProject("my-app", "/home/user/my-app", undefined, db);
-    const second = registerProject("my-app", "/home/user/my-app", undefined, db);
-
-    expect(second.id).toBe(first.id);
-    expect(new Date(second.updated_at).getTime()).toBeGreaterThanOrEqual(
-      new Date(first.updated_at).getTime()
-    );
-  });
-
-  test("creates separate projects for different paths", () => {
-    const p1 = registerProject("app-a", "/path/a", undefined, db);
-    const p2 = registerProject("app-b", "/path/b", undefined, db);
-
-    expect(p1.id).not.toBe(p2.id);
-    expect(listProjects(db)).toHaveLength(2);
-  });
-});
-
-describe("getProject", () => {
-  test("finds project by ID", () => {
-    const created = registerProject("test", "/tmp/test", undefined, db);
-    const found = getProject(created.id, db);
-    expect(found).toBeDefined();
-    expect(found!.name).toBe("test");
-  });
-
-  test("finds project by path", () => {
-    registerProject("finder", "/unique/path/here", undefined, db);
-    const found = getProject("/unique/path/here", db);
-    expect(found).toBeDefined();
-    expect(found!.name).toBe("finder");
-  });
-
-  test("returns null for non-existent project", () => {
-    const found = getProject("nonexistent", db);
-    expect(found).toBeNull();
-  });
-});
-
-describe("listProjects", () => {
-  test("returns empty array when no projects", () => {
-    const projects = listProjects(db);
-    expect(projects).toEqual([]);
-  });
-
-  test("returns all projects ordered by updated_at DESC", () => {
-    registerProject("alpha", "/path/alpha", undefined, db);
-    registerProject("beta", "/path/beta", undefined, db);
-    registerProject("gamma", "/path/gamma", undefined, db);
-
-    const projects = listProjects(db);
-    expect(projects).toHaveLength(3);
-    // All three projects are present
-    const names = projects.map((p) => p.name);
-    expect(names).toContain("alpha");
-    expect(names).toContain("beta");
-    expect(names).toContain("gamma");
-  });
-});

package/src/__tests__/recorder.test.ts

@@ -1,278 +0,0 @@
-import { describe, test, expect, beforeEach, afterEach, mock } from "bun:test";
-import { tmpdir } from "os";
-import { join } from "path";
-import { mkdirSync, rmSync, existsSync } from "fs";
-import { DEFAULT_CONFIG } from "../lib/config.js";
-import type { RecordingsConfig } from "../types/index.js";
-import { RecordingError } from "../types/index.js";
-import { EventEmitter } from "events";
-
-let tempDir: string;
-
-beforeEach(() => {
-  tempDir = join(tmpdir(), `open-recordings-test-rec-${Date.now()}-${Math.random().toString(36).slice(2)}`);
-  mkdirSync(tempDir, { recursive: true });
-});
-
-afterEach(() => {
-  if (existsSync(tempDir)) {
-    rmSync(tempDir, { recursive: true, force: true });
-  }
-});
-
-describe("checkRecordingDeps", () => {
-  test("returns an object with available, tool, and message fields", async () => {
-    const { checkRecordingDeps } = await import("../lib/recorder.js");
-    const result = await checkRecordingDeps();
-    expect(result).toHaveProperty("available");
-    expect(result).toHaveProperty("tool");
-    expect(result).toHaveProperty("message");
-    expect(typeof result.available).toBe("boolean");
-    expect(typeof result.tool).toBe("string");
-    expect(typeof result.message).toBe("string");
-  });
-
-  test("detects at least one recording tool on this system or returns none", async () => {
-    const { checkRecordingDeps } = await import("../lib/recorder.js");
-    const result = await checkRecordingDeps();
-    if (result.available) {
-      expect(["sox", "rec", "ffmpeg"]).toContain(result.tool);
-      expect(result.message).toContain("is available");
-    } else {
-      expect(result.tool).toBe("none");
-      expect(result.message).toContain("No recording tool found");
-      expect(result.message).toContain("Install sox");
-    }
-  });
-});
-
-describe("isRecording", () => {
-  test("returns false when not recording", async () => {
-    const { isRecording } = await import("../lib/recorder.js");
-    expect(isRecording()).toBe(false);
-  });
-});
-
-describe("getCurrentFile", () => {
-  test("returns null when not recording", async () => {
-    const { getCurrentFile } = await import("../lib/recorder.js");
-    expect(getCurrentFile()).toBeNull();
-  });
-});
-
-describe("stopRecording", () => {
-  test("returns null when not recording", async () => {
-    const { stopRecording } = await import("../lib/recorder.js");
-    const result = stopRecording();
-    expect(result).toBeNull();
-  });
-});
-
-describe("startRecording with mocked spawn", () => {
-  test("spawns rec process and returns filepath", async () => {
-    // Create a mock child process
-    const mockProcess = new EventEmitter() as any;
-    mockProcess.kill = mock(() => {});
-    mockProcess.stdin = null;
-    mockProcess.stdout = null;
-    mockProcess.stderr = null;
-
-    mock.module("child_process", () => ({
-      spawn: mock(() => mockProcess),
-    }));
-
-    // Re-import to pick up mock
-    const recorder = await import("../lib/recorder.js");
-
-    // Reset any existing recording state by stopping
-    recorder.stopRecording();
-
-    const config: RecordingsConfig = {
-      ...DEFAULT_CONFIG,
-      audio_dir: tempDir,
-      audio_format: "wav",
-      sample_rate: 16000,
-      max_recording_seconds: 300,
-    };
-
-    const filepath = recorder.startRecording(config);
-    expect(filepath).toContain(tempDir);
-    expect(filepath).toContain("recording-");
-    expect(filepath).toContain(".wav");
-    expect(recorder.isRecording()).toBe(true);
-    expect(recorder.getCurrentFile()).toBe(filepath);
-
-    // Stop
-    const stopped = recorder.stopRecording();
-    expect(stopped).toBe(filepath);
-    expect(recorder.isRecording()).toBe(false);
-  });
-
-  test("throws RecordingError when already recording", async () => {
-    const mockProcess = new EventEmitter() as any;
-    mockProcess.kill = mock(() => {});
-    mockProcess.stdin = null;
-    mockProcess.stdout = null;
-    mockProcess.stderr = null;
-
-    mock.module("child_process", () => ({
-      spawn: mock(() => mockProcess),
-    }));
-
-    const recorder = await import("../lib/recorder.js");
-    recorder.stopRecording(); // Clean state
-
-    const config: RecordingsConfig = {
-      ...DEFAULT_CONFIG,
-      audio_dir: tempDir,
-    };
-
-    recorder.startRecording(config);
-
-    try {
-      recorder.startRecording(config);
-      expect(true).toBe(false);
-    } catch (err) {
-      expect(err).toBeInstanceOf(RecordingError);
-      expect((err as Error).message).toContain("Already recording");
-    }
-
-    recorder.stopRecording();
-  });
-
-  test("generates filename with mp3 format", async () => {
-    const mockProcess = new EventEmitter() as any;
-    mockProcess.kill = mock(() => {});
-    mockProcess.stdin = null;
-    mockProcess.stdout = null;
-    mockProcess.stderr = null;
-
-    mock.module("child_process", () => ({
-      spawn: mock(() => mockProcess),
-    }));
-
-    const recorder = await import("../lib/recorder.js");
-    recorder.stopRecording();
-
-    const config: RecordingsConfig = {
-      ...DEFAULT_CONFIG,
-      audio_dir: tempDir,
-      audio_format: "mp3",
-    };
-
-    const filepath = recorder.startRecording(config);
-    expect(filepath).toContain(".mp3");
-
-    recorder.stopRecording();
-  });
-
-  test("handles process exit event", async () => {
-    const mockProcess = new EventEmitter() as any;
-    mockProcess.kill = mock(() => {});
-    mockProcess.stdin = null;
-    mockProcess.stdout = null;
-    mockProcess.stderr = null;
-
-    mock.module("child_process", () => ({
-      spawn: mock(() => mockProcess),
-    }));
-
-    const recorder = await import("../lib/recorder.js");
-    recorder.stopRecording();
-
-    const config: RecordingsConfig = {
-      ...DEFAULT_CONFIG,
-      audio_dir: tempDir,
-    };
-
-    recorder.startRecording(config);
-    expect(recorder.isRecording()).toBe(true);
-
-    // Simulate process exit
-    mockProcess.emit("exit", 0);
-
-    // After exit, _recordProcess should be null but _currentFile may still be set
-    // until stopRecording is called
-    recorder.stopRecording();
-  });
-
-  test("handles process error event by clearing state", async () => {
-    const mockProcess = new EventEmitter() as any;
-    mockProcess.kill = mock(() => {});
-    mockProcess.stdin = null;
-    mockProcess.stdout = null;
-    mockProcess.stderr = null;
-
-    mock.module("child_process", () => ({
-      spawn: mock(() => mockProcess),
-    }));
-
-    const recorder = await import("../lib/recorder.js");
-    recorder.stopRecording();
-
-    const config: RecordingsConfig = {
-      ...DEFAULT_CONFIG,
-      audio_dir: tempDir,
-    };
-
-    recorder.startRecording(config);
-    expect(recorder.isRecording()).toBe(true);
-
-    // Simulate process error - the error handler throws, but we catch it
-    try {
-      mockProcess.emit("error", new Error("spawn ENOENT"));
-    } catch {
-      // The error handler throws a RecordingError
-    }
-
-    // After error, state should be cleared
-    // Note: the error handler sets _recordProcess = null and _currentFile = null
-    // but since it throws, the state cleanup depends on the throw being caught
-    recorder.stopRecording(); // Cleanup
-  });
-
-  test("stopRecording sends SIGINT to process", async () => {
-    const mockProcess = new EventEmitter() as any;
-    mockProcess.kill = mock(() => {});
-    mockProcess.stdin = null;
-    mockProcess.stdout = null;
-    mockProcess.stderr = null;
-
-    mock.module("child_process", () => ({
-      spawn: mock(() => mockProcess),
-    }));
-
-    const recorder = await import("../lib/recorder.js");
-    recorder.stopRecording();
-
-    const config: RecordingsConfig = {
-      ...DEFAULT_CONFIG,
-      audio_dir: tempDir,
-    };
-
-    recorder.startRecording(config);
-    recorder.stopRecording();
-
-    expect(mockProcess.kill).toHaveBeenCalledWith("SIGINT");
-  });
-});
-
-describe("recordDuration", () => {
-  test("throws when rec is not available", async () => {
-    const { recordDuration } = await import("../lib/recorder.js");
-
-    const config: RecordingsConfig = {
-      ...DEFAULT_CONFIG,
-      audio_dir: tempDir,
-    };
-
-    try {
-      await recordDuration(1, config);
-      // If rec is installed, this might succeed
-    } catch (err) {
-      // Expected to fail since rec isn't typically installed in CI
-      // Could be ENOENT from Bun.spawn or RecordingError
-      expect(err).toBeDefined();
-    }
-  });
-});