Mathis/lemma
1
0
Fork 0
mirror of https://github.com/lordmathis/lemma.git synced 2025-11-06 16:04:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove duplicate error handling tests from useWorkspaceOperations test

Browse Source
This commit is contained in:
Matúš Námešný
2025-06-04 19:22:21 +02:00
parent 07af3f6e39
commit 0fd87c072d
1 changed files with 0 additions and 147 deletions
Download Patch File Download Diff File Expand all files Collapse all files

147
app/src/hooks/useWorkspaceOperations.test.ts
View File

@@ -204,39 +204,6 @@ describe('useWorkspaceOperations', () => {
consoleSpy.mockRestore();
});
it('handles load workspaces errors during switch', async () => {
const mockUpdateLastWorkspaceName = vi.mocked(
workspaceApi.updateLastWorkspaceName
);
const consoleSpy = vi
.spyOn(console, 'error')
.mockImplementation(() => {});
mockUpdateLastWorkspaceName.mockResolvedValue(undefined);
mockWorkspaceData.loadWorkspaceData.mockResolvedValue(undefined);
mockWorkspaceData.loadWorkspaces.mockRejectedValue(
new Error('Load workspaces failed')
);
const { result } = renderHook(() => useWorkspaceOperations());
await act(async () => {
await result.current.switchWorkspace('error-workspace');
});
expect(consoleSpy).toHaveBeenCalledWith(
'Failed to switch workspace:',
expect.any(Error)
);
expect(notifications.show).toHaveBeenCalledWith({
title: 'Error',
message: 'Failed to switch workspace',
color: 'red',
});
consoleSpy.mockRestore();
});
});
describe('deleteCurrentWorkspace', () => {
@@ -325,65 +292,6 @@ describe('useWorkspaceOperations', () => {
consoleSpy.mockRestore();
});
it('handles load workspace data errors after deletion', async () => {
const mockDeleteWorkspace = vi.mocked(workspaceApi.deleteWorkspace);
const consoleSpy = vi
.spyOn(console, 'error')
.mockImplementation(() => {});
mockWorkspaceData.loadWorkspaces.mockResolvedValue(mockWorkspaces);
mockDeleteWorkspace.mockResolvedValue('next-workspace');
mockWorkspaceData.loadWorkspaceData.mockRejectedValue(
new Error('Load failed')
);
const { result } = renderHook(() => useWorkspaceOperations());
await act(async () => {
await result.current.deleteCurrentWorkspace();
});
expect(consoleSpy).toHaveBeenCalledWith(
'Failed to delete workspace:',
expect.any(Error)
);
expect(notifications.show).toHaveBeenCalledWith({
title: 'Error',
message: 'Failed to delete workspace',
color: 'red',
});
consoleSpy.mockRestore();
});
it('handles load workspaces errors during deletion check', async () => {
const consoleSpy = vi
.spyOn(console, 'error')
.mockImplementation(() => {});
mockWorkspaceData.loadWorkspaces.mockRejectedValue(
new Error('Load workspaces failed')
);
const { result } = renderHook(() => useWorkspaceOperations());
await act(async () => {
await result.current.deleteCurrentWorkspace();
});
expect(consoleSpy).toHaveBeenCalledWith(
'Failed to delete workspace:',
expect.any(Error)
);
expect(notifications.show).toHaveBeenCalledWith({
title: 'Error',
message: 'Failed to delete workspace',
color: 'red',
});
consoleSpy.mockRestore();
});
});
describe('updateSettings', () => {
@@ -524,40 +432,6 @@ describe('useWorkspaceOperations', () => {
consoleSpy.mockRestore();
});
it('handles load workspaces errors after update', async () => {
const mockUpdateWorkspace = vi.mocked(workspaceApi.updateWorkspace);
const consoleSpy = vi
.spyOn(console, 'error')
.mockImplementation(() => {});
const updatedWorkspace: Workspace = {
...mockWorkspaceData.currentWorkspace!,
autoSave: true,
};
mockUpdateWorkspace.mockResolvedValue(updatedWorkspace);
mockWorkspaceData.loadWorkspaces.mockRejectedValue(
new Error('Load workspaces failed')
);
const { result } = renderHook(() => useWorkspaceOperations());
await act(async () => {
try {
await result.current.updateSettings({ autoSave: true });
} catch (error) {
expect(error).toBeInstanceOf(Error);
expect((error as Error).message).toBe('Load workspaces failed');
}
});
expect(consoleSpy).toHaveBeenCalledWith(
'Failed to save settings:',
expect.any(Error)
);
consoleSpy.mockRestore();
});
it('handles empty settings update', async () => {
const mockUpdateWorkspace = vi.mocked(workspaceApi.updateWorkspace);
const updatedWorkspace = mockWorkspaceData.currentWorkspace!;
@@ -667,27 +541,6 @@ describe('useWorkspaceOperations', () => {
});
describe('concurrent operations', () => {
it('handles multiple concurrent switch operations', async () => {
const mockUpdateLastWorkspaceName = vi.mocked(
workspaceApi.updateLastWorkspaceName
);
mockUpdateLastWorkspaceName.mockResolvedValue(undefined);
mockWorkspaceData.loadWorkspaceData.mockResolvedValue(undefined);
mockWorkspaceData.loadWorkspaces.mockResolvedValue(mockWorkspaces);
const { result } = renderHook(() => useWorkspaceOperations());
await act(async () => {
await Promise.all([
result.current.switchWorkspace('workspace-1'),
result.current.switchWorkspace('workspace-2'),
]);
});
expect(mockUpdateLastWorkspaceName).toHaveBeenCalledTimes(2);
expect(mockWorkspaceData.loadWorkspaceData).toHaveBeenCalledTimes(2);
expect(mockWorkspaceData.loadWorkspaces).toHaveBeenCalledTimes(2);
});
it('handles update settings after switch workspace', async () => {
const mockUpdateLastWorkspaceName = vi.mocked(