@portnet/ui 6.0.2 → 6.0.3

package/dist/v2/__tests__/rowActions.test.js

@@ -0,0 +1,186 @@
+"use strict";
+
+require("@testing-library/jest-dom");
+var _react = require("@testing-library/react");
+var _userEvent = _interopRequireDefault(require("@testing-library/user-event"));
+var _rowActions = require("../patterns/rowActions");
+var _jsxRuntime = require("react/jsx-runtime");
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+/**
+ * @jest-environment jsdom
+ */
+
+/**
+ * rowActions (G-41) — promoted from `refonte-frontend` (Area 5).
+ *
+ * Each assertion pins a Charter rule the descriptor layer carries, or a defect
+ * the promoted implementation had been written to prevent. The per-row
+ * evaluation of the predicates is the load-bearing part: a boolean would force
+ * the host to rebuild the array per row, which is how a table row and a mobile
+ * card end up offering different actions for the same record.
+ */
+
+const BROUILLON = {
+  id: 1,
+  status: "DRAFT"
+};
+const SOUMIS = {
+  id: 2,
+  status: "SOUMIS"
+};
+describe("buildRowActionMenu", () => {
+  test("hidden is evaluated PER ROW, not once for the array", () => {
+    const descriptors = [{
+      id: "open",
+      label: "Consulter",
+      onClick: jest.fn()
+    }, {
+      id: "del",
+      label: "Supprimer",
+      onClick: jest.fn(),
+      hidden: row => row.status !== "DRAFT"
+    }];
+    expect((0, _rowActions.buildRowActionMenu)(descriptors, BROUILLON)).toHaveLength(2);
+    expect((0, _rowActions.buildRowActionMenu)(descriptors, SOUMIS)).toHaveLength(1);
+  });
+  test("disabled is evaluated per row too, and keeps the item visible", () => {
+    const items = (0, _rowActions.buildRowActionMenu)([{
+      id: "edit",
+      label: "Modifier",
+      onClick: jest.fn(),
+      disabled: row => row.status !== "DRAFT"
+    }], SOUMIS);
+    expect(items).toHaveLength(1);
+    expect(items[0]?.disabled).toBe(true);
+  });
+
+  /** R-B.6 — a destructive action is separated and marked, not merely last. */
+  test("a divider precedes the first destructive action", () => {
+    const items = (0, _rowActions.buildRowActionMenu)([{
+      id: "open",
+      label: "Consulter",
+      onClick: jest.fn()
+    }, {
+      id: "del",
+      label: "Supprimer",
+      onClick: jest.fn(),
+      danger: true
+    }, {
+      id: "purge",
+      label: "Purger",
+      onClick: jest.fn(),
+      danger: true
+    }], BROUILLON);
+    expect(items.map(i => Boolean(i.divider))).toEqual([false, true, false, false]);
+  });
+
+  /**
+   * A menu does not open on a rule. The separator only appears if something
+   * precedes it — a row whose ONLY action is destructive gets no divider.
+   */
+  test("no leading divider when the destructive action comes first", () => {
+    const items = (0, _rowActions.buildRowActionMenu)([{
+      id: "del",
+      label: "Supprimer",
+      onClick: jest.fn(),
+      danger: true
+    }], BROUILLON);
+    expect(items).toHaveLength(1);
+    expect(items[0]?.divider).toBeUndefined();
+  });
+  test("nothing visible yields an empty list, so a caller can skip the trigger", () => {
+    expect((0, _rowActions.buildRowActionMenu)([{
+      id: "del",
+      label: "Supprimer",
+      onClick: jest.fn(),
+      hidden: () => true
+    }], BROUILLON)).toEqual([]);
+  });
+  test("onClick is bound to the row it was built for", () => {
+    const onClick = jest.fn();
+    const items = (0, _rowActions.buildRowActionMenu)([{
+      id: "open",
+      label: "Consulter",
+      onClick
+    }], SOUMIS);
+    items[0]?.onClick?.();
+    expect(onClick).toHaveBeenCalledWith(SOUMIS);
+  });
+
+  /**
+   * R-3.3 caps what one screen may offer at once, and `OverflowMenu`'s own
+   * contract recommends ≤ 7 items. Exceeding it is a host decision, so it warns
+   * in development rather than truncating — silently dropping an action would
+   * hide a business capability.
+   */
+  test(`warns past ${_rowActions.MAX_ROW_ACTIONS} visible actions, without dropping any`, () => {
+    const warn = jest.spyOn(console, "warn").mockImplementation(() => undefined);
+    const descriptors = Array.from({
+      length: _rowActions.MAX_ROW_ACTIONS + 1
+    }, (_, i) => ({
+      id: `a${i}`,
+      label: `Action ${i}`,
+      onClick: jest.fn()
+    }));
+    const items = (0, _rowActions.buildRowActionMenu)(descriptors, BROUILLON);
+    expect(items).toHaveLength(_rowActions.MAX_ROW_ACTIONS + 1);
+    expect(warn).toHaveBeenCalled();
+    warn.mockRestore();
+  });
+});
+describe("buildRowActions", () => {
+  /**
+   * R-K.1 — the reason this layer exists. The implementation it replaced
+   * revealed its actions on hover, with a `mouseup` listener on `document` and
+   * no keyboard equivalent: consulting or deleting a record was unreachable
+   * without a mouse. `OverflowMenu` is an ARIA menu, so the trigger is a
+   * button and the items are reachable by keyboard.
+   */
+  test("renders a keyboard-reachable trigger, named by the caller", async () => {
+    const onClick = jest.fn();
+    const render$ = (0, _rowActions.buildRowActions)([{
+      id: "open",
+      label: "Consulter le dossier",
+      onClick
+    }], "Actions sur le dossier");
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_jsxRuntime.Fragment, {
+      children: render$(SOUMIS)
+    }));
+    const trigger = _react.screen.getByRole("button", {
+      name: "Actions sur le dossier"
+    });
+    await _userEvent.default.click(trigger);
+    await _userEvent.default.click(_react.screen.getByRole("menuitem", {
+      name: "Consulter le dossier"
+    }));
+    expect(onClick).toHaveBeenCalledWith(SOUMIS);
+  });
+  test("renders nothing when no action applies to the row", () => {
+    const render$ = (0, _rowActions.buildRowActions)([{
+      id: "del",
+      label: "Supprimer",
+      onClick: jest.fn(),
+      hidden: () => true
+    }]);
+    const {
+      container
+    } = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_jsxRuntime.Fragment, {
+      children: render$(BROUILLON)
+    }));
+    expect(container).toBeEmptyDOMElement();
+  });
+  test("the default trigger name is generic, so hosts localise it per resource", () => {
+    const render$ = (0, _rowActions.buildRowActions)([{
+      id: "open",
+      label: "Consulter",
+      onClick: jest.fn()
+    }]);
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_jsxRuntime.Fragment, {
+      children: render$(SOUMIS)
+    }));
+    expect(_react.screen.getByRole("button", {
+      name: "Actions"
+    })).toBeInTheDocument();
+  });
+});
+//# sourceMappingURL=rowActions.test.js.map

