wip

odoo · Apr 11, 2024 · 0fbcc09 · 0fbcc09
1 parent 96f8c40
commit 0fbcc09
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/tests/pivots/pivot_menu_items.test.ts b/tests/pivots/pivot_menu_items.test.ts
@@ -1,4 +1,5 @@
 import { Model, SpreadsheetChildEnv } from "../../src";
+import { toZone } from "../../src/helpers";
 import { cellMenuRegistry } from "../../src/registries";
 import { selectCell, setCellContent } from "../test_helpers/commands_helpers";
 import { makeTestEnv } from "../test_helpers/helpers";
@@ -20,32 +21,32 @@ describe("Pivot menu items", () => {
 
   it("should display pivot_properties if there is a pivot in the cell", () => {
     selectCell(model, "A1");
-    addPivot(model, { range: "M1:N1" }, "1");
+    addPivot(model, { zone: toZone("M1:N1") }, "1");
     setCellContent(model, "A1", `=PIVOT("1")`);
     expect(model.getters.getPivotIdFromPosition(model.getters.getActivePosition())).toBe("1");
     expect(cellMenuRegistry.get("pivot_properties").isVisible!(env)).toBe(true);
   });
 
   it("should not display pivot_properties if the pivot does not exist", () => {
     selectCell(model, "A1");
-    addPivot(model, { range: "M1:N1" }, "1");
+    addPivot(model, { zone: toZone("M1:N1") }, "1");
     setCellContent(model, "A1", `=PIVOT("2")`);
     expect(model.getters.getPivotIdFromPosition(model.getters.getActivePosition())).toBeUndefined();
     expect(cellMenuRegistry.get("pivot_properties").isVisible!(env)).toBe(false);
   });
 
   it("should display pivot_properties if there are multiple pivots in the cell", () => {
     selectCell(model, "A1");
-    addPivot(model, { range: "M1:N1" }, "1");
-    addPivot(model, { range: "M1:N1" }, "2");
+    addPivot(model, { zone: toZone("M1:N1") }, "1");
+    addPivot(model, { zone: toZone("M1:N1") }, "2");
     setCellContent(model, "A1", `=PIVOT("1") + PIVOT("2")`);
     expect(model.getters.getPivotIdFromPosition(model.getters.getActivePosition())).toBe("1");
     expect(cellMenuRegistry.get("pivot_properties").isVisible!(env)).toBe(true);
   });
 
   it("should open the pivot side panel when clicking on pivot_properties", () => {
     selectCell(model, "A1");
-    addPivot(model, { range: "M1:N1" }, "1");
+    addPivot(model, { zone: toZone("M1:N1") }, "1");
     setCellContent(model, "A1", `=PIVOT("1")`);
     const openSidePanel = jest.spyOn(env, "openSidePanel");
     cellMenuRegistry.get("pivot_properties").execute!(env);
@@ -54,8 +55,8 @@ describe("Pivot menu items", () => {
 
   it("should open the pivot side panel when clicking on pivot_properties with the first pivot id", () => {
     selectCell(model, "A1");
-    addPivot(model, { range: "M1:N1" }, "1");
-    addPivot(model, { range: "M1:N1" }, "2");
+    addPivot(model, { zone: toZone("M1:N1") }, "1");
+    addPivot(model, { zone: toZone("M1:N1") }, "2");
     setCellContent(model, "A1", `=PIVOT("1") + PIVOT("2")`);
     const openSidePanel = jest.spyOn(env, "openSidePanel");
     cellMenuRegistry.get("pivot_properties").execute!(env);