create-ailk 0.18.0 → 0.19.1

package/templates/apps/web/app/[locale]/(authed)/account/__tests__/page.test.tsx

@@ -1,12 +1,13 @@
 /**
  * Tests for /[locale]/account — the base account Inbox (#7430, spec 7423
- * U4 · D6a, D9, D12; AC-7 as amended 2026-10-01).
+ * U4 · D6a, D9, D12; AC-7 as amended 2026-10-01; #8231 D3, D5–D7).
  *
  * Covers:
- *   1. The D6a tabs Recents · Suggested · Scheduled · All, each with its
- *      count (zeros included), Recents selected.
- *   2. `result` and `purchase` rows sit under Recents and All, newest first,
- *      each linking to its item page.
+ *   1. The tabs All · Purchases · Scheduled, each with its count (zeros
+ *      included), All selected (#8231 D3).
+ *   2. All holds every row newest first, each linking to its item page;
+ *      Purchases holds the kinds whose section is `purchases`; Scheduled the
+ *      `upcoming` kinds, upcoming soonest first, then past newest first.
  *   3. `email-unverified` and `payment-due` are For-you items (the phone
  *      For you tab), never Inbox rows.
  *   4. A member with no name gets an "Add your name" For-you item (needs-you,
@@ -20,13 +21,13 @@
  *   7. #7920 — the member area's `piece` and `message` kinds and the
  *      needs-client For-you source. Each reads the hub with `ctx.init`
  *      unchanged (the caller's own credential); 404 and an unverified 401/403
- *      load `[]`, any other failure rejects (D12). With no hub credential on
- *      `ctx.init` — a cookie-session site — they make no call and load `[]`.
+ *      load `[]`, any other failure rejects (D12). On a cookie-session site
+ *      they call with the cookie too — no gate (#8231 D7).
  */
 import { fireEvent, render, screen, within } from "@testing-library/react";
 
 // The global setup mock resolves only top-level namespace keys against
-// en.json; the area reads nested keys (`inbox.tabs.recents`, …) — a call
+// en.json; the area reads nested keys (`inbox.tabs.all`, …) — a call
 // shape real next-intl supports — so resolve dotted namespaces AND dotted
 // keys here against the same catalog. `{name}` placeholders are filled from
 // `values` the way next-intl does.
@@ -67,18 +68,26 @@ jest.mock("../../../../../auth", () => ({
 }));
 
 const mockSignupMe = jest.fn();
-const mockCheckoutsMe = jest.fn();
 const mockSubscriptionMe = jest.fn();
 const mockMemberPieces = jest.fn();
 const mockMemberMessages = jest.fn();
+const mockMemberPurchases = jest.fn();
+const mockMemberAppointments = jest.fn();
 jest.mock("@working-theory/api-client", () => ({
   __esModule: true,
   waitlistSignups: { me: (...args: unknown[]) => mockSignupMe(...args) },
-  flowCheckouts: { me: (...args: unknown[]) => mockCheckoutsMe(...args) },
   subscription: { me: (...args: unknown[]) => mockSubscriptionMe(...args) },
   member: {
     pieces: (...args: unknown[]) => mockMemberPieces(...args),
     messages: (...args: unknown[]) => mockMemberMessages(...args),
+    purchases: (...args: unknown[]) => mockMemberPurchases(...args),
+    appointments: (...args: unknown[]) => mockMemberAppointments(...args),
+    // #8233 C4 — the New menu's availability reads; 404 here, so the Inbox
+    // cases see no New action and no error row.
+    meetingTypes: () =>
+      Promise.resolve({ ok: false, status: 404, body: { ok: false, error: { code: "not_found", message: "Not found" } } }),
+    requestForm: () =>
+      Promise.resolve({ ok: false, status: 404, body: { ok: false, error: { code: "not_found", message: "Not found" } } }),
   },
 }));
 
@@ -123,19 +132,21 @@ const SIGNUP = {
   answers: { q3: null },
 };
 