package/dist/v2/__tests__/rowActions.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"rowActions.test.js","names":["require","_react","_userEvent","_interopRequireDefault","_rowActions","_jsxRuntime","e","__esModule","default","BROUILLON","id","status","SOUMIS","describe","test","descriptors","label","onClick","jest","fn","hidden","row","expect","buildRowActionMenu","toHaveLength","items","disabled","toBe","danger","map","i","Boolean","divider","toEqual","toBeUndefined","toHaveBeenCalledWith","MAX_ROW_ACTIONS","warn","spyOn","console","mockImplementation","undefined","Array","from","length","_","toHaveBeenCalled","mockRestore","render$","buildRowActions","render","jsx","Fragment","children","trigger","screen","getByRole","name","userEvent","click","container","toBeEmptyDOMElement","toBeInTheDocument"],"sources":["../../../src/lib/v2/__tests__/rowActions.test.tsx"],"sourcesContent":["/**\r\n * @jest-environment jsdom\r\n */\r\nimport \"@testing-library/jest-dom\";\r\nimport { render, screen } from \"@testing-library/react\";\r\nimport userEvent from \"@testing-library/user-event\";\r\nimport {\r\n  buildRowActions,\r\n  buildRowActionMenu,\r\n  MAX_ROW_ACTIONS,\r\n} from \"../patterns/rowActions\";\r\nimport type { RowActionDescriptor } from \"../patterns/rowActions\";\r\n\r\n/**\r\n * rowActions (G-41) — promoted from `refonte-frontend` (Area 5).\r\n *\r\n * Each assertion pins a Charter rule the descriptor layer carries, or a defect\r\n * the promoted implementation had been written to prevent. The per-row\r\n * evaluation of the predicates is the load-bearing part: a boolean would force\r\n * the host to rebuild the array per row, which is how a table row and a mobile\r\n * card end up offering different actions for the same record.\r\n */\r\n\r\ninterface Dossier {\r\n  id: number;\r\n  status: string;\r\n}\r\n\r\nconst BROUILLON: Dossier = { id: 1, status: \"DRAFT\" };\r\nconst SOUMIS: Dossier = { id: 2, status: \"SOUMIS\" };\r\n\r\ndescribe(\"buildRowActionMenu\", () => {\r\n  test(\"hidden is evaluated PER ROW, not once for the array\", () => {\r\n    const descriptors: RowActionDescriptor<Dossier>[] = [\r\n      { id: \"open\", label: \"Consulter\", onClick: jest.fn() },\r\n      {\r\n        id: \"del\",\r\n        label: \"Supprimer\",\r\n        onClick: jest.fn(),\r\n        hidden: (row) => row.status !== \"DRAFT\",\r\n      },\r\n    ];\r\n\r\n    expect(buildRowActionMenu(descriptors, BROUILLON)).toHaveLength(2);\r\n    expect(buildRowActionMenu(descriptors, SOUMIS)).toHaveLength(1);\r\n  });\r\n\r\n  test(\"disabled is evaluated per row too, and keeps the item visible\", () => {\r\n    const items = buildRowActionMenu<Dossier>(\r\n      [\r\n        {\r\n          id: \"edit\",\r\n          label: \"Modifier\",\r\n          onClick: jest.fn(),\r\n          disabled: (row) => row.status !== \"DRAFT\",\r\n        },\r\n      ],\r\n      SOUMIS\r\n    );\r\n    expect(items).toHaveLength(1);\r\n    expect(items[0]?.disabled).toBe(true);\r\n  });\r\n\r\n  /** R-B.6 — a destructive action is separated and marked, not merely last. */\r\n  test(\"a divider precedes the first destructive action\", () => {\r\n    const items = buildRowActionMenu<Dossier>(\r\n      [\r\n        { id: \"open\", label: \"Consulter\", onClick: jest.fn() },\r\n        { id: \"del\", label: \"Supprimer\", onClick: jest.fn(), danger: true },\r\n        { id: \"purge\", label: \"Purger\", onClick: jest.fn(), danger: true },\r\n      ],\r\n      BROUILLON\r\n    );\r\n    expect(items.map((i) => Boolean(i.divider))).toEqual([false, true, false, false]);\r\n  });\r\n\r\n  /**\r\n   * A menu does not open on a rule. The separator only appears if something\r\n   * precedes it — a row whose ONLY action is destructive gets no divider.\r\n   */\r\n  test(\"no leading divider when the destructive action comes first\", () => {\r\n    const items = buildRowActionMenu<Dossier>(\r\n      [{ id: \"del\", label: \"Supprimer\", onClick: jest.fn(), danger: true }],\r\n      BROUILLON\r\n    );\r\n    expect(items).toHaveLength(1);\r\n    expect(items[0]?.divider).toBeUndefined();\r\n  });\r\n\r\n  test(\"nothing visible yields an empty list, so a caller can skip the trigger\", () => {\r\n    expect(\r\n      buildRowActionMenu<Dossier>(\r\n        [{ id: \"del\", label: \"Supprimer\", onClick: jest.fn(), hidden: () => true }],\r\n        BROUILLON\r\n      )\r\n    ).toEqual([]);\r\n  });\r\n\r\n  test(\"onClick is bound to the row it was built for\", () => {\r\n    const onClick = jest.fn();\r\n    const items = buildRowActionMenu<Dossier>(\r\n      [{ id: \"open\", label: \"Consulter\", onClick }],\r\n      SOUMIS\r\n    );\r\n    items[0]?.onClick?.();\r\n    expect(onClick).toHaveBeenCalledWith(SOUMIS);\r\n  });\r\n\r\n  /**\r\n   * R-3.3 caps what one screen may offer at once, and `OverflowMenu`'s own\r\n   * contract recommends ≤ 7 items. Exceeding it is a host decision, so it warns\r\n   * in development rather than truncating — silently dropping an action would\r\n   * hide a business capability.\r\n   */\r\n  test(`warns past ${MAX_ROW_ACTIONS} visible actions, without dropping any`, () => {\r\n    const warn = jest.spyOn(console, \"warn\").mockImplementation(() => undefined);\r\n    const descriptors: RowActionDescriptor<Dossier>[] = Array.from(\r\n      { length: MAX_ROW_ACTIONS + 1 },\r\n      (_, i) => ({ id: `a${i}`, label: `Action ${i}`, onClick: jest.fn() })\r\n    );\r\n\r\n    const items = buildRowActionMenu(descriptors, BROUILLON);\r\n\r\n    expect(items).toHaveLength(MAX_ROW_ACTIONS + 1);\r\n    expect(warn).toHaveBeenCalled();\r\n    warn.mockRestore();\r\n  });\r\n});\r\n\r\ndescribe(\"buildRowActions\", () => {\r\n  /**\r\n   * R-K.1 — the reason this layer exists. The implementation it replaced\r\n   * revealed its actions on hover, with a `mouseup` listener on `document` and\r\n   * no keyboard equivalent: consulting or deleting a record was unreachable\r\n   * without a mouse. `OverflowMenu` is an ARIA menu, so the trigger is a\r\n   * button and the items are reachable by keyboard.\r\n   */\r\n  test(\"renders a keyboard-reachable trigger, named by the caller\", async () => {\r\n    const onClick = jest.fn();\r\n    const render$ = buildRowActions<Dossier>(\r\n      [{ id: \"open\", label: \"Consulter le dossier\", onClick }],\r\n      \"Actions sur le dossier\"\r\n    );\r\n\r\n    render(<>{render$(SOUMIS)}</>);\r\n\r\n    const trigger = screen.getByRole(\"button\", { name: \"Actions sur le dossier\" });\r\n    await userEvent.click(trigger);\r\n    await userEvent.click(\r\n      screen.getByRole(\"menuitem\", { name: \"Consulter le dossier\" })\r\n    );\r\n\r\n    expect(onClick).toHaveBeenCalledWith(SOUMIS);\r\n  });\r\n\r\n  test(\"renders nothing when no action applies to the row\", () => {\r\n    const render$ = buildRowActions<Dossier>([\r\n      { id: \"del\", label: \"Supprimer\", onClick: jest.fn(), hidden: () => true },\r\n    ]);\r\n    const { container } = render(<>{render$(BROUILLON)}</>);\r\n    expect(container).toBeEmptyDOMElement();\r\n  });\r\n\r\n  test(\"the default trigger name is generic, so hosts localise it per resource\", () => {\r\n    const render$ = buildRowActions<Dossier>([\r\n      { id: \"open\", label: \"Consulter\", onClick: jest.fn() },\r\n    ]);\r\n    render(<>{render$(SOUMIS)}</>);\r\n    expect(screen.getByRole(\"button\", { name: \"Actions\" })).toBeInTheDocument();\r\n  });\r\n});\r\n"],"mappings":";;AAGAA,OAAA;AACA,IAAAC,MAAA,GAAAD,OAAA;AACA,IAAAE,UAAA,GAAAC,sBAAA,CAAAH,OAAA;AACA,IAAAI,WAAA,GAAAJ,OAAA;AAIgC,IAAAK,WAAA,GAAAL,OAAA;AAAA,SAAAG,uBAAAG,CAAA,WAAAA,CAAA,IAAAA,CAAA,CAAAC,UAAA,GAAAD,CAAA,KAAAE,OAAA,EAAAF,CAAA;AAVhC;AACA;AACA;;AAWA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;;AAOA,MAAMG,SAAkB,GAAG;EAAEC,EAAE,EAAE,CAAC;EAAEC,MAAM,EAAE;AAAQ,CAAC;AACrD,MAAMC,MAAe,GAAG;EAAEF,EAAE,EAAE,CAAC;EAAEC,MAAM,EAAE;AAAS,CAAC;AAEnDE,QAAQ,CAAC,oBAAoB,EAAE,MAAM;EACnCC,IAAI,CAAC,qDAAqD,EAAE,MAAM;IAChE,MAAMC,WAA2C,GAAG,CAClD;MAAEL,EAAE,EAAE,MAAM;MAAEM,KAAK,EAAE,WAAW;MAAEC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC;IAAE,CAAC,EACtD;MACET,EAAE,EAAE,KAAK;MACTM,KAAK,EAAE,WAAW;MAClBC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC,CAAC;MAClBC,MAAM,EAAGC,GAAG,IAAKA,GAAG,CAACV,MAAM,KAAK;IAClC,CAAC,CACF;IAEDW,MAAM,CAAC,IAAAC,8BAAkB,EAACR,WAAW,EAAEN,SAAS,CAAC,CAAC,CAACe,YAAY,CAAC,CAAC,CAAC;IAClEF,MAAM,CAAC,IAAAC,8BAAkB,EAACR,WAAW,EAAEH,MAAM,CAAC,CAAC,CAACY,YAAY,CAAC,CAAC,CAAC;EACjE,CAAC,CAAC;EAEFV,IAAI,CAAC,+DAA+D,EAAE,MAAM;IAC1E,MAAMW,KAAK,GAAG,IAAAF,8BAAkB,EAC9B,CACE;MACEb,EAAE,EAAE,MAAM;MACVM,KAAK,EAAE,UAAU;MACjBC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC,CAAC;MAClBO,QAAQ,EAAGL,GAAG,IAAKA,GAAG,CAACV,MAAM,KAAK;IACpC,CAAC,CACF,EACDC,MACF,CAAC;IACDU,MAAM,CAACG,KAAK,CAAC,CAACD,YAAY,CAAC,CAAC,CAAC;IAC7BF,MAAM,CAACG,KAAK,CAAC,CAAC,CAAC,EAAEC,QAAQ,CAAC,CAACC,IAAI,CAAC,IAAI,CAAC;EACvC,CAAC,CAAC;;EAEF;EACAb,IAAI,CAAC,iDAAiD,EAAE,MAAM;IAC5D,MAAMW,KAAK,GAAG,IAAAF,8BAAkB,EAC9B,CACE;MAAEb,EAAE,EAAE,MAAM;MAAEM,KAAK,EAAE,WAAW;MAAEC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC;IAAE,CAAC,EACtD;MAAET,EAAE,EAAE,KAAK;MAAEM,KAAK,EAAE,WAAW;MAAEC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC,CAAC;MAAES,MAAM,EAAE;IAAK,CAAC,EACnE;MAAElB,EAAE,EAAE,OAAO;MAAEM,KAAK,EAAE,QAAQ;MAAEC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC,CAAC;MAAES,MAAM,EAAE;IAAK,CAAC,CACnE,EACDnB,SACF,CAAC;IACDa,MAAM,CAACG,KAAK,CAACI,GAAG,CAAEC,CAAC,IAAKC,OAAO,CAACD,CAAC,CAACE,OAAO,CAAC,CAAC,CAAC,CAACC,OAAO,CAAC,CAAC,KAAK,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,CAAC,CAAC;EACnF,CAAC,CAAC;;EAEF;AACF;AACA;AACA;EACEnB,IAAI,CAAC,4DAA4D,EAAE,MAAM;IACvE,MAAMW,KAAK,GAAG,IAAAF,8BAAkB,EAC9B,CAAC;MAAEb,EAAE,EAAE,KAAK;MAAEM,KAAK,EAAE,WAAW;MAAEC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC,CAAC;MAAES,MAAM,EAAE;IAAK,CAAC,CAAC,EACrEnB,SACF,CAAC;IACDa,MAAM,CAACG,KAAK,CAAC,CAACD,YAAY,CAAC,CAAC,CAAC;IAC7BF,MAAM,CAACG,KAAK,CAAC,CAAC,CAAC,EAAEO,OAAO,CAAC,CAACE,aAAa,CAAC,CAAC;EAC3C,CAAC,CAAC;EAEFpB,IAAI,CAAC,wEAAwE,EAAE,MAAM;IACnFQ,MAAM,CACJ,IAAAC,8BAAkB,EAChB,CAAC;MAAEb,EAAE,EAAE,KAAK;MAAEM,KAAK,EAAE,WAAW;MAAEC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC,CAAC;MAAEC,MAAM,EAAEA,CAAA,KAAM;IAAK,CAAC,CAAC,EAC3EX,SACF,CACF,CAAC,CAACwB,OAAO,CAAC,EAAE,CAAC;EACf,CAAC,CAAC;EAEFnB,IAAI,CAAC,8CAA8C,EAAE,MAAM;IACzD,MAAMG,OAAO,GAAGC,IAAI,CAACC,EAAE,CAAC,CAAC;IACzB,MAAMM,KAAK,GAAG,IAAAF,8BAAkB,EAC9B,CAAC;MAAEb,EAAE,EAAE,MAAM;MAAEM,KAAK,EAAE,WAAW;MAAEC;IAAQ,CAAC,CAAC,EAC7CL,MACF,CAAC;IACDa,KAAK,CAAC,CAAC,CAAC,EAAER,OAAO,GAAG,CAAC;IACrBK,MAAM,CAACL,OAAO,CAAC,CAACkB,oBAAoB,CAACvB,MAAM,CAAC;EAC9C,CAAC,CAAC;;EAEF;AACF;AACA;AACA;AACA;AACA;EACEE,IAAI,CAAC,cAAcsB,2BAAe,wCAAwC,EAAE,MAAM;IAChF,MAAMC,IAAI,GAAGnB,IAAI,CAACoB,KAAK,CAACC,OAAO,EAAE,MAAM,CAAC,CAACC,kBAAkB,CAAC,MAAMC,SAAS,CAAC;IAC5E,MAAM1B,WAA2C,GAAG2B,KAAK,CAACC,IAAI,CAC5D;MAAEC,MAAM,EAAER,2BAAe,GAAG;IAAE,CAAC,EAC/B,CAACS,CAAC,EAAEf,CAAC,MAAM;MAAEpB,EAAE,EAAE,IAAIoB,CAAC,EAAE;MAAEd,KAAK,EAAE,UAAUc,CAAC,EAAE;MAAEb,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC;IAAE,CAAC,CACtE,CAAC;IAED,MAAMM,KAAK,GAAG,IAAAF,8BAAkB,EAACR,WAAW,EAAEN,SAAS,CAAC;IAExDa,MAAM,CAACG,KAAK,CAAC,CAACD,YAAY,CAACY,2BAAe,GAAG,CAAC,CAAC;IAC/Cd,MAAM,CAACe,IAAI,CAAC,CAACS,gBAAgB,CAAC,CAAC;IAC/BT,IAAI,CAACU,WAAW,CAAC,CAAC;EACpB,CAAC,CAAC;AACJ,CAAC,CAAC;AAEFlC,QAAQ,CAAC,iBAAiB,EAAE,MAAM;EAChC;AACF;AACA;AACA;AACA;AACA;AACA;EACEC,IAAI,CAAC,2DAA2D,EAAE,YAAY;IAC5E,MAAMG,OAAO,GAAGC,IAAI,CAACC,EAAE,CAAC,CAAC;IACzB,MAAM6B,OAAO,GAAG,IAAAC,2BAAe,EAC7B,CAAC;MAAEvC,EAAE,EAAE,MAAM;MAAEM,KAAK,EAAE,sBAAsB;MAAEC;IAAQ,CAAC,CAAC,EACxD,wBACF,CAAC;IAED,IAAAiC,aAAM,eAAC,IAAA7C,WAAA,CAAA8C,GAAA,EAAA9C,WAAA,CAAA+C,QAAA;MAAAC,QAAA,EAAGL,OAAO,CAACpC,MAAM;IAAC,CAAG,CAAC,CAAC;IAE9B,MAAM0C,OAAO,GAAGC,aAAM,CAACC,SAAS,CAAC,QAAQ,EAAE;MAAEC,IAAI,EAAE;IAAyB,CAAC,CAAC;IAC9E,MAAMC,kBAAS,CAACC,KAAK,CAACL,OAAO,CAAC;IAC9B,MAAMI,kBAAS,CAACC,KAAK,CACnBJ,aAAM,CAACC,SAAS,CAAC,UAAU,EAAE;MAAEC,IAAI,EAAE;IAAuB,CAAC,CAC/D,CAAC;IAEDnC,MAAM,CAACL,OAAO,CAAC,CAACkB,oBAAoB,CAACvB,MAAM,CAAC;EAC9C,CAAC,CAAC;EAEFE,IAAI,CAAC,mDAAmD,EAAE,MAAM;IAC9D,MAAMkC,OAAO,GAAG,IAAAC,2BAAe,EAAU,CACvC;MAAEvC,EAAE,EAAE,KAAK;MAAEM,KAAK,EAAE,WAAW;MAAEC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC,CAAC;MAAEC,MAAM,EAAEA,CAAA,KAAM;IAAK,CAAC,CAC1E,CAAC;IACF,MAAM;MAAEwC;IAAU,CAAC,GAAG,IAAAV,aAAM,eAAC,IAAA7C,WAAA,CAAA8C,GAAA,EAAA9C,WAAA,CAAA+C,QAAA;MAAAC,QAAA,EAAGL,OAAO,CAACvC,SAAS;IAAC,CAAG,CAAC,CAAC;IACvDa,MAAM,CAACsC,SAAS,CAAC,CAACC,mBAAmB,CAAC,CAAC;EACzC,CAAC,CAAC;EAEF/C,IAAI,CAAC,wEAAwE,EAAE,MAAM;IACnF,MAAMkC,OAAO,GAAG,IAAAC,2BAAe,EAAU,CACvC;MAAEvC,EAAE,EAAE,MAAM;MAAEM,KAAK,EAAE,WAAW;MAAEC,OAAO,EAAEC,IAAI,CAACC,EAAE,CAAC;IAAE,CAAC,CACvD,CAAC;IACF,IAAA+B,aAAM,eAAC,IAAA7C,WAAA,CAAA8C,GAAA,EAAA9C,WAAA,CAAA+C,QAAA;MAAAC,QAAA,EAAGL,OAAO,CAACpC,MAAM;IAAC,CAAG,CAAC,CAAC;IAC9BU,MAAM,CAACiC,aAAM,CAACC,SAAS,CAAC,QAAQ,EAAE;MAAEC,IAAI,EAAE;IAAU,CAAC,CAAC,CAAC,CAACK,iBAAiB,CAAC,CAAC;EAC7E,CAAC,CAAC;AACJ,CAAC,CAAC","ignoreList":[]}

