@goplusvn/core 0.1.55 → 0.1.57

package/src/crud/components/crud-page.tsx

@@ -91,10 +91,10 @@ const CrudSheet = dynamic(
   },
 );
 
-const CrudImportDialog = dynamic(
+const ImportDialog = dynamic(
   () =>
-    import("./crud-import-dialog").then((m) => ({
-      default: m.CrudImportDialog,
+    import("../../import/import-dialog").then((m) => ({
+      default: m.ImportDialog,
     })),
   {
     ssr: false, // Import dialog doesn't need SSR
@@ -139,7 +139,6 @@ function CrudPageContent({
   dictionary,
   customActions,
 }: CrudPageContentProps) {
-
   const router = useRouter();
   const params = useParams();
   const lang = (params?.lang as string) || "vi";
@@ -861,9 +860,9 @@ function CrudPageContent({
                   <div className="flex items-center gap-2 flex-wrap sm:flex-nowrap">
                     {permissions.import && config.features?.import && (
                       <Suspense fallback={null}>
-                        <CrudImportDialog
+                        <ImportDialog
                           config={config}
-                          endpoint={(() => {
+                          apiUrl={(() => {
                             // Handle endpoint with query params correctly
                             const baseUrl = config.apiEndpoint.split("?")[0];
                             const queryParams = config.apiEndpoint.includes("?")
@@ -913,9 +912,7 @@ function CrudPageContent({
                 onEdit={handleEdit}
                 onDelete={handleDelete}
                 onCustomAction={handleCustomAction}
-                onRowClick={
-                  showDetailOnRowClick ? handleRowClick : undefined
-                }
+                onRowClick={showDetailOnRowClick ? handleRowClick : undefined}
                 onTableReady={setTableInstance}
                 onEmptyStateAction={{
                   onCreate: handleCreate,
@@ -934,9 +931,7 @@ function CrudPageContent({
                   onEdit={handleEdit}
                   onDelete={handleDelete}
                   onCustomAction={handleCustomAction}
-                  onRowClick={
-                    showDetailOnRowClick ? handleRowClick : undefined
-                  }
+                  onRowClick={showDetailOnRowClick ? handleRowClick : undefined}
                   onEmptyStateAction={{
                     onCreate: handleCreate,
                     onClearSearch: () => setSearch(""),

package/src/import/__tests__/import-dialog.test.tsx

@@ -0,0 +1,141 @@
+/**
+ * Hộp thoại nhập hợp nhất. Ba điều dễ vỡ:
+ *  - cổng quyền phải nằm SAU mọi hook (bản CrudImportDialog cũ đặt trước nên
+ *    khi quyền đổi false → true React ném "Rendered more hooks…"),
+ *  - hai hợp đồng server (harness workbook và CRUD theo EntityConfig) phải quy
+ *    về một mô hình hiển thị,
+ *  - 202 nghĩa là nhập nền: đóng dialog, KHÔNG hiện báo cáo "0 dòng".
+ */
+import { render, screen, waitFor } from "@testing-library/react";
+import { afterEach, describe, expect, it, vi } from "vitest";
+
+import { ImportDialog, normalizeImportResult } from "../import-dialog";
+
+const toastSuccess = vi.fn();
+vi.mock("sonner", () => ({
+  toast: {
+    success: (...args: unknown[]) => toastSuccess(...args),
+    error: vi.fn(),
+  },
+}));
+
+afterEach(() => {
+  vi.restoreAllMocks();
+  toastSuccess.mockClear();
+});
+
+describe("normalizeImportResult", () => {
+  it("đọc được hợp đồng harness workbook", () => {
+    expect(
+      normalizeImportResult({
+        success: true,
+        totalRows: 10,
+        importedCount: 8,
+        skippedCount: 2,
+        errors: [{ row: 3, column: "Số lượng", message: "phải là số" }],
+        warnings: [{ row: 4, message: "trùng mã" }],
+      }),
+    ).toEqual({
+      success: true,
+      totalRows: 10,
+      importedCount: 8,
+      skippedCount: 2,
+      errors: [{ row: 3, label: "Số lượng", message: "phải là số" }],
+      warnings: [{ row: 4, message: "trùng mã" }],
+    });
+  });
+
+  it("đọc được hợp đồng CRUD (imported/failed/field) và tự suy totalRows", () => {
+    const r = normalizeImportResult({
+      success: false,
+      imported: 3,
+      failed: 1,
+      errors: [{ row: 2, field: "email", message: "sai định dạng" }],
+    });
+    expect(r.importedCount).toBe(3);
+    expect(r.skippedCount).toBe(1);
+    expect(r.totalRows).toBe(4);
+    expect(r.errors[0].label).toBe("email");
+    expect(r.warnings).toBeUndefined();
+  });
+
+  it("lỗi không kèm cột lẫn field thì nhãn để trống, không bịa", () => {
+    const r = normalizeImportResult({
+      success: false,
+      errors: [{ row: 0, message: "File rỗng" }],
+    });
+    expect(r.errors[0]).toEqual({
+      row: 0,
+      label: undefined,
+      message: "File rỗng",
+    });
+  });
+
+  it("chịu được phản hồi rỗng/rác mà không ném", () => {
+    expect(normalizeImportResult(null)).toMatchObject({
+      success: false,
+      importedCount: 0,
+      errors: [],
+    });
+    expect(normalizeImportResult({ errors: "x" }).errors).toEqual([]);
+  });
+});
+
+describe("ImportDialog", () => {
+  it("không có quyền thì không render gì, và BẬT quyền giữa chừng không vỡ hook", () => {
+    const { rerender, container } = render(
+      <ImportDialog apiUrl="/api/x/import" canImport={false} />,
+    );
+    expect(container.innerHTML).toBe("");
+
+    // Chính là ca làm vỡ bản cũ (early return đứng trước useMemo/useCallback).
+    rerender(<ImportDialog apiUrl="/api/x/import" canImport />);
+    expect(screen.getByRole("button", { name: "Import" })).toBeTruthy();
+
+    rerender(<ImportDialog apiUrl="/api/x/import" canImport={false} />);
+    expect(container.innerHTML).toBe("");
+  });
+
+  it("chế độ điều khiển ngoài: mở sẵn, không render nút kích hoạt", () => {
+    render(
+      <ImportDialog apiUrl="/api/x/import" open onOpenChange={() => {}} />,
+    );
+    expect(screen.getByText("Import dữ liệu")).toBeTruthy();
+    expect(screen.getByText("Bước 1: Tải file mẫu")).toBeTruthy();
+
+    // Chỉ còn nút gửi trong footer (đang tắt vì chưa chọn file) — không có nút mở.
+    const importButtons = screen.getAllByRole("button", { name: "Import" });
+    expect(importButtons).toHaveLength(1);
+    expect((importButtons[0] as HTMLButtonElement).disabled).toBe(true);
+  });
+
+  it("tải mẫu từ server lấy tên file theo content-disposition", async () => {
+    const createUrl = vi
+      .spyOn(URL, "createObjectURL")
+      .mockReturnValue("blob:x");
+    vi.spyOn(URL, "revokeObjectURL").mockImplementation(() => {});
+    global.fetch = vi.fn(
+      async () =>
+        new Response(new Blob(["x"]), {
+          headers: {
+            "content-disposition": 'attachment; filename="mau-don-mua.xlsx"',
+          },
+        }),
+    ) as unknown as typeof fetch;
+
+    render(
+      <ImportDialog
+        apiUrl="/api/purchase-orders/import"
+        templateApiUrl="/api/purchase-orders/import/template"
+        open
+        onOpenChange={() => {}}
+      />,
+    );
+    screen.getByText("Tải mẫu XLSX").click();
+
+    await waitFor(() => expect(createUrl).toHaveBeenCalled());
+    expect(String((global.fetch as any).mock.calls[0][0])).toContain(
+      "/import/template",
+    );
+  });
+});

package/src/import/__tests__/import-engine.test.ts

@@ -0,0 +1,235 @@
+/**
+ * Harness nhập workbook. Tập trung vào chỗ dữ liệu thật hay làm vỡ: người dùng
+ * dán từ Excel bản địa nên số là "1.234.567,89" và ngày là "31/07/2026", còn
+ * tên cột thì lệch hoa/thường và thừa khoảng trắng.
+ */
+import * as XLSX from "xlsx";
+import { describe, expect, it } from "vitest";
+
+import {
+  buildValidationReport,
+  findSheet,
+  getHeaders,
+  normalizeRow,
+  parseExcelBuffer,
+  parseFlexibleDate,
+  parseVietnameseNumber,
+  validateHeaders,
+} from "../import-engine";
+import type { ImportColumnDef } from "../types";
+
+const COLUMNS: ImportColumnDef[] = [
+  { excelHeader: "Mã hàng", fieldKey: "code", type: "string", required: true },
+  { excelHeader: "Số lượng", fieldKey: "qty", type: "number", required: true },
+  { excelHeader: "Ngày", fieldKey: "date", type: "date" },
+  {
+    excelHeader: "Loại",
+    fieldKey: "kind",
+    type: "enum",
+    enumValues: ["gold", "silver"],
+    defaultValue: "gold",
+  },
+];
+
+describe("parseVietnameseNumber", () => {
+  it("đọc được cách viết Việt Nam và cách viết Anh", () => {
+    expect(parseVietnameseNumber("1.234.567,89")).toBe(1234567.89);
+    expect(parseVietnameseNumber("1234,56")).toBe(1234.56);
+    expect(parseVietnameseNumber("1234.56")).toBe(1234.56);
+    expect(parseVietnameseNumber(" 42 ")).toBe(42);
+    expect(parseVietnameseNumber(42)).toBe(42);
+  });
+
+  it("trả NaN cho thứ không phải số — caller phân biệt được với 0", () => {
+    expect(parseVietnameseNumber("abc")).toBeNaN();
+    expect(parseVietnameseNumber(null)).toBeNaN();
+  });
+});
+
+describe("parseFlexibleDate", () => {
+  it("đọc dd/MM/yyyy theo lối Việt Nam, KHÔNG phải MM/dd", () => {
+    const d = parseFlexibleDate("03/07/2026")!;
+    expect(d.getDate()).toBe(3);
+    expect(d.getMonth()).toBe(6); // tháng 7
+    expect(d.getFullYear()).toBe(2026);
+  });
+
+  it("đọc yyyy-MM-dd và số serial của Excel", () => {
+    const iso = parseFlexibleDate("2026-07-31")!;
+    expect(iso.getMonth()).toBe(6);
+    expect(iso.getDate()).toBe(31);
+    // 45000 = 2023-03-15 tính từ mốc 1899-12-30
+    const serial = parseFlexibleDate(45000)!;
+    expect(serial.getFullYear()).toBe(2023);
+    expect(serial.getMonth()).toBe(2);
+    expect(serial.getDate()).toBe(15);
+  });
+
+  it("trả null cho rác thay vì Invalid Date", () => {
+    expect(parseFlexibleDate("hôm qua")).toBeNull();
+    expect(parseFlexibleDate(new Date("x"))).toBeNull();
+    expect(parseFlexibleDate({})).toBeNull();
+  });
+
+  it("từ chối ngày không tồn tại thay vì cuộn sang tháng sau", () => {
+    // 31/06 và 29/02/2026 là lỗi gõ; cuộn ngầm sẽ đổi ngày chứng từ.
+    expect(parseFlexibleDate("31/06/2026")).toBeNull();
+    expect(parseFlexibleDate("29/02/2026")).toBeNull();
+    expect(parseFlexibleDate("32/13/2026")).toBeNull();
+    expect(parseFlexibleDate("2026-06-31")).toBeNull();
+    expect(parseFlexibleDate("29/02/2024")).not.toBeNull(); // năm nhuận thì hợp lệ
+  });
+});
+
+describe("validateHeaders", () => {
+  it("bỏ qua lệch hoa/thường và khoảng trắng thừa", () => {
+    expect(validateHeaders([" mã  HÀNG ", "Số lượng"], COLUMNS)).toEqual([]);
+  });
+
+  it("chỉ báo thiếu cột BẮT BUỘC", () => {
+    const errors = validateHeaders(["Mã hàng"], COLUMNS);
+    expect(errors).toHaveLength(1);
+    expect(errors[0].column).toBe("Số lượng");
+  });
+});
+
+describe("normalizeRow", () => {
+  it("ánh xạ header lệch chuẩn, ép kiểu và trim", () => {
+    const { data, errors } = normalizeRow(
+      { " MÃ hàng ": "  SP01  ", "Số lượng": "1.500,5", Ngày: "31/07/2026" },
+      COLUMNS,
+      1,
+    );
+    expect(errors).toEqual([]);
+    expect(data.code).toBe("SP01");
+    expect(data.qty).toBe(1500.5);
+    expect((data.date as Date).getDate()).toBe(31);
+  });
+
+  it("ô trống bắt buộc thì báo lỗi, ô trống tuỳ chọn thì lấy mặc định", () => {
+    const { data, errors } = normalizeRow({ "Mã hàng": "SP01" }, COLUMNS, 2);
+    expect(errors.map((e) => e.column)).toEqual(["Số lượng"]);
+    expect(data.qty).toBeNull();
+    expect(data.kind).toBe("gold"); // defaultValue
+    expect(data.date).toBeNull();
+  });
+
+  it("số/ngày sai vẫn ghi giá trị an toàn để caller không phải kiểm null khắp nơi", () => {
+    const { data, errors } = normalizeRow(
+      { "Mã hàng": "SP01", "Số lượng": "abc", Ngày: "32/13/2026" },
+      COLUMNS,
+      3,
+    );
+    expect(errors).toHaveLength(2);
+    expect(data.qty).toBe(0);
+    expect(data.date).toBeNull();
+  });
+
+  it("enum khớp không phân biệt hoa/thường, trả về đúng giá trị chuẩn", () => {
+    expect(
+      normalizeRow(
+        { "Mã hàng": "A", "Số lượng": "1", Loại: "SILVER" },
+        COLUMNS,
+        4,
+      ).data.kind,
+    ).toBe("silver");
+
+    const bad = normalizeRow(
+      { "Mã hàng": "A", "Số lượng": "1", Loại: "bạch kim" },
+      COLUMNS,
+      5,
+    );
+    expect(bad.errors).toHaveLength(1);
+    expect(bad.data.kind).toBe("gold"); // rơi về mặc định
+  });
+
+  it("chạy validate riêng của cột", () => {
+    const cols: ImportColumnDef[] = [
+      {
+        excelHeader: "Số lượng",
+        fieldKey: "qty",
+        type: "number",
+        validate: (v, row) =>
+          (v as number) > 0
+            ? null
+            : { row, column: "Số lượng", value: v, message: "Phải dương" },
+      },
+    ];
+    expect(normalizeRow({ "Số lượng": "-3" }, cols, 7).errors[0].message).toBe(
+      "Phải dương",
+    );
+  });
+});
+
+describe("buildValidationReport", () => {
+  const base = {
+    totalRows: 10,
+    importedCount: 8,
+    skippedCount: 2,
+    warnings: [],
+    importedIds: ["a", "b"],
+    dryRun: false,
+  };
+  const oneError = [{ row: 1, column: "x", value: null, message: "sai" }];
+
+  it("'full' hỏng vì một lỗi, 'partial' thì không", () => {
+    expect(
+      buildValidationReport({ ...base, mode: "full", errors: oneError })
+        .success,
+    ).toBe(false);
+    expect(
+      buildValidationReport({ ...base, mode: "partial", errors: oneError })
+        .success,
+    ).toBe(true);
+  });
+
+  it("chạy thử không được khoe đã nhập gì", () => {
+    const r = buildValidationReport({
+      ...base,
+      mode: "partial",
+      errors: [],
+      dryRun: true,
+    });
+    expect(r.success).toBe(true);
+    expect(r.importedCount).toBe(0);
+    expect(r.importedIds).toEqual([]);
+  });
+
+  it("chạy thử có lỗi thì KHÔNG thành công, kể cả chế độ partial", () => {
+    expect(
+      buildValidationReport({
+        ...base,
+        mode: "partial",
+        errors: oneError,
+        dryRun: true,
+      }).success,
+    ).toBe(false);
+  });
+});
+
+describe("parseExcelBuffer + tiện ích sheet", () => {
+  function workbookBuffer(): ArrayBuffer {
+    const wb = XLSX.utils.book_new();
+    const ws = XLSX.utils.aoa_to_sheet([
+      ["Mã hàng", "Số lượng"],
+      ["SP01", 3],
+      ["SP02", ""],
+    ]);
+    XLSX.utils.book_append_sheet(wb, ws, "Dữ liệu");
+    return XLSX.write(wb, { type: "array", bookType: "xlsx" }) as ArrayBuffer;
+  }
+
+  it("đọc buffer thật thành dòng, ô trống là chuỗi rỗng chứ không undefined", () => {
+    const parsed = parseExcelBuffer(workbookBuffer());
+    const rows = findSheet(parsed, "  dữ LIỆU ")!;
+    expect(rows).toHaveLength(2);
+    expect(rows[0]["Mã hàng"]).toBe("SP01");
+    expect(rows[1]["Số lượng"]).toBe("");
+    expect(getHeaders(rows)).toEqual(["Mã hàng", "Số lượng"]);
+  });
+
+  it("findSheet trả null khi không có sheet; getHeaders chịu được mảng rỗng", () => {
+    expect(findSheet(parseExcelBuffer(workbookBuffer()), "Sheet2")).toBeNull();
+    expect(getHeaders([])).toEqual([]);
+  });
+});