Browse Source

update tests

Zsolt Viczian 2 years ago
parent
commit
e68e11ff9f
3 changed files with 9 additions and 9 deletions
  1. 5 5
      src/tests/dragCreate.test.tsx
  2. 2 2
      src/tests/move.test.tsx
  3. 2 2
      src/tests/multiPointCreate.test.tsx

+ 5 - 5
src/tests/dragCreate.test.tsx

@@ -42,7 +42,7 @@ describe("Test dragCreate", () => {
       // finish (position does not matter)
       // finish (position does not matter)
       fireEvent.pointerUp(canvas);
       fireEvent.pointerUp(canvas);
 
 
-      expect(renderScene).toHaveBeenCalledTimes(8);
+      expect(renderScene).toHaveBeenCalledTimes(7);
       expect(h.state.selectionElement).toBeNull();
       expect(h.state.selectionElement).toBeNull();
 
 
       expect(h.elements.length).toEqual(1);
       expect(h.elements.length).toEqual(1);
@@ -73,7 +73,7 @@ describe("Test dragCreate", () => {
       // finish (position does not matter)
       // finish (position does not matter)
       fireEvent.pointerUp(canvas);
       fireEvent.pointerUp(canvas);
 
 
-      expect(renderScene).toHaveBeenCalledTimes(8);
+      expect(renderScene).toHaveBeenCalledTimes(7);
       expect(h.state.selectionElement).toBeNull();
       expect(h.state.selectionElement).toBeNull();
 
 
       expect(h.elements.length).toEqual(1);
       expect(h.elements.length).toEqual(1);
@@ -104,7 +104,7 @@ describe("Test dragCreate", () => {
       // finish (position does not matter)
       // finish (position does not matter)
       fireEvent.pointerUp(canvas);
       fireEvent.pointerUp(canvas);
 
 
-      expect(renderScene).toHaveBeenCalledTimes(8);
+      expect(renderScene).toHaveBeenCalledTimes(7);
       expect(h.state.selectionElement).toBeNull();
       expect(h.state.selectionElement).toBeNull();
 
 
       expect(h.elements.length).toEqual(1);
       expect(h.elements.length).toEqual(1);
@@ -135,7 +135,7 @@ describe("Test dragCreate", () => {
       // finish (position does not matter)
       // finish (position does not matter)
       fireEvent.pointerUp(canvas);
       fireEvent.pointerUp(canvas);
 
 
-      expect(renderScene).toHaveBeenCalledTimes(8);
+      expect(renderScene).toHaveBeenCalledTimes(7);
       expect(h.state.selectionElement).toBeNull();
       expect(h.state.selectionElement).toBeNull();
 
 
       expect(h.elements.length).toEqual(1);
       expect(h.elements.length).toEqual(1);
@@ -170,7 +170,7 @@ describe("Test dragCreate", () => {
       // finish (position does not matter)
       // finish (position does not matter)
       fireEvent.pointerUp(canvas);
       fireEvent.pointerUp(canvas);
 
 
-      expect(renderScene).toHaveBeenCalledTimes(8);
+      expect(renderScene).toHaveBeenCalledTimes(7);
       expect(h.state.selectionElement).toBeNull();
       expect(h.state.selectionElement).toBeNull();
 
 
       expect(h.elements.length).toEqual(1);
       expect(h.elements.length).toEqual(1);

+ 2 - 2
src/tests/move.test.tsx

@@ -38,7 +38,7 @@ describe("move element", () => {
       fireEvent.pointerMove(canvas, { clientX: 60, clientY: 70 });
       fireEvent.pointerMove(canvas, { clientX: 60, clientY: 70 });
       fireEvent.pointerUp(canvas);
       fireEvent.pointerUp(canvas);
 
 
-      expect(renderScene).toHaveBeenCalledTimes(8);
+      expect(renderScene).toHaveBeenCalledTimes(7);
       expect(h.state.selectionElement).toBeNull();
       expect(h.state.selectionElement).toBeNull();
       expect(h.elements.length).toEqual(1);
       expect(h.elements.length).toEqual(1);
       expect(h.state.selectedElementIds[h.elements[0].id]).toBeTruthy();
       expect(h.state.selectedElementIds[h.elements[0].id]).toBeTruthy();
@@ -120,7 +120,7 @@ describe("duplicate element on move when ALT is clicked", () => {
       fireEvent.pointerMove(canvas, { clientX: 60, clientY: 70 });
       fireEvent.pointerMove(canvas, { clientX: 60, clientY: 70 });
       fireEvent.pointerUp(canvas);
       fireEvent.pointerUp(canvas);
 
 
-      expect(renderScene).toHaveBeenCalledTimes(8);
+      expect(renderScene).toHaveBeenCalledTimes(7);
       expect(h.state.selectionElement).toBeNull();
       expect(h.state.selectionElement).toBeNull();
       expect(h.elements.length).toEqual(1);
       expect(h.elements.length).toEqual(1);
       expect(h.state.selectedElementIds[h.elements[0].id]).toBeTruthy();
       expect(h.state.selectedElementIds[h.elements[0].id]).toBeTruthy();

+ 2 - 2
src/tests/multiPointCreate.test.tsx

@@ -102,7 +102,7 @@ describe("multi point mode in linear elements", () => {
       key: KEYS.ENTER,
       key: KEYS.ENTER,
     });
     });
 
 
-    expect(renderScene).toHaveBeenCalledTimes(14);
+    expect(renderScene).toHaveBeenCalledTimes(13);
     expect(h.elements.length).toEqual(1);
     expect(h.elements.length).toEqual(1);
 
 
     const element = h.elements[0] as ExcalidrawLinearElement;
     const element = h.elements[0] as ExcalidrawLinearElement;
@@ -145,7 +145,7 @@ describe("multi point mode in linear elements", () => {
       key: KEYS.ENTER,
       key: KEYS.ENTER,
     });
     });
 
 
-    expect(renderScene).toHaveBeenCalledTimes(14);
+    expect(renderScene).toHaveBeenCalledTimes(13);
     expect(h.elements.length).toEqual(1);
     expect(h.elements.length).toEqual(1);
 
 
     const element = h.elements[0] as ExcalidrawLinearElement;
     const element = h.elements[0] as ExcalidrawLinearElement;