package/dist/v2/__tests__/useFocusOnRouteChange.test.js

@@ -0,0 +1,156 @@
+"use strict";
+
+require("@testing-library/jest-dom");
+var _react = require("@testing-library/react");
+var _AppMain = _interopRequireDefault(require("../patterns/AppMain"));
+var _useFocusOnRouteChange = _interopRequireDefault(require("../hooks/useFocusOnRouteChange"));
+var _jsxRuntime = require("react/jsx-runtime");
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+/**
+ * G-60 — focus on page change.
+ *
+ * What this guards is invisible by construction: after a route change in a
+ * single-page application the pixels are perfect and the focus is on
+ * `document.body`, so the next Tab starts at the top of the BROWSER rather than
+ * the page. Only an assertion catches it.
+ *
+ * The harness mirrors the real arrangement — the hook is called by a component
+ * INSIDE the landmark it targets, because that is where a router puts it — and
+ * drives the key by prop rather than importing a router: the hook is
+ * router-agnostic and its tests should be too.
+ */function Page({
+  routeKey
+}) {
+  (0, _useFocusOnRouteChange.default)(routeKey);
+  return /*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
+    type: "button",
+    children: "Action de la page"
+  });
+}
+function Harness({
+  routeKey
+}) {
+  return /*#__PURE__*/(0, _jsxRuntime.jsx)(_AppMain.default, {
+    children: /*#__PURE__*/(0, _jsxRuntime.jsx)(Page, {
+      routeKey: routeKey
+    })
+  });
+}
+describe("useFocusOnRouteChange", () => {
+  /* The browser already places focus at the start of the document on load.
+   * Moving it then would fight the browser and scroll a deep-linked page away
+   * from its anchor. */
+  it("leaves focus alone on the first render", () => {
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(Harness, {
+      routeKey: "/demandes"
+    }));
+    expect(_react.screen.getByRole("main")).not.toHaveFocus();
+  });
+  it("moves focus to the landmark when the key changes", () => {
+    const {
+      rerender
+    } = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(Harness, {
+      routeKey: "/demandes"
+    }));
+
+    /* Focus somewhere inside the old page, as a user would have it after
+       clicking the action that navigates. */
+    _react.screen.getByRole("button").focus();
+    expect(_react.screen.getByRole("button")).toHaveFocus();
+    rerender(/*#__PURE__*/(0, _jsxRuntime.jsx)(Harness, {
+      routeKey: "/demandes/42"
+    }));
+    expect(_react.screen.getByRole("main")).toHaveFocus();
+  });
+
+  /* A re-render with the SAME key must not steal focus — a listing re-renders
+   * on every keystroke of its search field, and the host is told to watch the
+   * pathname precisely so that state changes are not page changes. */
+  it("does not move focus when the key is unchanged", () => {
+    const {
+      rerender
+    } = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(Harness, {
+      routeKey: "/demandes"
+    }));
+    _react.screen.getByRole("button").focus();
+    rerender(/*#__PURE__*/(0, _jsxRuntime.jsx)(Harness, {
+      routeKey: "/demandes"
+    }));
+    expect(_react.screen.getByRole("button")).toHaveFocus();
+  });
+  it("honours a custom target id", () => {
+    function CustomPage({
+      routeKey
+    }) {
+      (0, _useFocusOnRouteChange.default)(routeKey, {
+        targetId: "zone-principale"
+      });
+      return null;
+    }
+    const {
+      rerender
+    } = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_AppMain.default, {
+      id: "zone-principale",
+      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(CustomPage, {
+        routeKey: "/a"
+      })
+    }));
+    rerender(/*#__PURE__*/(0, _jsxRuntime.jsx)(_AppMain.default, {
+      id: "zone-principale",
+      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(CustomPage, {
+        routeKey: "/b"
+      })
+    }));
+    expect(_react.screen.getByRole("main")).toHaveFocus();
+  });
+  it("can be suspended", () => {
+    function DisabledPage({
+      routeKey
+    }) {
+      (0, _useFocusOnRouteChange.default)(routeKey, {
+        enabled: false
+      });
+      return /*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
+        type: "button",
+        children: "Action"
+      });
+    }
+    const {
+      rerender
+    } = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_AppMain.default, {
+      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(DisabledPage, {
+        routeKey: "/a"
+      })
+    }));
+    _react.screen.getByRole("button").focus();
+    rerender(/*#__PURE__*/(0, _jsxRuntime.jsx)(_AppMain.default, {
+      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(DisabledPage, {
+        routeKey: "/b"
+      })
+    }));
+    expect(_react.screen.getByRole("button")).toHaveFocus();
+  });
+
+  /* A host may render the hook on a shell that mounts before its landmark —
+   * a pre-auth screen, an error boundary. Throwing there would replace a
+   * missing focus move with a crash. */
+  it("does nothing when the target is absent", () => {
+    function Orphan({
+      routeKey
+    }) {
+      (0, _useFocusOnRouteChange.default)(routeKey);
+      return /*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
+        children: "sans landmark"
+      });
+    }
+    const {
+      rerender
+    } = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(Orphan, {
+      routeKey: "/a"
+    }));
+    expect(() => rerender(/*#__PURE__*/(0, _jsxRuntime.jsx)(Orphan, {
+      routeKey: "/b"
+    }))).not.toThrow();
+  });
+});
+//# sourceMappingURL=useFocusOnRouteChange.test.js.map

