@tuturuuu/ui 0.17.0 → 0.19.0

package/src/components/ui/tu-do/shared/task-edit-dialog/hooks/__tests__/use-task-yjs-sync.test.ts

@@ -1,190 +0,0 @@
-import { renderHook, waitFor } from '@testing-library/react';
-import type { JSONContent } from '@tiptap/react';
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import * as Y from 'yjs';
-import { useTaskYjsSync } from '../use-task-yjs-sync';
-
-const taskApiMocks = vi.hoisted(() => ({
-  mockFetchWorkspaceTaskDescription: vi.fn(),
-  mockUpdateWorkspaceTaskDescription: vi.fn(),
-}));
-
-const yjsHelperMocks = vi.hoisted(() => ({
-  mockConvertJsonContentToYjsState: vi.fn(),
-  mockConvertYjsStateToJsonContent: vi.fn(),
-}));
-
-vi.mock('../task-api', () => ({
-  fetchWorkspaceTaskDescription: taskApiMocks.mockFetchWorkspaceTaskDescription,
-  updateWorkspaceTaskDescription:
-    taskApiMocks.mockUpdateWorkspaceTaskDescription,
-}));
-
-vi.mock('@tuturuuu/utils/yjs-helper', () => ({
-  convertJsonContentToYjsState: yjsHelperMocks.mockConvertJsonContentToYjsState,
-  convertYjsStateToJsonContent: yjsHelperMocks.mockConvertYjsStateToJsonContent,
-}));
-
-function createValidYjsState(): Uint8Array {
-  const doc = new Y.Doc();
-  doc.getMap('seed').set('k', 'v');
-  return Y.encodeStateAsUpdate(doc);
-}
-
-describe('useTaskYjsSync', () => {
-  const description: JSONContent = {
-    type: 'doc',
-    content: [
-      {
-        type: 'paragraph',
-        content: [{ type: 'text', text: 'Canonical description' }],
-      },
-    ],
-  };
-
-  const makeProps = () => ({
-    taskId: 'task-1',
-    wsId: 'ws-1',
-    boardId: 'board-1',
-    isOpen: true,
-    isCreateMode: false,
-    realtimeEnabled: false,
-    description,
-    editorInstance: { schema: {} } as any,
-    doc: new Y.Doc(),
-    queryClient: {} as any,
-    flushEditorPendingRef: { current: undefined },
-  });
-
-  beforeEach(() => {
-    vi.resetAllMocks();
-  });
-
-  it('initializes missing yjs state from canonical description', async () => {
-    const yjsState = createValidYjsState();
-    yjsHelperMocks.mockConvertJsonContentToYjsState.mockReturnValueOnce(
-      yjsState
-    );
-    taskApiMocks.mockFetchWorkspaceTaskDescription.mockResolvedValueOnce({
-      description: JSON.stringify(description),
-      description_yjs_state: null,
-    });
-    taskApiMocks.mockUpdateWorkspaceTaskDescription.mockResolvedValueOnce({});
-
-    renderHook(() => useTaskYjsSync(makeProps()));
-
-    await waitFor(() => {
-      expect(
-        taskApiMocks.mockUpdateWorkspaceTaskDescription
-      ).toHaveBeenCalledWith('ws-1', 'task-1', {
-        description_yjs_state: Array.from(yjsState),
-      });
-    });
-  });
-
-  it('does not rewrite persisted yjs state when description data is stale', async () => {
-    yjsHelperMocks.mockConvertYjsStateToJsonContent.mockReturnValueOnce({
-      type: 'doc',
-      content: [
-        {
-          type: 'paragraph',
-          content: [{ type: 'text', text: 'Stale content' }],
-        },
-      ],
-    });
-    taskApiMocks.mockFetchWorkspaceTaskDescription.mockResolvedValueOnce({
-      description: JSON.stringify(description),
-      description_yjs_state: [7, 7, 7],
-    });
-    taskApiMocks.mockUpdateWorkspaceTaskDescription.mockResolvedValueOnce({});
-
-    renderHook(() => useTaskYjsSync(makeProps()));
-
-    await waitFor(() => {
-      expect(
-        taskApiMocks.mockFetchWorkspaceTaskDescription
-      ).toHaveBeenCalledWith('ws-1', 'task-1');
-    });
-
-    expect(
-      taskApiMocks.mockUpdateWorkspaceTaskDescription
-    ).not.toHaveBeenCalled();
-    expect(
-      yjsHelperMocks.mockConvertJsonContentToYjsState
-    ).not.toHaveBeenCalled();
-  });
-
-  it('heals unreadable persisted yjs state from the persisted description', async () => {
-    const healedState = createValidYjsState();
-    yjsHelperMocks.mockConvertYjsStateToJsonContent.mockImplementationOnce(
-      () => {
-        throw new Error('Corrupt Yjs state');
-      }
-    );
-    yjsHelperMocks.mockConvertJsonContentToYjsState.mockReturnValueOnce(
-      healedState
-    );
-    taskApiMocks.mockFetchWorkspaceTaskDescription.mockResolvedValueOnce({
-      description: JSON.stringify(description),
-      description_yjs_state: [7, 7, 7],
-    });
-    taskApiMocks.mockUpdateWorkspaceTaskDescription.mockResolvedValueOnce({});
-
-    renderHook(() => useTaskYjsSync(makeProps()));
-
-    await waitFor(() => {
-      expect(
-        taskApiMocks.mockUpdateWorkspaceTaskDescription
-      ).toHaveBeenCalledWith('ws-1', 'task-1', {
-        description_yjs_state: Array.from(healedState),
-      });
-    });
-  });
-
-  it('does not rewrite yjs state when persisted state already matches description', async () => {
-    yjsHelperMocks.mockConvertYjsStateToJsonContent.mockReturnValueOnce(
-      description
-    );
-    taskApiMocks.mockFetchWorkspaceTaskDescription.mockResolvedValueOnce({
-      description: JSON.stringify(description),
-      description_yjs_state: [1, 2, 3],
-    });
-
-    renderHook(() => useTaskYjsSync(makeProps()));
-
-    await waitFor(() => {
-      expect(
-        taskApiMocks.mockFetchWorkspaceTaskDescription
-      ).toHaveBeenCalledWith('ws-1', 'task-1');
-    });
-
-    expect(
-      taskApiMocks.mockUpdateWorkspaceTaskDescription
-    ).not.toHaveBeenCalled();
-  });
-
-  it('does not rewrite yjs state when description payload is null and persisted state exists', async () => {
-    const props = makeProps();
-    const yjsState = createValidYjsState();
-
-    taskApiMocks.mockFetchWorkspaceTaskDescription.mockResolvedValueOnce({
-      description: null,
-      description_yjs_state: Array.from(yjsState),
-    });
-
-    renderHook(() => useTaskYjsSync(props));
-
-    await waitFor(() => {
-      expect(
-        taskApiMocks.mockFetchWorkspaceTaskDescription
-      ).toHaveBeenCalledWith('ws-1', 'task-1');
-    });
-
-    expect(
-      taskApiMocks.mockUpdateWorkspaceTaskDescription
-    ).not.toHaveBeenCalled();
-    expect(
-      yjsHelperMocks.mockConvertJsonContentToYjsState
-    ).not.toHaveBeenCalled();
-  });
-});