-const CHECKOUT = {
-  id: "fc_done",
-  flowSlug: "founder-flow",
-  experimentKey: "ailk-founder-diagnostic",
-  offerKey: "pro",
-  status: "completed",
+const PURCHASE = {
+  id: "pur_done",
+  offering: "pro",
   amountMinor: 4900,
   currency: "usd",
   period: "monthly",
-  completedAt: "2026-09-04T10:00:00.000Z",
-  createdAt: "2026-09-04T09:58:00.000Z",
+  purchasedAt: "2026-09-04T10:00:00.000Z",
 };
 
+function appointment(id: string, title: string, startTime: string) {
+  return { id, title, startTime, endTime: startTime, status: "confirmed" as const };
+}
+
+const NO_APPOINTMENTS = { ok: true, body: { upcoming: [], past: [] } };
+
 const NO_SUBSCRIPTION = {
   ok: true,
   body: { actionRequired: false, status: null, accessRevokesAt: null, portalAvailable: false },
@@ -236,74 +247,133 @@ beforeEach(() => {
   accountExperimentKeys.length = 0;
   mockGetSession.mockReset().mockResolvedValue(SESSION);
   mockSignupMe.mockReset().mockResolvedValue({ ok: true, body: SIGNUP });
-  mockCheckoutsMe.mockReset().mockResolvedValue({ ok: true, body: [CHECKOUT] });
   mockSubscriptionMe.mockReset().mockResolvedValue(NO_SUBSCRIPTION);
-  mockMemberPieces.mockReset().mockResolvedValue({ ok: true, body: { items: [PIECE, PIECE_NEEDS_CLIENT] } });
-  mockMemberMessages.mockReset().mockResolvedValue({ ok: true, body: { items: [MESSAGE, MESSAGE_WITH_FILE] } });
+  mockMemberPurchases.mockReset().mockResolvedValue({ ok: true, body: { items: [PURCHASE] } });
+  mockMemberAppointments.mockReset().mockResolvedValue(NO_APPOINTMENTS);
+  // The page's pieces and messages answer 404 unless a case fills them.
+  mockMemberPieces.mockReset().mockResolvedValue(notFound);
+  mockMemberMessages.mockReset().mockResolvedValue(notFound);
   mockRefresh.mockReset();
 });
 
+/** The member kinds' own cases read rows from both member reads. */
+function memberRows(): void {
+  mockMemberPieces.mockResolvedValue({ ok: true, body: { items: [PIECE, PIECE_NEEDS_CLIENT] } });
+  mockMemberMessages.mockResolvedValue({ ok: true, body: { items: [MESSAGE, MESSAGE_WITH_FILE] } });
+}
+
 afterAll(() => {
   accountExperimentKeys.length = 0;
 });
 
-describe("the Inbox — D6a tabs (AC-7)", () => {
-  it("renders Recents · Suggested · Scheduled · All with counts, Recents selected", async () => {
+describe("the Inbox — tabs All · Purchases · Scheduled (#8231 D3, AC-3)", () => {
+  /** The rows of one tab, by title, after selecting it. */
+  function tabRows(id: string): string[] {
+    fireEvent.mouseDown(screen.getByTestId(`inbox-tab-${id}`));
+    return within(screen.getByTestId(`inbox-${id}`))
+      .queryAllByTestId(`inbox-${id}-row`)
+      .map((row) => within(row).getByRole("link").textContent ?? "");
+  }
+
+  it("renders All · Purchases · Scheduled with counts, All selected — no Recents or Suggested", async () => {
     accountExperimentKeys.push("ailk-founder-diagnostic");
+    mockMemberPieces.mockResolvedValue({ ok: true, body: { items: [PIECE] } });
+    mockMemberMessages.mockResolvedValue({ ok: true, body: { items: [MESSAGE] } });
+    mockMemberAppointments.mockResolvedValue({
+      ok: true,
+      body: { upcoming: [appointment("appt_1", "Kickoff call", "2026-09-10T15:00:00.000Z")], past: [] },
+    });
     await renderPage();
     expect(screen.getByRole("heading", { level: 1 })).toHaveTextContent("Inbox");
     const tabs = screen.getAllByRole("tab");
     expect(tabs.map((tab) => tab.getAttribute("data-testid"))).toEqual([
-      "inbox-tab-recents",
-      "inbox-tab-suggested",
-      "inbox-tab-scheduled",
       "inbox-tab-all",
+      "inbox-tab-purchases",
+      "inbox-tab-scheduled",
     ]);
-    expect(tabs.map((tab) => tab.textContent)).toEqual([
-      "Recents2",
-      "Suggested0",
-      "Scheduled0",
-      "All2",
+    expect(tabs.map((tab) => tab.textContent)).toEqual(["All5", "Purchases1", "Scheduled1"]);
+    expect(screen.getByTestId("inbox-tab-all")).toHaveAttribute("aria-selected", "true");
+    // All: every row, newest first.
+    const all = within(screen.getByTestId("inbox-all")).getAllByTestId("inbox-all-row");
+    expect(all.map((row) => row.textContent)).toEqual([
+      expect.stringContaining("Kickoff call"),
+      expect.stringContaining("Kickoff notes"),
+      expect.stringContaining("Spring newsletter"),
+      expect.stringContaining("pro"),
+      expect.stringContaining("ailk-founder-diagnostic"),
     ]);
-    expect(screen.getByTestId("inbox-tab-recents")).toHaveAttribute("aria-selected", "true");
+    // Purchases: the purchase row only. Scheduled: the appointment only.
+    expect(tabRows("purchases")).toEqual([expect.stringContaining("pro")]);
+    expect(tabRows("scheduled")).toEqual([expect.stringContaining("Kickoff call")]);
+    // The result row is under All and in no other tab.
+    expect(screen.queryByText("ailk-founder-diagnostic")).not.toBeInTheDocument();
   });
 
-  it("result and purchase rows sit under Recents, newest first, each linking to its item page", async () => {
+  it("each All row links to its item page, prefixed off the default locale", async () => {
     accountExperimentKeys.push("ailk-founder-diagnostic");
-    await renderPage();
-    const recents = screen.getByTestId("inbox-recents");
-    const rows = within(recents).getAllByTestId("inbox-recents-row");
-    expect(rows).toHaveLength(2);
-    // The purchase (Sep 4) is newer than the result (Sep 3).
-    expect(rows[0]).toHaveTextContent("pro");
-    expect(rows[1]).toHaveTextContent("ailk-founder-diagnostic");
-    expect(within(recents).getByText("pro").closest("a")).toHaveAttribute(
-      "href",
-      "/account/inbox/purchase/fc_done",
-    );
-    expect(
-      within(recents).getByText("ailk-founder-diagnostic").closest("a"),
-    ).toHaveAttribute("href", "/account/inbox/result/ailk-founder-diagnostic");
-  });
-
-  it("the same rows are in All; a non-default locale prefixes the item href", async () => {
     await renderPage("fr");
-    fireEvent.mouseDown(screen.getByTestId("inbox-tab-all"));
     const all = screen.getByTestId("inbox-all");
     expect(within(all).getByText("pro").closest("a")).toHaveAttribute(
       "href",
-      "/fr/account/inbox/purchase/fc_done",
+      "/fr/account/inbox/purchase/pur_done",
     );
+    expect(
+      within(all).getByText("ailk-founder-diagnostic").closest("a"),
+    ).toHaveAttribute("href", "/fr/account/inbox/result/ailk-founder-diagnostic");
+  });
+
+  it("Scheduled lists the upcoming soonest first, then the past newest first", async () => {
+    mockMemberAppointments.mockResolvedValue({
+      ok: true,
+      body: {
+        // Out of order on purpose: the page orders them, not the read.
+        upcoming: [
+          appointment("u1", "U1", "2026-10-25T15:00:00.000Z"),
+          appointment("u2", "U2", "2026-10-20T15:00:00.000Z"),
+        ],
+        past: [
+          appointment("p1", "P1", "2026-09-01T15:00:00.000Z"),
+          appointment("p2", "P2", "2026-09-05T15:00:00.000Z"),
+        ],
+      },
+    });
+    await renderPage();
+    expect(tabRows("scheduled")).toEqual([
+      expect.stringContaining("U2"),
+      expect.stringContaining("U1"),
+      expect.stringContaining("P2"),
+      expect.stringContaining("P1"),
+    ]);
   });
 
-  it("with no rows, every tab still renders and Recents shows the empty line", async () => {
-    mockCheckoutsMe.mockResolvedValue({ ok: true, body: [] });
+  it("a kind declaring section purchases lands under Purchases, not Scheduled", async () => {
+    const licence: (typeof accountInboxKinds)[number] = {
+      kind: "licence",
+      label: "purchases.heading",
+      section: "purchases",
+      load: async () => [{ id: "lic_1", title: "Site licence", at: "2026-09-05T00:00:00.000Z", data: null }],
+      renderDetail: () => ({}),
+    };
+    accountInboxKinds.push(licence);
+    try {
+      await renderPage();
+      expect(tabRows("purchases")).toEqual([
+        expect.stringContaining("Site licence"),
+        expect.stringContaining("pro"),
+      ]);
+      expect(tabRows("scheduled")).toEqual([]);
+    } finally {
+      accountInboxKinds.splice(accountInboxKinds.indexOf(licence), 1);
+    }
+  });
+
+  it("with no rows, every tab still renders and All shows the empty line", async () => {
+    mockMemberPurchases.mockResolvedValue({ ok: true, body: { items: [] } });
     await renderPage();
     expect(screen.getAllByRole("tab").map((tab) => tab.textContent)).toEqual([
-      "Recents0",
-      "Suggested0",
-      "Scheduled0",
       "All0",
+      "Purchases0",
+      "Scheduled0",
     ]);
     expect(screen.getByTestId("inbox-empty")).toHaveTextContent("Nothing here yet.");
   });
@@ -333,7 +403,7 @@ describe("For-you items, not Inbox rows (AC-7, D9 as amended)", () => {
     expect(within(item).getByTestId("for-you-tag").querySelector("svg")).not.toBeNull();
     expect(within(item).getByTestId("for-you-open")).toHaveAttribute(
       "href",
-      "/account/settings#general",
+      "/account/general",
     );
   });
 });
@@ -435,7 +505,7 @@ describe("the phone For you tab (#7469, DESIGN.md §2b · §5.15)", () => {
 
 describe("Hub unavailable (D12)", () => {
   it("a failed read renders an error row with Retry, never an empty Inbox", async () => {
-    mockCheckoutsMe.mockResolvedValue({ ok: false, status: 500, body: {} });
+    mockMemberPurchases.mockResolvedValue({ ok: false, status: 500, body: {} });
     await renderPage();
     expect(screen.getByTestId("inbox-error")).toHaveTextContent(
       "We couldn't load everything in your Inbox.",
@@ -465,7 +535,7 @@ describe("Hub unavailable (D12)", () => {
   });
 
   it("a failed Inbox-kind read leaves the For you tab without an error row", async () => {
-    mockCheckoutsMe.mockResolvedValue({ ok: false, status: 500, body: {} });
+    mockMemberPurchases.mockResolvedValue({ ok: false, status: 500, body: {} });
     await renderPage("en", "for-you");
     const panel = screen.getByTestId("inbox-panel-for-you");
     expect(within(panel).queryByTestId("inbox-error")).not.toBeInTheDocument();
@@ -473,6 +543,8 @@ describe("Hub unavailable (D12)", () => {
 });
 
 describe("the member kinds — piece and message (#7920, spec 7423 D8 · D12)", () => {
+  beforeEach(memberRows);
+
   it("declares piece and message as Inbox kinds in the updates section, labelled from the account namespace", async () => {
     const ctx = await memberCtx();
     expect(kindOf("piece").section).toBe("updates");
@@ -536,22 +608,39 @@ describe("the member kinds — piece and message (#7920, spec 7423 D8 · D12)",
     await expect(kindOf(id).load(await memberCtx())).rejects.toMatchObject({ status });
   });
 
-  it("with no hub credential on ctx.init (a cookie-session site) both kinds and the source load [] without a call", async () => {
-    const ctx = await memberCtx({ init: { headers: { cookie: "session=abc" } } });
-    await expect(kindOf("piece").load(ctx)).resolves.toEqual([]);
-    await expect(kindOf("message").load(ctx)).resolves.toEqual([]);
-    await expect(needsClientSource().load(ctx)).resolves.toEqual([]);
-    expect(mockMemberPieces).not.toHaveBeenCalled();
-    expect(mockMemberMessages).not.toHaveBeenCalled();
+  it("on a cookie-session site both kinds and the source call with the cookie and return rows (#8231 D7)", async () => {
+    const cookie = { headers: { cookie: "session=abc" } };
+    const ctx = await memberCtx({ init: cookie });
+    expect((await kindOf("piece").load(ctx)).map((p) => p.id)).toEqual([PIECE.id, PIECE_NEEDS_CLIENT.id]);
+    expect((await kindOf("message").load(ctx)).map((m) => m.id)).toEqual([MESSAGE.id, MESSAGE_WITH_FILE.id]);
+    expect((await needsClientSource().load(ctx)).map((i) => i.id)).toEqual([`piece:${PIECE_NEEDS_CLIENT.id}`]);
+    expect(mockMemberPieces).toHaveBeenCalledWith(cookie);
+    expect(mockMemberMessages).toHaveBeenCalledWith(cookie);
   });
 
-  it("the cookie-session Inbox stays as it was: no member rows, no error row, no member call", async () => {
+  it("the cookie-session Inbox lists the member rows under All, read with the forwarded cookie (#8231 D7)", async () => {
     await renderPage();
+    const sent = expect.objectContaining({ headers: expect.objectContaining({ cookie: "session=abc" }) });
+    expect(mockMemberPieces).toHaveBeenCalledWith(sent);
+    expect(mockMemberMessages).toHaveBeenCalledWith(sent);
+    expect(mockMemberPurchases).toHaveBeenCalledWith(sent);
+    expect(mockMemberAppointments).toHaveBeenCalledWith(sent);
+    const all = screen.getByTestId("inbox-all");
+    expect(within(all).getByText("Spring newsletter")).toBeInTheDocument();
+    expect(within(all).getByText("Kickoff notes")).toBeInTheDocument();
     expect(screen.queryByTestId("inbox-error")).not.toBeInTheDocument();
-    expect(screen.queryByText("Spring newsletter")).not.toBeInTheDocument();
-    expect(screen.queryByText("Kickoff notes")).not.toBeInTheDocument();
-    expect(mockMemberPieces).not.toHaveBeenCalled();
-    expect(mockMemberMessages).not.toHaveBeenCalled();
+  });
+
+  it.each([
+    ["pieces", mockMemberPieces],
+    ["messages", mockMemberMessages],
+  ] as const)("a 500 from member.%s shows the Inbox error row with Retry (#8231 D7)", async (_name, mock) => {
+    mock.mockResolvedValue({ ok: false, status: 500, body: {} });
+    await renderPage();
+    const inbox = screen.getByTestId("inbox-panel-inbox");
+    expect(within(inbox).getByTestId("inbox-error")).toBeInTheDocument();
+    fireEvent.click(within(inbox).getByTestId("inbox-retry"));
+    expect(mockRefresh).toHaveBeenCalledTimes(1);
   });
 
   it("a piece's detail shows its stage and dates", async () => {
@@ -598,6 +687,8 @@ describe("the member kinds — piece and message (#7920, spec 7423 D8 · D12)",
 });
 
 describe("the needs-client For-you source (#7920)", () => {
+  beforeEach(memberRows);
+
   it("a needsClient piece is a needs-you For-you item with an Open action to its Inbox row", async () => {
     const items = await needsClientSource().load(await memberCtx());
     expect(items).toEqual([