@simplysm/excel 13.0.75 → 13.0.77

package/tests/excel-wrapper.spec.ts

@@ -24,16 +24,16 @@ describe("ExcelWrapper", () => {
       const ws = await wb.getWorksheet("Users");
 
       // Check headers
-      expect(await ws.cell(0, 0).getVal()).toBe("Name");
-      expect(await ws.cell(0, 1).getVal()).toBe("Age");
-      expect(await ws.cell(0, 2).getVal()).toBe("Email");
-      expect(await ws.cell(0, 3).getVal()).toBe("Active");
+      expect(await ws.cell(0, 0).getValue()).toBe("Name");
+      expect(await ws.cell(0, 1).getValue()).toBe("Age");
+      expect(await ws.cell(0, 2).getValue()).toBe("Email");
+      expect(await ws.cell(0, 3).getValue()).toBe("Active");
 
       // Check data
-      expect(await ws.cell(1, 0).getVal()).toBe("John Doe");
-      expect(await ws.cell(1, 1).getVal()).toBe(30);
-      expect(await ws.cell(2, 0).getVal()).toBe("Jane Smith");
-      expect(await ws.cell(2, 1).getVal()).toBe(25);
+      expect(await ws.cell(1, 0).getValue()).toBe("John Doe");
+      expect(await ws.cell(1, 1).getValue()).toBe(30);
+      expect(await ws.cell(2, 0).getValue()).toBe("Jane Smith");
+      expect(await ws.cell(2, 1).getValue()).toBe(25);
 
       await wb.close();
     });