package/src/components/ui/tu-do/shared/task-edit-dialog/hooks/__tests__/use-update-shared-task.test.ts

@@ -1,107 +0,0 @@
-import { renderHook } from '@testing-library/react';
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { useUpdateSharedTask } from '../use-update-shared-task';
-
-// Mock TanStack Query's useMutation
-const mockMutate = vi.fn();
-vi.mock('@tanstack/react-query', async () => {
-  const actual = await vi.importActual('@tanstack/react-query');
-  return {
-    ...actual,
-    useMutation: vi.fn((options) => {
-      return {
-        mutate: mockMutate,
-        mutateAsync: async (variables: any) => {
-          return options.mutationFn(variables);
-        },
-        ...options,
-      };
-    }),
-    useQueryClient: () => ({
-      invalidateQueries: vi.fn(),
-    }),
-  };
-});
-
-// Mock fetch
-global.fetch = vi.fn();
-
-describe('useUpdateSharedTask', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('should successfully update a task', async () => {
-    const mockResponse = { id: 'task-1', name: 'Updated Task' };
-    (global.fetch as any).mockResolvedValueOnce({
-      ok: true,
-      json: async () => mockResponse,
-    });
-
-    const { result } = renderHook(() => useUpdateSharedTask());
-
-    const payload = {
-      shareCode: 'SHARE123',
-      updates: { name: 'Updated Task' },
-    };
-
-    // Cast to any to access mutationFn which is available in the mock return value
-    const response = await (result.current as any).mutationFn(payload);
-
-    expect(global.fetch).toHaveBeenCalledWith(
-      'http://localhost:7809/api/v1/shared/tasks/SHARE123',
-      {
-        method: 'PATCH',
-        credentials: 'include',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify(payload.updates),
-      }
-    );
-    expect(response).toEqual(mockResponse);
-  });
-
-  it('should handle API errors', async () => {
-    (global.fetch as any).mockResolvedValueOnce({
-      ok: false,
-      json: async () => ({ error: 'Update failed' }),
-    });
-
-    const { result } = renderHook(() => useUpdateSharedTask());
-
-    const payload = {
-      shareCode: 'SHARE123',
-      updates: { name: 'Fail Task' },
-    };
-
-    await expect((result.current as any).mutationFn(payload)).rejects.toThrow(
-      'Update failed'
-    );
-  });
-
-  it('should send correct payload structure', async () => {
-    (global.fetch as any).mockResolvedValueOnce({
-      ok: true,
-      json: async () => ({}),
-    });
-
-    const { result } = renderHook(() => useUpdateSharedTask());
-
-    const updates = {
-      name: 'New Name',
-      priority: 'high',
-      description: 'New Description',
-    };
-
-    await (result.current as any).mutationFn({
-      shareCode: 'CODE',
-      updates,
-    });
-
-    expect(global.fetch).toHaveBeenCalledWith(
-      expect.stringContaining('CODE'),
-      expect.objectContaining({
-        body: JSON.stringify(updates),
-      })
-    );
-  });
-});