package/dist/v2/__tests__/useFocusOnRouteChange.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"useFocusOnRouteChange.test.js","names":["require","_react","_AppMain","_interopRequireDefault","_useFocusOnRouteChange","_jsxRuntime","e","__esModule","default","Page","routeKey","useFocusOnRouteChange","jsx","type","children","Harness","describe","it","render","expect","screen","getByRole","not","toHaveFocus","rerender","focus","CustomPage","targetId","id","DisabledPage","enabled","Orphan","toThrow"],"sources":["../../../src/lib/v2/__tests__/useFocusOnRouteChange.test.tsx"],"sourcesContent":["import \"@testing-library/jest-dom\";\r\nimport { render, screen } from \"@testing-library/react\";\r\n\r\nimport AppMain from \"../patterns/AppMain\";\r\nimport useFocusOnRouteChange from \"../hooks/useFocusOnRouteChange\";\r\n\r\n/**\r\n * G-60 — focus on page change.\r\n *\r\n * What this guards is invisible by construction: after a route change in a\r\n * single-page application the pixels are perfect and the focus is on\r\n * `document.body`, so the next Tab starts at the top of the BROWSER rather than\r\n * the page. Only an assertion catches it.\r\n *\r\n * The harness mirrors the real arrangement — the hook is called by a component\r\n * INSIDE the landmark it targets, because that is where a router puts it — and\r\n * drives the key by prop rather than importing a router: the hook is\r\n * router-agnostic and its tests should be too.\r\n */\r\n\r\nfunction Page({ routeKey }: { routeKey: string }) {\r\n  useFocusOnRouteChange(routeKey);\r\n  return <button type=\"button\">Action de la page</button>;\r\n}\r\n\r\nfunction Harness({ routeKey }: { routeKey: string }) {\r\n  return (\r\n    <AppMain>\r\n      <Page routeKey={routeKey} />\r\n    </AppMain>\r\n  );\r\n}\r\n\r\ndescribe(\"useFocusOnRouteChange\", () => {\r\n  /* The browser already places focus at the start of the document on load.\r\n   * Moving it then would fight the browser and scroll a deep-linked page away\r\n   * from its anchor. */\r\n  it(\"leaves focus alone on the first render\", () => {\r\n    render(<Harness routeKey=\"/demandes\" />);\r\n    expect(screen.getByRole(\"main\")).not.toHaveFocus();\r\n  });\r\n\r\n  it(\"moves focus to the landmark when the key changes\", () => {\r\n    const { rerender } = render(<Harness routeKey=\"/demandes\" />);\r\n\r\n    /* Focus somewhere inside the old page, as a user would have it after\r\n       clicking the action that navigates. */\r\n    screen.getByRole(\"button\").focus();\r\n    expect(screen.getByRole(\"button\")).toHaveFocus();\r\n\r\n    rerender(<Harness routeKey=\"/demandes/42\" />);\r\n\r\n    expect(screen.getByRole(\"main\")).toHaveFocus();\r\n  });\r\n\r\n  /* A re-render with the SAME key must not steal focus — a listing re-renders\r\n   * on every keystroke of its search field, and the host is told to watch the\r\n   * pathname precisely so that state changes are not page changes. */\r\n  it(\"does not move focus when the key is unchanged\", () => {\r\n    const { rerender } = render(<Harness routeKey=\"/demandes\" />);\r\n\r\n    screen.getByRole(\"button\").focus();\r\n    rerender(<Harness routeKey=\"/demandes\" />);\r\n\r\n    expect(screen.getByRole(\"button\")).toHaveFocus();\r\n  });\r\n\r\n  it(\"honours a custom target id\", () => {\r\n    function CustomPage({ routeKey }: { routeKey: string }) {\r\n      useFocusOnRouteChange(routeKey, { targetId: \"zone-principale\" });\r\n      return null;\r\n    }\r\n\r\n    const { rerender } = render(\r\n      <AppMain id=\"zone-principale\">\r\n        <CustomPage routeKey=\"/a\" />\r\n      </AppMain>\r\n    );\r\n    rerender(\r\n      <AppMain id=\"zone-principale\">\r\n        <CustomPage routeKey=\"/b\" />\r\n      </AppMain>\r\n    );\r\n\r\n    expect(screen.getByRole(\"main\")).toHaveFocus();\r\n  });\r\n\r\n  it(\"can be suspended\", () => {\r\n    function DisabledPage({ routeKey }: { routeKey: string }) {\r\n      useFocusOnRouteChange(routeKey, { enabled: false });\r\n      return <button type=\"button\">Action</button>;\r\n    }\r\n\r\n    const { rerender } = render(\r\n      <AppMain>\r\n        <DisabledPage routeKey=\"/a\" />\r\n      </AppMain>\r\n    );\r\n    screen.getByRole(\"button\").focus();\r\n    rerender(\r\n      <AppMain>\r\n        <DisabledPage routeKey=\"/b\" />\r\n      </AppMain>\r\n    );\r\n\r\n    expect(screen.getByRole(\"button\")).toHaveFocus();\r\n  });\r\n\r\n  /* A host may render the hook on a shell that mounts before its landmark —\r\n   * a pre-auth screen, an error boundary. Throwing there would replace a\r\n   * missing focus move with a crash. */\r\n  it(\"does nothing when the target is absent\", () => {\r\n    function Orphan({ routeKey }: { routeKey: string }) {\r\n      useFocusOnRouteChange(routeKey);\r\n      return <span>sans landmark</span>;\r\n    }\r\n\r\n    const { rerender } = render(<Orphan routeKey=\"/a\" />);\r\n    expect(() => rerender(<Orphan routeKey=\"/b\" />)).not.toThrow();\r\n  });\r\n});\r\n"],"mappings":";;AAAAA,OAAA;AACA,IAAAC,MAAA,GAAAD,OAAA;AAEA,IAAAE,QAAA,GAAAC,sBAAA,CAAAH,OAAA;AACA,IAAAI,sBAAA,GAAAD,sBAAA,CAAAH,OAAA;AAAmE,IAAAK,WAAA,GAAAL,OAAA;AAAA,SAAAG,uBAAAG,CAAA,WAAAA,CAAA,IAAAA,CAAA,CAAAC,UAAA,GAAAD,CAAA,KAAAE,OAAA,EAAAF,CAAA;AAEnE;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA,GAEA,SAASG,IAAIA,CAAC;EAAEC;AAA+B,CAAC,EAAE;EAChD,IAAAC,8BAAqB,EAACD,QAAQ,CAAC;EAC/B,oBAAO,IAAAL,WAAA,CAAAO,GAAA;IAAQC,IAAI,EAAC,QAAQ;IAAAC,QAAA,EAAC;EAAiB,CAAQ,CAAC;AACzD;AAEA,SAASC,OAAOA,CAAC;EAAEL;AAA+B,CAAC,EAAE;EACnD,oBACE,IAAAL,WAAA,CAAAO,GAAA,EAACV,QAAA,CAAAM,OAAO;IAAAM,QAAA,eACN,IAAAT,WAAA,CAAAO,GAAA,EAACH,IAAI;MAACC,QAAQ,EAAEA;IAAS,CAAE;EAAC,CACrB,CAAC;AAEd;AAEAM,QAAQ,CAAC,uBAAuB,EAAE,MAAM;EACtC;AACF;AACA;EACEC,EAAE,CAAC,wCAAwC,EAAE,MAAM;IACjD,IAAAC,aAAM,eAAC,IAAAb,WAAA,CAAAO,GAAA,EAACG,OAAO;MAACL,QAAQ,EAAC;IAAW,CAAE,CAAC,CAAC;IACxCS,MAAM,CAACC,aAAM,CAACC,SAAS,CAAC,MAAM,CAAC,CAAC,CAACC,GAAG,CAACC,WAAW,CAAC,CAAC;EACpD,CAAC,CAAC;EAEFN,EAAE,CAAC,kDAAkD,EAAE,MAAM;IAC3D,MAAM;MAAEO;IAAS,CAAC,GAAG,IAAAN,aAAM,eAAC,IAAAb,WAAA,CAAAO,GAAA,EAACG,OAAO;MAACL,QAAQ,EAAC;IAAW,CAAE,CAAC,CAAC;;IAE7D;AACJ;IACIU,aAAM,CAACC,SAAS,CAAC,QAAQ,CAAC,CAACI,KAAK,CAAC,CAAC;IAClCN,MAAM,CAACC,aAAM,CAACC,SAAS,CAAC,QAAQ,CAAC,CAAC,CAACE,WAAW,CAAC,CAAC;IAEhDC,QAAQ,cAAC,IAAAnB,WAAA,CAAAO,GAAA,EAACG,OAAO;MAACL,QAAQ,EAAC;IAAc,CAAE,CAAC,CAAC;IAE7CS,MAAM,CAACC,aAAM,CAACC,SAAS,CAAC,MAAM,CAAC,CAAC,CAACE,WAAW,CAAC,CAAC;EAChD,CAAC,CAAC;;EAEF;AACF;AACA;EACEN,EAAE,CAAC,+CAA+C,EAAE,MAAM;IACxD,MAAM;MAAEO;IAAS,CAAC,GAAG,IAAAN,aAAM,eAAC,IAAAb,WAAA,CAAAO,GAAA,EAACG,OAAO;MAACL,QAAQ,EAAC;IAAW,CAAE,CAAC,CAAC;IAE7DU,aAAM,CAACC,SAAS,CAAC,QAAQ,CAAC,CAACI,KAAK,CAAC,CAAC;IAClCD,QAAQ,cAAC,IAAAnB,WAAA,CAAAO,GAAA,EAACG,OAAO;MAACL,QAAQ,EAAC;IAAW,CAAE,CAAC,CAAC;IAE1CS,MAAM,CAACC,aAAM,CAACC,SAAS,CAAC,QAAQ,CAAC,CAAC,CAACE,WAAW,CAAC,CAAC;EAClD,CAAC,CAAC;EAEFN,EAAE,CAAC,4BAA4B,EAAE,MAAM;IACrC,SAASS,UAAUA,CAAC;MAAEhB;IAA+B,CAAC,EAAE;MACtD,IAAAC,8BAAqB,EAACD,QAAQ,EAAE;QAAEiB,QAAQ,EAAE;MAAkB,CAAC,CAAC;MAChE,OAAO,IAAI;IACb;IAEA,MAAM;MAAEH;IAAS,CAAC,GAAG,IAAAN,aAAM,eACzB,IAAAb,WAAA,CAAAO,GAAA,EAACV,QAAA,CAAAM,OAAO;MAACoB,EAAE,EAAC,iBAAiB;MAAAd,QAAA,eAC3B,IAAAT,WAAA,CAAAO,GAAA,EAACc,UAAU;QAAChB,QAAQ,EAAC;MAAI,CAAE;IAAC,CACrB,CACX,CAAC;IACDc,QAAQ,cACN,IAAAnB,WAAA,CAAAO,GAAA,EAACV,QAAA,CAAAM,OAAO;MAACoB,EAAE,EAAC,iBAAiB;MAAAd,QAAA,eAC3B,IAAAT,WAAA,CAAAO,GAAA,EAACc,UAAU;QAAChB,QAAQ,EAAC;MAAI,CAAE;IAAC,CACrB,CACX,CAAC;IAEDS,MAAM,CAACC,aAAM,CAACC,SAAS,CAAC,MAAM,CAAC,CAAC,CAACE,WAAW,CAAC,CAAC;EAChD,CAAC,CAAC;EAEFN,EAAE,CAAC,kBAAkB,EAAE,MAAM;IAC3B,SAASY,YAAYA,CAAC;MAAEnB;IAA+B,CAAC,EAAE;MACxD,IAAAC,8BAAqB,EAACD,QAAQ,EAAE;QAAEoB,OAAO,EAAE;MAAM,CAAC,CAAC;MACnD,oBAAO,IAAAzB,WAAA,CAAAO,GAAA;QAAQC,IAAI,EAAC,QAAQ;QAAAC,QAAA,EAAC;MAAM,CAAQ,CAAC;IAC9C;IAEA,MAAM;MAAEU;IAAS,CAAC,GAAG,IAAAN,aAAM,eACzB,IAAAb,WAAA,CAAAO,GAAA,EAACV,QAAA,CAAAM,OAAO;MAAAM,QAAA,eACN,IAAAT,WAAA,CAAAO,GAAA,EAACiB,YAAY;QAACnB,QAAQ,EAAC;MAAI,CAAE;IAAC,CACvB,CACX,CAAC;IACDU,aAAM,CAACC,SAAS,CAAC,QAAQ,CAAC,CAACI,KAAK,CAAC,CAAC;IAClCD,QAAQ,cACN,IAAAnB,WAAA,CAAAO,GAAA,EAACV,QAAA,CAAAM,OAAO;MAAAM,QAAA,eACN,IAAAT,WAAA,CAAAO,GAAA,EAACiB,YAAY;QAACnB,QAAQ,EAAC;MAAI,CAAE;IAAC,CACvB,CACX,CAAC;IAEDS,MAAM,CAACC,aAAM,CAACC,SAAS,CAAC,QAAQ,CAAC,CAAC,CAACE,WAAW,CAAC,CAAC;EAClD,CAAC,CAAC;;EAEF;AACF;AACA;EACEN,EAAE,CAAC,wCAAwC,EAAE,MAAM;IACjD,SAASc,MAAMA,CAAC;MAAErB;IAA+B,CAAC,EAAE;MAClD,IAAAC,8BAAqB,EAACD,QAAQ,CAAC;MAC/B,oBAAO,IAAAL,WAAA,CAAAO,GAAA;QAAAE,QAAA,EAAM;MAAa,CAAM,CAAC;IACnC;IAEA,MAAM;MAAEU;IAAS,CAAC,GAAG,IAAAN,aAAM,eAAC,IAAAb,WAAA,CAAAO,GAAA,EAACmB,MAAM;MAACrB,QAAQ,EAAC;IAAI,CAAE,CAAC,CAAC;IACrDS,MAAM,CAAC,MAAMK,QAAQ,cAAC,IAAAnB,WAAA,CAAAO,GAAA,EAACmB,MAAM;MAACrB,QAAQ,EAAC;IAAI,CAAE,CAAC,CAAC,CAAC,CAACY,GAAG,CAACU,OAAO,CAAC,CAAC;EAChE,CAAC,CAAC;AACJ,CAAC,CAAC","ignoreList":[]}