Refactor tests in useGitOperations and useWorkspaceOperations to handle undefined values gracefully

app/src/hooks/useGitOperations.test.ts

@@ -380,7 +380,7 @@ describe('useGitOperations', () => {
     it('handles undefined workspace name gracefully', async () => {
       mockWorkspaceData.currentWorkspace = {
         id: 1,
-        name: undefined as unknown as string,
+        name: undefined!,
       };
 
       const { result } = renderHook(() => useGitOperations());
@@ -396,7 +396,7 @@ describe('useGitOperations', () => {
 
     it('handles missing settings gracefully', async () => {
       mockWorkspaceData.settings = {
-        gitEnabled: undefined as unknown as boolean,
+        gitEnabled: undefined!,
       };
 
       const { result } = renderHook(() => useGitOperations());
@@ -413,7 +413,7 @@ describe('useGitOperations', () => {
     it('handles API returning non-string commit hash', async () => {
       const mockCommitAndPush = vi.mocked(gitApi.commitAndPush);
       // API might return something unexpected
-      mockCommitAndPush.mockResolvedValue(null as unknown as string);
+      mockCommitAndPush.mockResolvedValue(null!);
 
       const { result } = renderHook(() => useGitOperations());
 

app/src/hooks/useWorkspaceOperations.test.ts

@@ -390,7 +390,7 @@ describe('useWorkspaceOperations', () => {
     it('updates workspace settings successfully', async () => {
       const mockUpdateWorkspace = vi.mocked(workspaceApi.updateWorkspace);
       const updatedWorkspace: Workspace = {
-        ...(mockWorkspaceData.currentWorkspace as Workspace),
+        ...mockWorkspaceData.currentWorkspace!,
         autoSave: true,
         showHiddenFiles: true,
       };
@@ -421,25 +421,8 @@ describe('useWorkspaceOperations', () => {
     it('updates theme and calls updateColorScheme', async () => {
       const mockUpdateWorkspace = vi.mocked(workspaceApi.updateWorkspace);
       const updatedWorkspace: Workspace = {
-        ...(mockWorkspaceData.currentWorkspace ?? {
+        ...mockWorkspaceData.currentWorkspace!,
-          id: 1,
-          userId: 1,
-          name: 'test-workspace',
-          createdAt: '2024-01-01T00:00:00Z',
-          theme: Theme.Light,
-          autoSave: false,
-          showHiddenFiles: false,
-          gitEnabled: false,
-          gitUrl: '',
-          gitUser: '',
-          gitToken: '',
-          gitAutoCommit: false,
-          gitCommitMsgTemplate: '${action} ${filename}',
-          gitCommitName: '',
-          gitCommitEmail: '',
-        }),
         theme: Theme.Dark,
-        name: mockWorkspaceData.currentWorkspace?.name ?? 'test-workspace',
       };
       mockUpdateWorkspace.mockResolvedValue(updatedWorkspace);
       mockWorkspaceData.loadWorkspaces.mockResolvedValue(mockWorkspaces);
@@ -467,23 +450,7 @@ describe('useWorkspaceOperations', () => {
     it('updates multiple settings including theme', async () => {
       const mockUpdateWorkspace = vi.mocked(workspaceApi.updateWorkspace);
       const updatedWorkspace: Workspace = {
-        ...(mockWorkspaceData.currentWorkspace ?? {
+        ...mockWorkspaceData.currentWorkspace!,
-          id: 1,
-          userId: 1,
-          name: 'test-workspace',
-          createdAt: '2024-01-01T00:00:00Z',
-          theme: Theme.Light,
-          autoSave: false,
-          showHiddenFiles: false,
-          gitEnabled: false,
-          gitUrl: '',
-          gitUser: '',
-          gitToken: '',
-          gitAutoCommit: false,
-          gitCommitMsgTemplate: '${action} ${filename}',
-          gitCommitName: '',
-          gitCommitEmail: '',
-        }),
         theme: Theme.Dark,
         autoSave: true,
         gitEnabled: true,
@@ -564,23 +531,7 @@ describe('useWorkspaceOperations', () => {
         .mockImplementation(() => {});
 
       const updatedWorkspace: Workspace = {
-        ...(mockWorkspaceData.currentWorkspace ?? {
+        ...mockWorkspaceData.currentWorkspace!,
-          id: 1,
-          userId: 1,
-          name: 'test-workspace',
-          createdAt: '2024-01-01T00:00:00Z',
-          theme: Theme.Light,
-          autoSave: false,
-          showHiddenFiles: false,
-          gitEnabled: false,
-          gitUrl: '',
-          gitUser: '',
-          gitToken: '',
-          gitAutoCommit: false,
-          gitCommitMsgTemplate: '${action} ${filename}',
-          gitCommitName: '',
-          gitCommitEmail: '',
-        }),
         autoSave: true,
       };
       mockUpdateWorkspace.mockResolvedValue(updatedWorkspace);
@@ -609,23 +560,7 @@ describe('useWorkspaceOperations', () => {
 
     it('handles empty settings update', async () => {
       const mockUpdateWorkspace = vi.mocked(workspaceApi.updateWorkspace);
-      const updatedWorkspace = mockWorkspaceData.currentWorkspace ?? {
+      const updatedWorkspace = mockWorkspaceData.currentWorkspace!;
-        id: 1,
-        userId: 1,
-        name: 'test-workspace',
-        createdAt: '2024-01-01T00:00:00Z',
-        theme: Theme.Light,
-        autoSave: false,
-        showHiddenFiles: false,
-        gitEnabled: false,
-        gitUrl: '',
-        gitUser: '',
-        gitToken: '',
-        gitAutoCommit: false,
-        gitCommitMsgTemplate: '${action} ${filename}',
-        gitCommitName: '',
-        gitCommitEmail: '',
-      };
       mockUpdateWorkspace.mockResolvedValue(updatedWorkspace);
       mockWorkspaceData.loadWorkspaces.mockResolvedValue(mockWorkspaces);
 
@@ -684,23 +619,7 @@ describe('useWorkspaceOperations', () => {
 
       // Update workspace name
       mockWorkspaceData.currentWorkspace = {
-        ...(mockWorkspaceData.currentWorkspace ?? {
+        ...mockWorkspaceData.currentWorkspace!,
-          id: 1,
-          userId: 1,
-          name: 'different-workspace',
-          createdAt: '2024-01-01T00:00:00Z',
-          theme: Theme.Light,
-          autoSave: false,
-          showHiddenFiles: false,
-          gitEnabled: false,
-          gitUrl: '',
-          gitUser: '',
-          gitToken: '',
-          gitAutoCommit: false,
-          gitCommitMsgTemplate: '${action} ${filename}',
-          gitCommitName: '',
-          gitCommitEmail: '',
-        }),
         name: 'different-workspace',
       };
 
@@ -734,45 +653,8 @@ describe('useWorkspaceOperations', () => {
 
       // Change workspace
       mockWorkspaceData.currentWorkspace = {
-        ...(mockWorkspaceData.currentWorkspace ?? {
+        ...mockWorkspaceData.currentWorkspace!,
-          id: 1,
-          userId: 1,
-          name: 'new-workspace',
-          createdAt: '2024-01-01T00:00:00Z',
-          theme: Theme.Light,
-          autoSave: false,
-          showHiddenFiles: false,
-          gitEnabled: false,
-          gitUrl: '',
-          gitUser: '',
-          gitToken: '',
-          gitAutoCommit: false,
-          gitCommitMsgTemplate: '${action} ${filename}',
-          gitCommitName: '',
-          gitCommitEmail: '',
-        }),
         name: 'new-workspace',
-        createdAt:
-          mockWorkspaceData.currentWorkspace?.createdAt ??
-          '2024-01-01T00:00:00Z',
-        id: mockWorkspaceData.currentWorkspace?.id ?? 1,
-        userId: mockWorkspaceData.currentWorkspace?.userId ?? 1,
-        theme: mockWorkspaceData.currentWorkspace?.theme ?? Theme.Light,
-        autoSave: mockWorkspaceData.currentWorkspace?.autoSave ?? false,
-        showHiddenFiles:
-          mockWorkspaceData.currentWorkspace?.showHiddenFiles ?? false,
-        gitEnabled: mockWorkspaceData.currentWorkspace?.gitEnabled ?? false,
-        gitUrl: mockWorkspaceData.currentWorkspace?.gitUrl ?? '',
-        gitUser: mockWorkspaceData.currentWorkspace?.gitUser ?? '',
-        gitToken: mockWorkspaceData.currentWorkspace?.gitToken ?? '',
-        gitAutoCommit:
-          mockWorkspaceData.currentWorkspace?.gitAutoCommit ?? false,
-        gitCommitMsgTemplate:
-          mockWorkspaceData.currentWorkspace?.gitCommitMsgTemplate ??
-          '${action} ${filename}',
-        gitCommitName: mockWorkspaceData.currentWorkspace?.gitCommitName ?? '',
-        gitCommitEmail:
-          mockWorkspaceData.currentWorkspace?.gitCommitEmail ?? '',
       };
 
       rerender();
@@ -816,27 +698,9 @@ describe('useWorkspaceOperations', () => {
       mockUpdateLastWorkspaceName.mockResolvedValue(undefined);
       mockWorkspaceData.loadWorkspaceData.mockResolvedValue(undefined);
       mockWorkspaceData.loadWorkspaces.mockResolvedValue(mockWorkspaces);
-
+      mockUpdateWorkspace.mockResolvedValue(
-      // Ensure we have a defined workspace to use
+        mockWorkspaceData.currentWorkspace!
-      const workspaceToUse: Workspace = {
+      );
-        id: 1,
-        userId: 1,
-        name: 'test-workspace',
-        createdAt: '2024-01-01T00:00:00Z',
-        theme: Theme.Light,
-        autoSave: false,
-        showHiddenFiles: false,
-        gitEnabled: false,
-        gitUrl: '',
-        gitUser: '',
-        gitToken: '',
-        gitAutoCommit: false,
-        gitCommitMsgTemplate: '${action} ${filename}',
-        gitCommitName: '',
-        gitCommitEmail: '',
-      };
-
-      mockUpdateWorkspace.mockResolvedValue(workspaceToUse);
 
       const { result } = renderHook(() => useWorkspaceOperations());