123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252 |
- import { CaptureUpdateAction, newElementWith } from "@excalidraw/excalidraw";
- import {
- createRedoAction,
- createUndoAction,
- } from "@excalidraw/excalidraw/actions/actionHistory";
- import { syncInvalidIndices } from "@excalidraw/element";
- import { API } from "@excalidraw/excalidraw/tests/helpers/api";
- import { act, render, waitFor } from "@excalidraw/excalidraw/tests/test-utils";
- import { vi } from "vitest";
- import { StoreIncrement } from "@excalidraw/element";
- import type { DurableIncrement, EphemeralIncrement } from "@excalidraw/element";
- import ExcalidrawApp from "../App";
- const { h } = window;
- Object.defineProperty(window, "crypto", {
- value: {
- getRandomValues: (arr: number[]) =>
- arr.forEach((v, i) => (arr[i] = Math.floor(Math.random() * 256))),
- subtle: {
- generateKey: () => {},
- exportKey: () => ({ k: "sTdLvMC_M3V8_vGa3UVRDg" }),
- },
- },
- });
- vi.mock("../../excalidraw-app/data/firebase.ts", () => {
- const loadFromFirebase = async () => null;
- const saveToFirebase = () => {};
- const isSavedToFirebase = () => true;
- const loadFilesFromFirebase = async () => ({
- loadedFiles: [],
- erroredFiles: [],
- });
- const saveFilesToFirebase = async () => ({
- savedFiles: new Map(),
- erroredFiles: new Map(),
- });
- return {
- loadFromFirebase,
- saveToFirebase,
- isSavedToFirebase,
- loadFilesFromFirebase,
- saveFilesToFirebase,
- };
- });
- vi.mock("socket.io-client", () => {
- return {
- default: () => {
- return {
- close: () => {},
- on: () => {},
- once: () => {},
- off: () => {},
- emit: () => {},
- };
- },
- };
- });
- /**
- * These test would deserve to be extended by testing collab with (at least) two clients simultanouesly,
- * while having access to both scenes, appstates stores, histories and etc.
- * i.e. multiplayer history tests could be a good first candidate, as we could test both history stacks simultaneously.
- */
- describe("collaboration", () => {
- it("should emit two ephemeral increments even though updates get batched", async () => {
- const durableIncrements: DurableIncrement[] = [];
- const ephemeralIncrements: EphemeralIncrement[] = [];
- await render(<ExcalidrawApp />);
- h.store.onStoreIncrementEmitter.on((increment) => {
- if (StoreIncrement.isDurable(increment)) {
- durableIncrements.push(increment);
- } else {
- ephemeralIncrements.push(increment);
- }
- });
- // eslint-disable-next-line dot-notation
- expect(h.store["scheduledMicroActions"].length).toBe(0);
- expect(durableIncrements.length).toBe(0);
- expect(ephemeralIncrements.length).toBe(0);
- const rectProps = {
- type: "rectangle",
- id: "A",
- height: 200,
- width: 100,
- x: 0,
- y: 0,
- } as const;
- const rect = API.createElement({ ...rectProps });
- API.updateScene({
- elements: [rect],
- captureUpdate: CaptureUpdateAction.IMMEDIATELY,
- });
- await waitFor(() => {
- // expect(commitSpy).toHaveBeenCalledTimes(1);
- expect(durableIncrements.length).toBe(1);
- });
- // simulate two batched remote updates
- act(() => {
- h.app.updateScene({
- elements: [newElementWith(h.elements[0], { x: 100 })],
- captureUpdate: CaptureUpdateAction.NEVER,
- });
- h.app.updateScene({
- elements: [newElementWith(h.elements[0], { x: 200 })],
- captureUpdate: CaptureUpdateAction.NEVER,
- });
- // we scheduled two micro actions,
- // which confirms they are going to be executed as part of one batched component update
- // eslint-disable-next-line dot-notation
- expect(h.store["scheduledMicroActions"].length).toBe(2);
- });
- await waitFor(() => {
- // altough the updates get batched,
- // we expect two ephemeral increments for each update,
- // and each such update should have the expected change
- expect(ephemeralIncrements.length).toBe(2);
- expect(ephemeralIncrements[0].change.elements.A).toEqual(
- expect.objectContaining({ x: 100 }),
- );
- expect(ephemeralIncrements[1].change.elements.A).toEqual(
- expect.objectContaining({ x: 200 }),
- );
- // eslint-disable-next-line dot-notation
- expect(h.store["scheduledMicroActions"].length).toBe(0);
- });
- });
- it("should allow to undo / redo even on force-deleted elements", async () => {
- await render(<ExcalidrawApp />);
- const rect1Props = {
- type: "rectangle",
- id: "A",
- height: 200,
- width: 100,
- } as const;
- const rect2Props = {
- type: "rectangle",
- id: "B",
- width: 100,
- height: 200,
- } as const;
- const rect1 = API.createElement({ ...rect1Props });
- const rect2 = API.createElement({ ...rect2Props });
- API.updateScene({
- elements: syncInvalidIndices([rect1, rect2]),
- captureUpdate: CaptureUpdateAction.IMMEDIATELY,
- });
- API.updateScene({
- elements: syncInvalidIndices([
- rect1,
- newElementWith(h.elements[1], { isDeleted: true }),
- ]),
- captureUpdate: CaptureUpdateAction.IMMEDIATELY,
- });
- await waitFor(() => {
- expect(API.getUndoStack().length).toBe(2);
- expect(API.getSnapshot()).toEqual([
- expect.objectContaining(rect1Props),
- expect.objectContaining({ ...rect2Props, isDeleted: true }),
- ]);
- expect(h.elements).toEqual([
- expect.objectContaining(rect1Props),
- expect.objectContaining({ ...rect2Props, isDeleted: true }),
- ]);
- });
- // one form of force deletion happens when starting the collab, not to sync potentially sensitive data into the server
- window.collab.startCollaboration(null);
- await waitFor(() => {
- expect(API.getUndoStack().length).toBe(2);
- // we never delete from the local snapshot as it is used for correct diff calculation
- expect(API.getSnapshot()).toEqual([
- expect.objectContaining(rect1Props),
- expect.objectContaining({ ...rect2Props, isDeleted: true }),
- ]);
- expect(h.elements).toEqual([expect.objectContaining(rect1Props)]);
- });
- const undoAction = createUndoAction(h.history);
- act(() => h.app.actionManager.executeAction(undoAction));
- // with explicit undo (as addition) we expect our item to be restored from the snapshot!
- await waitFor(() => {
- expect(API.getUndoStack().length).toBe(1);
- expect(API.getRedoStack().length).toBe(1);
- expect(API.getSnapshot()).toEqual([
- expect.objectContaining(rect1Props),
- expect.objectContaining({ ...rect2Props, isDeleted: false }),
- ]);
- expect(h.elements).toEqual([
- expect.objectContaining(rect1Props),
- expect.objectContaining({ ...rect2Props, isDeleted: false }),
- ]);
- });
- // simulate force deleting the element remotely
- API.updateScene({
- elements: syncInvalidIndices([rect1]),
- captureUpdate: CaptureUpdateAction.NEVER,
- });
- await waitFor(() => {
- expect(API.getUndoStack().length).toBe(1);
- expect(API.getRedoStack().length).toBe(1);
- expect(API.getSnapshot()).toEqual([
- expect.objectContaining(rect1Props),
- expect.objectContaining({ ...rect2Props, isDeleted: true }),
- ]);
- expect(h.elements).toEqual([expect.objectContaining(rect1Props)]);
- });
- const redoAction = createRedoAction(h.history);
- act(() => h.app.actionManager.executeAction(redoAction));
- // with explicit redo (as removal) we again restore the element from the snapshot!
- await waitFor(() => {
- expect(API.getUndoStack().length).toBe(2);
- expect(API.getRedoStack().length).toBe(0);
- expect(API.getSnapshot()).toEqual([
- expect.objectContaining(rect1Props),
- expect.objectContaining({ ...rect2Props, isDeleted: true }),
- ]);
- expect(h.elements).toEqual([
- expect.objectContaining(rect1Props),
- expect.objectContaining({ ...rect2Props, isDeleted: true }),
- ]);
- });
- });
- });
|