@@ -65,7 +65,7 @@ describe("ExcelWrapper", () => {
       ];
 
       const wb = await wrapper.write("Users", records);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       // Read from Excel
@@ -85,7 +85,7 @@ describe("ExcelWrapper", () => {
 
       const records = [{ name: "Test", age: 20 }];
       const wb = await wrapper.write("Sheet1", records);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       const readRecords = await wrapper.read(buffer, 0);
@@ -95,25 +95,12 @@ describe("ExcelWrapper", () => {
   });
 
   describe("Type conversion", () => {
-    it("can convert strings to numbers", async () => {
-      const wrapper = new ExcelWrapper(testSchema);
-
-      // Simulate Excel with values stored as strings manually
-      const wb = await wrapper.write("Test", [{ name: "Test", age: 25 }]);
-      const buffer = await wb.getBytes();
-      await wb.close();
-
-      const records = await wrapper.read(buffer);
-      expect(typeof records[0].age).toBe("number");
-      expect(records[0].age).toBe(25);
-    });
-
     it("applies default values", async () => {
       const wrapper = new ExcelWrapper(testSchema);
 
       // Save without active field
       const wb = await wrapper.write("Test", [{ name: "Test", age: 20 }]);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       const records = await wrapper.read(buffer);
@@ -133,7 +120,7 @@ describe("ExcelWrapper", () => {
       const records = [{ title: "Event 1", date: new DateOnly(2024, 6, 15) }, { title: "Event 2" }];
 
       const wb = await wrapper.write("Events", records);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       const readRecords = await wrapper.read(buffer, "Events");
@@ -157,7 +144,7 @@ describe("ExcelWrapper", () => {
       const records = [{ title: "Meeting", datetime: new DateTime(2024, 6, 15, 14, 30, 0) }];
 
       const wb = await wrapper.write("Events", records);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       const readRecords = await wrapper.read(buffer, "Events");
@@ -181,7 +168,7 @@ describe("ExcelWrapper", () => {
       const records = [{ title: "Alarm", time: new Time(9, 30, 0) }];
 
       const wb = await wrapper.write("Events", records);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       const readRecords = await wrapper.read(buffer, "Events");
@@ -198,7 +185,7 @@ describe("ExcelWrapper", () => {
 
       // Create empty Excel with only headers
       const wb = await wrapper.write("Empty", []);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       await expect(wrapper.read(buffer, "Empty")).rejects.toThrow(
@@ -210,22 +197,12 @@ describe("ExcelWrapper", () => {
       const wrapper = new ExcelWrapper(testSchema);
 
       const wb = await wrapper.write("Test", [{ name: "Test", age: 20 }]);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       await expect(wrapper.read(buffer, "NotExist")).rejects.toThrow();
     });
 
-    it("throws error when reading with non-existent worksheet index", async () => {
-      const wrapper = new ExcelWrapper(testSchema);
-
-      const wb = await wrapper.write("Test", [{ name: "Test", age: 20 }]);
-      const buffer = await wb.getBytes();
-      await wb.close();
-
-      await expect(wrapper.read(buffer, 99)).rejects.toThrow();
-    });
-
     it("throws error with worksheet name and detailed error when schema validation fails", async () => {
       const strictSchema = z.object({
         name: z.string().min(5).describe("이름"), // At least 5 characters
@@ -238,9 +215,9 @@ describe("ExcelWrapper", () => {
 
       // Modify data directly to trigger validation failure
       const ws = await wb.getWorksheet("Validation");
-      await ws.cell(1, 0).setVal("AB"); // Change to less than 5 characters
+      await ws.cell(1, 0).setValue("AB"); // Change to less than 5 characters
 
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       // Should throw validation error
@@ -264,13 +241,13 @@ describe("ExcelWrapper", () => {
       const ws = await wb.getWorksheet("Test");
 
       // Headers: only name and age exist
-      expect(await ws.cell(0, 0).getVal()).toBe("Name");
-      expect(await ws.cell(0, 1).getVal()).toBe("Age");
-      expect(await ws.cell(0, 2).getVal()).toBeUndefined();
+      expect(await ws.cell(0, 0).getValue()).toBe("Name");
+      expect(await ws.cell(0, 1).getValue()).toBe("Age");
+      expect(await ws.cell(0, 2).getValue()).toBeUndefined();
 
       // Check data
-      expect(await ws.cell(1, 0).getVal()).toBe("John Doe");
-      expect(await ws.cell(1, 1).getVal()).toBe(30);
+      expect(await ws.cell(1, 0).getValue()).toBe("John Doe");
+      expect(await ws.cell(1, 1).getValue()).toBe(30);
 
       await wb.close();
     });
@@ -281,7 +258,7 @@ describe("ExcelWrapper", () => {
       // Create Excel with all fields
       const records = [{ name: "John Doe", age: 30, email: "john@test.com", phone: "010-1234-5678" }];
       const wb = await wrapper.write("Test", records);
-      const buffer = await wb.getBytes();
+      const buffer = await wb.toBytes();
       await wb.close();
 
       // Read with excludes

package/tests/image-insert.spec.ts

@@ -33,7 +33,7 @@ async function loadPngFile(): Promise<Uint8Array> {
 describe("ExcelWorksheet.addImage integration", () => {
   it("should write media, drawing, drawing rels, worksheet rel and content types", async () => {
     const wb = new ExcelWorkbook();
-    const ws = await wb.createWorksheet("Sheet1");
+    const ws = await wb.addWorksheet("Sheet1");
 
     const bytes = await loadPngFile();
 
@@ -102,14 +102,14 @@ describe("ExcelWorksheet.addImage integration", () => {
     expect(drawingElems.some((d: any) => d.$ != null && d.$["r:id"] != null)).toBeTruthy();
 
     // Verify buffer creation
-    const resultBuffer = await wb.getBytes();
+    const resultBuffer = await wb.toBytes();
     expect(resultBuffer).toBeDefined();
     expect(resultBuffer.length).toBeGreaterThan(0);
   });
 
   it("can insert multiple images into the same worksheet", async () => {
     const wb = new ExcelWorkbook();
-    const ws = await wb.createWorksheet("Sheet1");
+    const ws = await wb.addWorksheet("Sheet1");
 
     const bytes = await loadPngFile();
 
@@ -170,14 +170,14 @@ describe("ExcelWorksheet.addImage integration", () => {
     expect(relsArr.some((r: any) => r.$.Target === "../media/image2.png")).toBeTruthy();
 
     // Verify buffer creation
-    const resultBuffer = await wb.getBytes();
+    const resultBuffer = await wb.toBytes();
     expect(resultBuffer).toBeDefined();
     expect(resultBuffer.length).toBeGreaterThan(0);
   });
 
   it("throws error for unsupported file extensions", async () => {
     const wb = new ExcelWorkbook();
-    const ws = await wb.createWorksheet("Sheet1");
+    const ws = await wb.addWorksheet("Sheet1");
 
     await expect(
       ws.addImage({

package/tests/utils/excel-utils.spec.ts

@@ -2,13 +2,7 @@ import { describe, expect, it } from "vitest";
 import { ExcelUtils } from "../../src/utils/excel-utils";
 
 describe("ExcelUtils", () => {
-  describe("stringifyColAddr / parseColAddrCode", () => {
-    it("converts 0-25 to A-Z", () => {
-      expect(ExcelUtils.stringifyColAddr(0)).toBe("A");
-      expect(ExcelUtils.stringifyColAddr(1)).toBe("B");
-      expect(ExcelUtils.stringifyColAddr(25)).toBe("Z");
-    });
-
+  describe("stringifyColAddr / parseColAddr", () => {
     it("converts 26 and above to AA, AB, etc.", () => {
       expect(ExcelUtils.stringifyColAddr(26)).toBe("AA");
       expect(ExcelUtils.stringifyColAddr(27)).toBe("AB");
@@ -24,32 +18,13 @@ describe("ExcelUtils", () => {
     it("handles Excel maximum column index (XFD, 16383)", () => {
       // Excel maximum column is XFD (index 16383, 0-based)
       expect(ExcelUtils.stringifyColAddr(16383)).toBe("XFD");
-      expect(ExcelUtils.parseColAddrCode("XFD")).toBe(16383);
-    });
-
-    it("parses A-Z to 0-25", () => {
-      expect(ExcelUtils.parseColAddrCode("A")).toBe(0);
-      expect(ExcelUtils.parseColAddrCode("B")).toBe(1);
-      expect(ExcelUtils.parseColAddrCode("Z")).toBe(25);
-    });
-
-    it("parses AA, AB, etc. to 26 and above", () => {
-      expect(ExcelUtils.parseColAddrCode("AA")).toBe(26);
-      expect(ExcelUtils.parseColAddrCode("AB")).toBe(27);
-      expect(ExcelUtils.parseColAddrCode("AZ")).toBe(51);
-      expect(ExcelUtils.parseColAddrCode("BA")).toBe(52);
-    });
-
-    it("parses column index from cell address", () => {
-      expect(ExcelUtils.parseColAddrCode("A1")).toBe(0);
-      expect(ExcelUtils.parseColAddrCode("B10")).toBe(1);
-      expect(ExcelUtils.parseColAddrCode("AA100")).toBe(26);
+      expect(ExcelUtils.parseColAddr("XFD")).toBe(16383);
     });
 
     it("round-trip: stringify → parse returns original value", () => {
       for (let i = 0; i < 100; i++) {
         const stringified = ExcelUtils.stringifyColAddr(i);
-        const parsed = ExcelUtils.parseColAddrCode(stringified);
+        const parsed = ExcelUtils.parseColAddr(stringified);
         expect(parsed).toBe(i);
       }
     });
@@ -60,7 +35,7 @@ describe("ExcelUtils", () => {
     });
   });
 
-  describe("stringifyRowAddr / parseRowAddrCode", () => {
+  describe("stringifyRowAddr / parseRowAddr", () => {
     it("converts 0-based index to 1-based string", () => {
       expect(ExcelUtils.stringifyRowAddr(0)).toBe("1");
       expect(ExcelUtils.stringifyRowAddr(9)).toBe("10");
@@ -68,13 +43,13 @@ describe("ExcelUtils", () => {
     });
 
     it("parses 1-based string to 0-based index", () => {
-      expect(ExcelUtils.parseRowAddrCode("A1")).toBe(0);
-      expect(ExcelUtils.parseRowAddrCode("B10")).toBe(9);
-      expect(ExcelUtils.parseRowAddrCode("AA100")).toBe(99);
+      expect(ExcelUtils.parseRowAddr("A1")).toBe(0);
+      expect(ExcelUtils.parseRowAddr("B10")).toBe(9);
+      expect(ExcelUtils.parseRowAddr("AA100")).toBe(99);
     });
   });
 
-  describe("stringifyAddr / parseCellAddrCode", () => {
+  describe("stringifyAddr / parseCellAddr", () => {
     it("correctly converts cell address", () => {
       expect(ExcelUtils.stringifyAddr({ r: 0, c: 0 })).toBe("A1");
       expect(ExcelUtils.stringifyAddr({ r: 9, c: 1 })).toBe("B10");
@@ -82,21 +57,21 @@ describe("ExcelUtils", () => {
     });
 
     it("correctly parses cell address", () => {
-      expect(ExcelUtils.parseCellAddrCode("A1")).toEqual({ r: 0, c: 0 });
-      expect(ExcelUtils.parseCellAddrCode("B10")).toEqual({ r: 9, c: 1 });
-      expect(ExcelUtils.parseCellAddrCode("AA100")).toEqual({ r: 99, c: 26 });
+      expect(ExcelUtils.parseCellAddr("A1")).toEqual({ r: 0, c: 0 });
+      expect(ExcelUtils.parseCellAddr("B10")).toEqual({ r: 9, c: 1 });
+      expect(ExcelUtils.parseCellAddr("AA100")).toEqual({ r: 99, c: 26 });
     });
   });
 
-  describe("parseRangeAddrCode / stringifyRangeAddr", () => {
+  describe("parseRangeAddr / stringifyRangeAddr", () => {
     it("parses single cell range", () => {
-      const range = ExcelUtils.parseRangeAddrCode("A1");
+      const range = ExcelUtils.parseRangeAddr("A1");
       expect(range.s).toEqual({ r: 0, c: 0 });
       expect(range.e).toEqual({ r: 0, c: 0 });
     });
 
     it("parses multi-cell range", () => {
-      const range = ExcelUtils.parseRangeAddrCode("A1:C3");
+      const range = ExcelUtils.parseRangeAddr("A1:C3");
       expect(range.s).toEqual({ r: 0, c: 0 });
       expect(range.e).toEqual({ r: 2, c: 2 });
     });
@@ -127,25 +102,6 @@ describe("ExcelUtils", () => {
       expect(excelNum).toBeCloseTo(25569, 0);
     });
 
-    it("correctly converts 2024-06-15", () => {
-      const date = new Date(Date.UTC(2024, 5, 15, 0, 0, 0));
-      const tick = date.getTime();
-      const excelNum = ExcelUtils.convertTimeTickToNumber(tick);
-      // Verify approximate value
-      expect(excelNum).toBeGreaterThan(45000);
-    });
-
-    it("correctly converts date with time", () => {
-      const date = new Date(Date.UTC(2024, 5, 15, 12, 0, 0));
-      const tick = date.getTime();
-      const excelNum = ExcelUtils.convertTimeTickToNumber(tick);
-      // 12:00 = 0.5 days additional
-      const baseNum = ExcelUtils.convertTimeTickToNumber(
-        new Date(Date.UTC(2024, 5, 15, 0, 0, 0)).getTime(),
-      );
-      expect(excelNum - baseNum).toBeCloseTo(0.5, 1);
-    });
-
     it("round-trip: tick → number → tick returns original value", () => {
       const originalDate = new Date(Date.UTC(2024, 5, 15, 14, 30, 45));
       const tick = originalDate.getTime();