From 6b03cdb6d8856f6872f552f4e14395d100ea86a3 Mon Sep 17 00:00:00 2001
From: Siddharth Kumar Sah <siddharth123sk@gmail.com>
Date: Sun, 5 Apr 2026 23:00:13 +0800
Subject: [PATCH] test: add unit tests for pdf-to-image page range parser

---
 tests/unit/pdf-to-image.test.ts | 64 +++++++++++++++++++++++++++++++++
 1 file changed, 64 insertions(+)
 create mode 100644 tests/unit/pdf-to-image.test.ts

diff --git a/tests/unit/pdf-to-image.test.ts b/tests/unit/pdf-to-image.test.ts
new file mode 100644
index 00000000..f65c6463
--- /dev/null
+++ b/tests/unit/pdf-to-image.test.ts
@@ -0,0 +1,64 @@
+import { describe, expect, it } from "vitest";
+import { parsePageRange } from "../../apps/api/src/routes/tools/pdf-to-image.js";
+
+describe("parsePageRange", () => {
+  it("returns all pages for 'all'", () => {
+    expect(parsePageRange("all", 5)).toEqual([1, 2, 3, 4, 5]);
+  });
+
+  it("returns all pages for empty string", () => {
+    expect(parsePageRange("", 3)).toEqual([1, 2, 3]);
+  });
+
+  it("parses a single page", () => {
+    expect(parsePageRange("2", 5)).toEqual([2]);
+  });
+
+  it("parses a range", () => {
+    expect(parsePageRange("1-3", 5)).toEqual([1, 2, 3]);
+  });
+
+  it("parses mixed ranges and singles", () => {
+    expect(parsePageRange("1-3, 5", 5)).toEqual([1, 2, 3, 5]);
+  });
+
+  it("deduplicates overlapping ranges", () => {
+    expect(parsePageRange("1-3, 2-4", 5)).toEqual([1, 2, 3, 4]);
+  });
+
+  it("sorts output", () => {
+    expect(parsePageRange("5, 1, 3", 5)).toEqual([1, 3, 5]);
+  });
+
+  it("handles whitespace", () => {
+    expect(parsePageRange(" 1 - 3 , 5 ", 5)).toEqual([1, 2, 3, 5]);
+  });
+
+  it("throws on page 0", () => {
+    expect(() => parsePageRange("0", 5)).toThrow();
+  });
+
+  it("throws on negative page", () => {
+    expect(() => parsePageRange("-1", 5)).toThrow();
+  });
+
+  it("throws on page exceeding total", () => {
+    expect(() => parsePageRange("6", 5)).toThrow(/out of range/);
+  });
+
+  it("throws on range exceeding total", () => {
+    expect(() => parsePageRange("3-7", 5)).toThrow(/out of range/);
+  });
+
+  it("throws on reversed range", () => {
+    expect(() => parsePageRange("5-2", 5)).toThrow();
+  });
+
+  it("throws on non-numeric input", () => {
+    expect(() => parsePageRange("abc", 5)).toThrow();
+  });
+
+  it("throws on empty range segment", () => {
+    expect(() => parsePageRange("1,,3", 5)).toThrow();
+  });
+});