Expand settings test coverage

Made-with: Cursor

Author: root

test/unit/pages/Setting/GeneralProxy.test.tsx

@@ -179,5 +179,50 @@ describe('SettingGeneral Network Proxy', () => {
 
     expect((window as any).electronAPI.restartApp).toHaveBeenCalled();
   });
-}
+  it('removes proxy when input cleared', async () => {
+    // Start with existing value
+    render(<SettingGeneral />);
+
+    const input = await screen.findByPlaceholderText(
+      'setting.proxy-placeholder'
+    );
+
+    await userEvent.clear(input);
+
+    const button = screen.getByRole('button', { name: 'setting.save' });
+    await userEvent.click(button);
+
+    await waitFor(() => {
+      expect((window as any).electronAPI.envRemove).toHaveBeenCalledWith(
+        'test@example.com',
+        'HTTP_PROXY'
+      );
+      expect(toastSuccessMock).toHaveBeenCalledWith(
+        'setting.proxy-saved-restart-required'
+      );
+    });
+  });
+
+  it('shows error when electron env APIs are missing', async () => {
+    (window as any).electronAPI = {
+      getPlatform: () => 'linux',
+      readGlobalEnv: vi.fn().mockResolvedValue({ value: '' }),
+      // envWrite/envRemove intentionally omitted
+    };
+
+    render(<SettingGeneral />);
+
+    const input = await screen.findByPlaceholderText(
+      'setting.proxy-placeholder'
+    );
+    await userEvent.type(input, 'http://proxy:8080');
+
+    const button = screen.getByRole('button', { name: 'setting.save' });
+    await userEvent.click(button);
+
+    await waitFor(() => {
+      expect(toastErrorMock).toHaveBeenCalledWith('setting.proxy-save-failed');
+    });
+  });
+});
 

test/unit/pages/Setting/Privacy.test.tsx

@@ -106,5 +106,28 @@ describe('SettingPrivacy', () => {
       expect(toastErrorMock).toHaveBeenCalledWith('setting.save-failed');
     });
   });
+
+  it('disables switch while loading and enables after load', async () => {
+    proxyFetchGetMock.mockResolvedValueOnce({ help_improve: false });
+
+    render(<SettingPrivacy />);
+
+    const switchEl = await screen.findByRole('switch');
+
+    // After initial load completes, loading should be false and switch enabled
+    expect(switchEl).not.toBeDisabled();
+  });
+
+  it('shows disabled status text when help improve is off', async () => {
+    proxyFetchGetMock.mockResolvedValueOnce({ help_improve: false });
+
+    render(<SettingPrivacy />);
+
+    await screen.findByRole('switch');
+
+    expect(
+      screen.getByText('setting.disabled', { exact: false })
+    ).toBeInTheDocument();
+  });
 });