Add some vibe coded tests

Author: chrarnoldus

src/integrations/misc/__tests__/extract-text-from-xlsx.test.ts

@@ -0,0 +1,241 @@
+import { describe, it, expect } from "@jest/globals"
+import ExcelJS from "exceljs"
+import { extractTextFromXLSX } from "../extract-text-from-xlsx"
+
+describe("extractTextFromXLSX", () => {
+	describe("basic functionality", () => {
+		it("should extract text with proper formatting", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			worksheet.getCell("A1").value = "Hello"
+			worksheet.getCell("B1").value = "World"
+			worksheet.getCell("A2").value = "Test"
+			worksheet.getCell("B2").value = 123
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("--- Sheet: Sheet1 ---")
+			expect(result).toContain("Hello\tWorld")
+			expect(result).toContain("Test\t123")
+		})
+
+		it("should skip rows with no content", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			worksheet.getCell("A1").value = "Row 1"
+			// Row 2 is completely empty
+			worksheet.getCell("A3").value = "Row 3"
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("Row 1")
+			expect(result).toContain("Row 3")
+			// Should not contain empty rows
+			expect(result).not.toMatch(/\n\t*\n/)
+		})
+	})
+
+	describe("sheet handling", () => {
+		it("should process multiple sheets", async () => {
+			const workbook = new ExcelJS.Workbook()
+
+			const sheet1 = workbook.addWorksheet("First Sheet")
+			sheet1.getCell("A1").value = "Sheet 1 Data"
+
+			const sheet2 = workbook.addWorksheet("Second Sheet")
+			sheet2.getCell("A1").value = "Sheet 2 Data"
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("--- Sheet: First Sheet ---")
+			expect(result).toContain("Sheet 1 Data")
+			expect(result).toContain("--- Sheet: Second Sheet ---")
+			expect(result).toContain("Sheet 2 Data")
+		})
+
+		it("should skip hidden sheets", async () => {
+			const workbook = new ExcelJS.Workbook()
+
+			const visibleSheet = workbook.addWorksheet("Visible Sheet")
+			visibleSheet.getCell("A1").value = "Visible Data"
+
+			const hiddenSheet = workbook.addWorksheet("Hidden Sheet")
+			hiddenSheet.getCell("A1").value = "Hidden Data"
+			hiddenSheet.state = "hidden"
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("--- Sheet: Visible Sheet ---")
+			expect(result).toContain("Visible Data")
+			expect(result).not.toContain("--- Sheet: Hidden Sheet ---")
+			expect(result).not.toContain("Hidden Data")
+		})
+
+		it("should skip very hidden sheets", async () => {
+			const workbook = new ExcelJS.Workbook()
+
+			const visibleSheet = workbook.addWorksheet("Visible Sheet")
+			visibleSheet.getCell("A1").value = "Visible Data"
+
+			const veryHiddenSheet = workbook.addWorksheet("Very Hidden Sheet")
+			veryHiddenSheet.getCell("A1").value = "Very Hidden Data"
+			veryHiddenSheet.state = "veryHidden"
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("--- Sheet: Visible Sheet ---")
+			expect(result).toContain("Visible Data")
+			expect(result).not.toContain("--- Sheet: Very Hidden Sheet ---")
+			expect(result).not.toContain("Very Hidden Data")
+		})
+	})
+
+	describe("formatCellValue logic", () => {
+		it("should handle null and undefined values", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			worksheet.getCell("A1").value = "Before"
+			worksheet.getCell("A2").value = null
+			worksheet.getCell("A3").value = undefined
+			worksheet.getCell("A4").value = "After"
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("Before")
+			expect(result).toContain("After")
+			// Should handle null/undefined as empty strings
+			const lines = result.split("\n")
+			const dataLines = lines.filter((line) => !line.startsWith("---") && line.trim())
+			expect(dataLines).toHaveLength(2) // Only 'Before' and 'After' should create content
+		})
+
+		it("should format dates correctly", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			const testDate = new Date("2023-12-25")
+			worksheet.getCell("A1").value = testDate
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("2023-12-25")
+		})
+
+		it("should handle error values", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			worksheet.getCell("A1").value = { error: "#DIV/0!" }
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("[Error: #DIV/0!]")
+		})
+
+		it("should handle rich text", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			worksheet.getCell("A1").value = {
+				richText: [{ text: "Hello " }, { text: "World", font: { bold: true } }],
+			}
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("Hello World")
+		})
+
+		it("should handle hyperlinks", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			worksheet.getCell("A1").value = {
+				text: "Google",
+				hyperlink: "https://www.google.com",
+			}
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("Google (https://www.google.com)")
+		})
+
+		it("should handle formulas with and without results", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			worksheet.getCell("A1").value = { formula: "A2+A3", result: 30 }
+			worksheet.getCell("A2").value = { formula: "SUM(B1:B10)" }
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("30") // Formula with result
+			expect(result).toContain("[Formula: SUM(B1:B10)]") // Formula without result
+		})
+	})
+
+	describe("row limit handling", () => {
+		it("should respect the ROW_LIMIT constant", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			// Add a reasonable number of rows for testing
+			for (let i = 1; i <= 100; i++) {
+				worksheet.getCell(`A${i}`).value = `Row ${i}`
+			}
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("Row 1")
+			expect(result).toContain("Row 100")
+			// Should not contain truncation message for 100 rows (under limit)
+			expect(result).not.toContain("[... truncated at row")
+		})
+	})
+
+	describe("edge cases", () => {
+		it("should handle empty workbook", async () => {
+			const workbook = new ExcelJS.Workbook()
+			workbook.addWorksheet("Empty Sheet")
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("--- Sheet: Empty Sheet ---")
+			expect(result.trim()).toBe("--- Sheet: Empty Sheet ---")
+		})
+
+		it("should handle workbook with only empty cells", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Sheet1")
+
+			// Set cells but leave them empty
+			worksheet.getCell("A1").value = ""
+			worksheet.getCell("B1").value = ""
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("--- Sheet: Sheet1 ---")
+			// Should not contain any data rows since empty strings don't count as content
+			const lines = result.split("\n").filter((line) => line.trim() && !line.startsWith("---"))
+			expect(lines).toHaveLength(0)
+		})
+	})
+
+	describe("function overloads", () => {
+		it("should work with workbook objects", async () => {
+			const workbook = new ExcelJS.Workbook()
+			const worksheet = workbook.addWorksheet("Test")
+			worksheet.getCell("A1").value = "Test Data"
+
+			const result = await extractTextFromXLSX(workbook)
+
+			expect(result).toContain("Test Data")
+		})
+
+		it("should reject invalid file paths", async () => {
+			await expect(extractTextFromXLSX("/non/existent/file.xlsx")).rejects.toThrow()
+		})
+	})
+})

src/integrations/misc/extract-text-from-xlsx.ts

@@ -40,11 +40,16 @@ function formatCellValue(cell: ExcelJS.Cell): string {
 	return value.toString()
 }
 
-export async function extractTextFromXLSX(filePath: string): Promise<string> {
-	const workbook = new ExcelJS.Workbook()
+export async function extractTextFromXLSX(filePathOrWorkbook: string | ExcelJS.Workbook): Promise<string> {
+	let workbook: ExcelJS.Workbook
 	let excelText = ""
 
-	await workbook.xlsx.readFile(filePath)
+	if (typeof filePathOrWorkbook === "string") {
+		workbook = new ExcelJS.Workbook()
+		await workbook.xlsx.readFile(filePathOrWorkbook)
+	} else {
+		workbook = filePathOrWorkbook
+	}
 
 	workbook.eachSheet((worksheet, sheetId) => {
 		if (worksheet.state === "hidden" || worksheet.state === "veryHidden") {