@isi-ui7/bos7-shared 0.3.2 → 0.3.3

package/src/wizard-step-gate.test.ts

@@ -0,0 +1,65 @@
+import { describe, it, expect, vi } from 'vitest';
+import type { CrudForm } from './form-types';
+
+type D = Record<string, unknown>;
+
+// Gerbang antar-langkah wizard. Diuji sebagai KONTRAK, bukan lewat render:
+// yang penting bentuk & urutan pemanggilannya, dan itu bisa dipastikan tanpa
+// memasang seluruh form.
+//
+// Kenapa diuji sama sekali: gerbang yang gagal-terbuka tidak terlihat sebagai
+// galat — operator sekadar melewati langkah yang seharusnya mengunci, dan
+// akibatnya (dua CIF untuk satu orang) baru muncul jauh kemudian.
+
+function makeSchema(gate: NonNullable<Extract<CrudForm<D>['layout'], { type: 'wizard' }>['canLeavePage']>): CrudForm<D> {
+  return {
+    title: { create: 'c', edit: 'e', view: 'v' },
+    emptyData: {},
+    layout: {
+      type: 'wizard',
+      validateOnNext: true,
+      canLeavePage: gate,
+      pages: [
+        { label: 'Verifikasi', sections: [] },
+        { label: 'Data', sections: [] },
+      ],
+    },
+  } as CrudForm<D>;
+}
+
+describe('wizard canLeavePage', () => {
+  it('menolak pindah ketika gerbang mengembalikan ok:false, dan membawa pesannya', () => {
+    const gate = vi.fn().mockReturnValue({ ok: false, message: 'Verifikasi identitas dulu.' });
+    const schema = makeSchema(gate);
+    const layout = schema.layout as Extract<typeof schema.layout, { type: 'wizard' }>;
+
+    const res = layout.canLeavePage!(0, { verified: false });
+    expect(res.ok).toBe(false);
+    expect(res.message).toBe('Verifikasi identitas dulu.');
+  });
+
+  it('mengizinkan pindah ketika gerbang ok', () => {
+    const layout = makeSchema(() => ({ ok: true })).layout as Extract<
+      CrudForm<D>['layout'],
+      { type: 'wizard' }
+    >;
+    expect(layout.canLeavePage!(0, { verified: true }).ok).toBe(true);
+  });
+
+  it('menerima nomor halaman ASAL, bukan tujuan — gerbang milik langkah yang ditinggalkan', () => {
+    const gate = vi.fn().mockReturnValue({ ok: true });
+    const layout = makeSchema(gate).layout as Extract<CrudForm<D>['layout'], { type: 'wizard' }>;
+    layout.canLeavePage!(0, {});
+    expect(gate).toHaveBeenCalledWith(0, {});
+  });
+
+  it('opsional — wizard tanpa gerbang tetap sah secara tipe', () => {
+    const schema: CrudForm<D> = {
+      title: { create: 'c', edit: 'e', view: 'v' },
+      emptyData: {},
+      layout: { type: 'wizard', pages: [{ label: 'A', sections: [] }] },
+    } as CrudForm<D>;
+    const layout = schema.layout as Extract<typeof schema.layout, { type: 'wizard' }>;
+    expect(layout.canLeavePage).toBeUndefined();
+  });
+});

package/src/workflow/use-crud-form-gate.test.tsx