package/src/components/ui/tu-do/shared/task-edit-dialog/hooks/task-api.test.ts

@@ -1,171 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-
-const internalApiMocks = vi.hoisted(() => ({
-  abortWorkspaceTaskDescriptionChunks: vi.fn(),
-  appendWorkspaceTaskDescriptionChunk: vi.fn(),
-  beginWorkspaceTaskDescriptionChunks: vi.fn(),
-  commitWorkspaceTaskDescriptionChunks: vi.fn(),
-  createWorkspaceTaskProject: vi.fn(),
-  getWorkspaceTask: vi.fn(),
-  getWorkspaceTaskDescription: vi.fn(),
-  updateWorkspaceTask: vi.fn(),
-  updateWorkspaceTaskDescription: vi.fn(),
-}));
-
-vi.mock('@tuturuuu/internal-api/tasks', () => internalApiMocks);
-
-import {
-  shouldChunkTaskDescriptionPayload,
-  updateWorkspaceTaskDescription,
-} from './task-api';
-
-describe('task-api description persistence', () => {
-  beforeEach(() => {
-    vi.resetAllMocks();
-    internalApiMocks.abortWorkspaceTaskDescriptionChunks.mockResolvedValue({
-      success: true,
-    });
-    internalApiMocks.appendWorkspaceTaskDescriptionChunk.mockResolvedValue({
-      success: true,
-    });
-    internalApiMocks.beginWorkspaceTaskDescriptionChunks.mockResolvedValue({
-      session_id: 'chunk-session-1',
-    });
-    internalApiMocks.commitWorkspaceTaskDescriptionChunks.mockResolvedValue({
-      description: 'persisted',
-      description_yjs_state: [1, 2, 3],
-    });
-    internalApiMocks.updateWorkspaceTaskDescription.mockResolvedValue({
-      description: 'small',
-      description_yjs_state: [1, 2, 3],
-    });
-  });
-
-  it('uses the direct description update for small payloads', async () => {
-    const payload = {
-      description: JSON.stringify({
-        type: 'doc',
-        content: [{ type: 'paragraph', content: [{ text: 'Small' }] }],
-      }),
-      description_yjs_state: [1, 2, 3],
-    };
-
-    await expect(
-      updateWorkspaceTaskDescription('ws-1', 'task-1', payload)
-    ).resolves.toEqual({
-      description: 'small',
-      description_yjs_state: [1, 2, 3],
-    });
-
-    expect(
-      internalApiMocks.updateWorkspaceTaskDescription
-    ).toHaveBeenCalledWith('ws-1', 'task-1', payload);
-    expect(
-      internalApiMocks.beginWorkspaceTaskDescriptionChunks
-    ).not.toHaveBeenCalled();
-  });
-
-  it('uploads large description updates through ordered chunks', async () => {
-    const yjsState = Array.from({ length: 220_000 }, (_, index) => index % 256);
-    const payload = {
-      description: JSON.stringify({
-        type: 'doc',
-        content: [
-          {
-            type: 'paragraph',
-            content: [{ type: 'text', text: 'Large paste' }],
-          },
-        ],
-      }),
-      description_yjs_state: yjsState,
-    };
-
-    expect(shouldChunkTaskDescriptionPayload(payload)).toBe(true);
-
-    await updateWorkspaceTaskDescription('ws-1', 'task-1', payload);
-
-    expect(
-      internalApiMocks.updateWorkspaceTaskDescription
-    ).not.toHaveBeenCalled();
-    expect(
-      internalApiMocks.beginWorkspaceTaskDescriptionChunks
-    ).toHaveBeenCalledWith(
-      'ws-1',
-      'task-1',
-      expect.objectContaining({
-        description: expect.objectContaining({
-          chunk_count: 1,
-        }),
-        description_yjs_state: expect.objectContaining({
-          chunk_count: expect.any(Number),
-        }),
-      })
-    );
-
-    const appendCalls =
-      internalApiMocks.appendWorkspaceTaskDescriptionChunk.mock.calls;
-    expect(appendCalls.length).toBeGreaterThan(2);
-    expect(appendCalls[0]).toEqual([
-      'ws-1',
-      'task-1',
-      expect.objectContaining({
-        chunk_index: 0,
-        field: 'description',
-        session_id: 'chunk-session-1',
-      }),
-    ]);
-    expect(
-      appendCalls
-        .filter((call) => call[2].field === 'description_yjs_state')
-        .map((call) => call[2].chunk_index)
-    ).toEqual(
-      appendCalls
-        .filter((call) => call[2].field === 'description_yjs_state')
-        .map((_, index) => index)
-    );
-    expect(
-      internalApiMocks.commitWorkspaceTaskDescriptionChunks
-    ).toHaveBeenCalledWith('ws-1', 'task-1', 'chunk-session-1');
-  });
-
-  it('falls back to chunked upload when a direct save hits the proxy body limit', async () => {
-    internalApiMocks.updateWorkspaceTaskDescription.mockRejectedValueOnce({
-      status: 413,
-    });
-
-    await updateWorkspaceTaskDescription('ws-1', 'task-1', {
-      description: 'small enough to try directly',
-      description_yjs_state: [1, 2, 3],
-    });
-
-    expect(
-      internalApiMocks.beginWorkspaceTaskDescriptionChunks
-    ).toHaveBeenCalled();
-    expect(
-      internalApiMocks.commitWorkspaceTaskDescriptionChunks
-    ).toHaveBeenCalled();
-  });
-
-  it('aborts the chunk session when an append fails', async () => {
-    internalApiMocks.appendWorkspaceTaskDescriptionChunk.mockRejectedValueOnce(
-      new Error('network down')
-    );
-
-    await expect(
-      updateWorkspaceTaskDescription('ws-1', 'task-1', {
-        description: 'x'.repeat(10),
-        description_yjs_state: Array.from(
-          { length: 220_000 },
-          (_, index) => index % 256
-        ),
-      })
-    ).rejects.toThrow('network down');
-
-    expect(
-      internalApiMocks.abortWorkspaceTaskDescriptionChunks
-    ).toHaveBeenCalledWith('ws-1', 'task-1', 'chunk-session-1');
-    expect(
-      internalApiMocks.commitWorkspaceTaskDescriptionChunks
-    ).not.toHaveBeenCalled();
-  });
-});