@@ -0,0 +1,138 @@
+import { describe, it, expect, vi } from "vitest";
+import { act, renderHook } from "@testing-library/react";
+import { useCrudForm } from "./use-crud-form";
+import type { CrudForm } from "../form-types";
+
+type D = Record<string, unknown>;
+
+// Gerbang antar-langkah wizard, diuji lewat PERILAKU goNext.
+//
+// Kenapa ada tes kedua padahal sudah ada wizard-step-gate.test.ts: tes itu
+// hanya memastikan BENTUK tipenya — ia akan tetap hijau meski goNext tidak
+// pernah memanggil canLeavePage sama sekali. Persis kegagalan yang gerbang ini
+// ada untuk mencegahnya: gagal-terbuka tidak berbunyi sebagai galat, operator
+// sekadar melewati langkah yang seharusnya mengunci, dan akibatnya (dua CIF
+// untuk satu orang) baru muncul jauh kemudian.
+//
+// Tes di bawah memeriksa `page` — halaman yang BENAR-BENAR ditempati — bukan
+// nilai balik gerbangnya. Itu satu-satunya yang membedakan "terpasang" dari
+// "sekadar ada".
+
+function schemaWith(
+  gate?: (from: number, d: D) => { ok: boolean; message?: string },
+  opts: { validateOnNext?: boolean } = {},
+): CrudForm<D> {
+  return {
+    title: { create: "c", edit: "e", view: "v" },
+    emptyData: { verified: false } as D,
+    layout: {
+      type: "wizard",
+      validateOnNext: opts.validateOnNext ?? false,
+      canLeavePage: gate,
+      pages: [
+        {
+          label: "Verifikasi",
+          sections: [
+            {
+              title: "s",
+              fields: [
+                {
+                  key: "nama",
+                  label: "Nama",
+                  type: "text",
+                  span: 4,
+                  validation: { required: true },
+                },
+              ],
+            },
+          ],
+        },
+        { label: "Data", sections: [] },
+      ],
+    },
+  } as unknown as CrudForm<D>;
+}
+
+describe("useCrudForm — gerbang canLeavePage", () => {
+  it("gerbang ok:false MENAHAN halaman di 0 dan menampilkan pesannya", async () => {
+    const { result } = renderHook(() =>
+      useCrudForm<D>({
+        schema: schemaWith(() => ({ ok: false, message: "Cek identitas dulu." })),
+        mode: "create",
+      }),
+    );
+
+    expect(result.current.page).toBe(0);
+    await act(async () => {
+      await result.current.goNext();
+    });
+
+    expect(result.current.page).toBe(0);
+    expect(result.current.error).toBe("Cek identitas dulu.");
+  });
+
+  it("gerbang ok:true MEMBIARKAN pindah ke halaman 1", async () => {
+    const { result } = renderHook(() =>
+      useCrudForm<D>({ schema: schemaWith(() => ({ ok: true })), mode: "create" }),
+    );
+
+    await act(async () => {
+      await result.current.goNext();
+    });
+
+    expect(result.current.page).toBe(1);
+    expect(result.current.error).toBeNull();
+  });
+
+  it("gerbang menerima nomor halaman ASAL beserta isi form saat itu", async () => {
+    const gate = vi.fn().mockReturnValue({ ok: true });
+    const { result } = renderHook(() =>
+      useCrudForm<D>({ schema: schemaWith(gate), mode: "create" }),
+    );
+
+    await act(async () => {
+      result.current.setForm({ verified: true });
+    });
+    await act(async () => {
+      await result.current.goNext();
+    });
+
+    expect(gate).toHaveBeenCalledWith(0, { verified: true });
+  });
+
+  it("gerbang berjalan SEBELUM validasi field — pesan yang muncul alasan sebenarnya, bukan galat field", async () => {
+    // `nama` wajib dan kosong, jadi validasi PASTI gagal juga. Kalau urutannya
+    // terbalik, yang tampil adalah galat validasi generik dan operator tidak
+    // pernah tahu bahwa yang sesungguhnya menahan adalah verifikasi identitas.
+    const { result } = renderHook(() =>
+      useCrudForm<D>({
+        schema: schemaWith(() => ({ ok: false, message: "Cek identitas dulu." }), {
+          validateOnNext: true,
+        }),
+        mode: "create",
+      }),
+    );
+
+    await act(async () => {
+      await result.current.goNext();
+    });
+
+    expect(result.current.page).toBe(0);
+    expect(result.current.error).toBe("Cek identitas dulu.");
+    expect(result.current.fieldErrors).toEqual({});
+  });
+
+  it("KONTROL negatif: wizard tanpa gerbang tetap bisa pindah halaman", async () => {
+    // Kalau tes ini ikut gagal ketika gerbang dicabut, berarti yang diukur
+    // bukan gerbangnya melainkan navigasinya secara umum.
+    const { result } = renderHook(() =>
+      useCrudForm<D>({ schema: schemaWith(undefined), mode: "create" }),
+    );
+
+    await act(async () => {
+      await result.current.goNext();
+    });
+
+    expect(result.current.page).toBe(1);
+  });
+});

package/src/workflow/use-crud-form.ts

@@ -100,6 +100,19 @@ export function useCrudForm<TData extends Record<string, unknown>>({
     setServerError(null);
     setHasValidationError(false);
     const layout = schema.layout;
+
+    // Gerbang antar-langkah — diperiksa SEBELUM validasi field, karena gerbang
+    // yang gagal berarti langkah ini belum boleh ditinggalkan sama sekali;
+    // menjalankan validasi lebih dulu akan menampilkan galat field yang tidak
+    // relevan dan menutupi alasan sebenarnya.
+    if (layout.type === "wizard" && layout.canLeavePage) {
+      const gate = layout.canLeavePage(page, form);
+      if (!gate.ok) {
+        setServerError(gate.message ?? "Langkah ini belum bisa dilanjutkan.");
+        return;
+      }
+    }
+
     if (layout.type === "wizard" && layout.validateOnNext) {
       const errs = validatePage(page);
       setFieldErrors(errs);