test: Add comprehensive tests for auto-start and system tray

- Add 8 Rust unit tests for StartupSettings struct
- Add 12 React unit tests for Switch component
- Add 13 E2E tests for startup settings functionality

All unit tests passing (Rust 8/8, React 12/12)
E2E tests: 21/39 passed (failures expected in web-only mode)

Author: Mohammod Al Amin Ashik

apps/desktop/src-tauri/src/commands/settings.rs

@@ -119,3 +119,99 @@ pub fn should_start_hidden() -> bool {
     let args: Vec<String> = std::env::args().collect();
     args.contains(&"--hidden".to_string())
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_startup_settings_default() {
+        let settings = StartupSettings::default();
+        assert_eq!(settings.auto_launch, false);
+        assert_eq!(settings.start_minimized, false);
+        assert_eq!(settings.close_to_tray, true);
+    }
+
+    #[test]
+    fn test_startup_settings_serialization() {
+        let settings = StartupSettings {
+            auto_launch: true,
+            start_minimized: false,
+            close_to_tray: true,
+        };
+
+        let json = serde_json::to_string(&settings).unwrap();
+        assert!(json.contains("\"autoLaunch\":true"));
+        assert!(json.contains("\"startMinimized\":false"));
+        assert!(json.contains("\"closeToTray\":true"));
+    }
+
+    #[test]
+    fn test_startup_settings_deserialization() {
+        let json = r#"{"autoLaunch":true,"startMinimized":true,"closeToTray":false}"#;
+        let settings: StartupSettings = serde_json::from_str(json).unwrap();
+
+        assert_eq!(settings.auto_launch, true);
+        assert_eq!(settings.start_minimized, true);
+        assert_eq!(settings.close_to_tray, false);
+    }
+
+    #[test]
+    fn test_should_start_hidden_without_flag() {
+        // This test might be tricky as it depends on actual process args
+        // In a real test environment, we'd mock std::env::args
+        // For now, we just verify the function exists and can be called
+        let _result = should_start_hidden();
+        // Can't assert the actual value since it depends on how tests are run
+    }
+
+    #[test]
+    fn test_startup_settings_clone() {
+        let settings = StartupSettings {
+            auto_launch: true,
+            start_minimized: false,
+            close_to_tray: true,
+        };
+
+        let cloned = settings.clone();
+        assert_eq!(settings.auto_launch, cloned.auto_launch);
+        assert_eq!(settings.start_minimized, cloned.start_minimized);
+        assert_eq!(settings.close_to_tray, cloned.close_to_tray);
+    }
+
+    #[test]
+    fn test_startup_settings_debug() {
+        let settings = StartupSettings::default();
+        let debug_str = format!("{:?}", settings);
+        assert!(debug_str.contains("StartupSettings"));
+        assert!(debug_str.contains("auto_launch"));
+        assert!(debug_str.contains("start_minimized"));
+        assert!(debug_str.contains("close_to_tray"));
+    }
+
+    #[test]
+    fn test_startup_settings_with_all_enabled() {
+        let settings = StartupSettings {
+            auto_launch: true,
+            start_minimized: true,
+            close_to_tray: true,
+        };
+
+        assert!(settings.auto_launch);
+        assert!(settings.start_minimized);
+        assert!(settings.close_to_tray);
+    }
+
+    #[test]
+    fn test_startup_settings_with_all_disabled() {
+        let settings = StartupSettings {
+            auto_launch: false,
+            start_minimized: false,
+            close_to_tray: false,
+        };
+
+        assert!(!settings.auto_launch);
+        assert!(!settings.start_minimized);
+        assert!(!settings.close_to_tray);
+    }
+}

packages/ui/package.json

@@ -18,7 +18,8 @@
     "dev": "tsc --watch",
     "lint": "eslint src",
     "lint:fix": "eslint src --fix",
-    "test": "echo \"No tests yet\""
+    "test": "vitest run",
+    "test:watch": "vitest"
   },
   "peerDependencies": {
     "react": "^19.0.0",

packages/ui/src/components/common/Switch.test.tsx

@@ -0,0 +1,133 @@
+/**
+ * Tests for Switch component
+ */
+
+import { describe, it, expect, vi } from 'vitest';
+import { render, screen, fireEvent } from '@testing-library/react';
+import { Switch } from './Switch';
+
+describe('Switch', () => {
+  it('renders with unchecked state', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={false} onCheckedChange={mockHandler} />);
+    
+    const button = screen.getByRole('switch');
+    expect(button).toBeInTheDocument();
+    expect(button).toHaveAttribute('aria-checked', 'false');
+  });
+
+  it('renders with checked state', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={true} onCheckedChange={mockHandler} />);
+    
+    const button = screen.getByRole('switch');
+    expect(button).toHaveAttribute('aria-checked', 'true');
+  });
+
+  it('calls onCheckedChange when clicked', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={false} onCheckedChange={mockHandler} />);
+    
+    const button = screen.getByRole('switch');
+    fireEvent.click(button);
+    
+    expect(mockHandler).toHaveBeenCalledWith(true);
+    expect(mockHandler).toHaveBeenCalledTimes(1);
+  });
+
+  it('toggles from checked to unchecked', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={true} onCheckedChange={mockHandler} />);
+    
+    const button = screen.getByRole('switch');
+    fireEvent.click(button);
+    
+    expect(mockHandler).toHaveBeenCalledWith(false);
+  });
+
+  it('does not call handler when disabled', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={false} onCheckedChange={mockHandler} disabled={true} />);
+    
+    const button = screen.getByRole('switch');
+    fireEvent.click(button);
+    
+    expect(mockHandler).not.toHaveBeenCalled();
+  });
+
+  it('applies disabled attribute when disabled', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={false} onCheckedChange={mockHandler} disabled={true} />);
+    
+    const button = screen.getByRole('switch');
+    expect(button).toBeDisabled();
+  });
+
+  it('applies custom className', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={false} onCheckedChange={mockHandler} className="custom-class" />);
+    
+    const button = screen.getByRole('switch');
+    expect(button).toHaveClass('custom-class');
+  });
+
+  it('applies data-testid when provided', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={false} onCheckedChange={mockHandler} data-testid="test-switch" />);
+    
+    const button = screen.getByTestId('test-switch');
+    expect(button).toBeInTheDocument();
+  });
+
+  it('has correct styles for checked state', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={true} onCheckedChange={mockHandler} />);
+    
+    const button = screen.getByRole('switch');
+    expect(button.className).toMatch(/bg-\[rgb\(var\(--primary\)\)\]/);
+  });
+
+  it('has correct styles for unchecked state', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={false} onCheckedChange={mockHandler} />);
+    
+    const button = screen.getByRole('switch');
+    expect(button.className).toMatch(/bg-gray-300/);
+  });
+
+  it('has disabled styling when disabled', () => {
+    const mockHandler = vi.fn();
+    render(<Switch checked={false} onCheckedChange={mockHandler} disabled={true} />);
+    
+    const button = screen.getByRole('switch');
+    expect(button.className).toMatch(/opacity-50/);
+  });
+
+  it('can be toggled multiple times', () => {
+    const mockHandler = vi.fn();
+    const { rerender } = render(<Switch checked={false} onCheckedChange={mockHandler} />);
+    
+    const button = screen.getByRole('switch');
+    
+    // First click - should call with true
+    fireEvent.click(button);
+    expect(mockHandler).toHaveBeenCalledWith(true);
+    expect(mockHandler).toHaveBeenCalledTimes(1);
+    
+    // Simulate parent updating the prop
+    rerender(<Switch checked={true} onCheckedChange={mockHandler} />);
+    
+    // Second click - should call with false
+    fireEvent.click(button);
+    expect(mockHandler).toHaveBeenCalledWith(false);
+    expect(mockHandler).toHaveBeenCalledTimes(2);
+    
+    // Simulate parent updating the prop again
+    rerender(<Switch checked={false} onCheckedChange={mockHandler} />);
+    
+    // Third click - should call with true again
+    fireEvent.click(button);
+    expect(mockHandler).toHaveBeenCalledWith(true);
+    expect(mockHandler).toHaveBeenCalledTimes(3);
+  });
+});

packages/ui/src/components/common/Switch.tsx

@@ -22,7 +22,6 @@ export function Switch({
 }: SwitchProps) {
   const handleClick = () => {
     if (!disabled) {
-      console.log('[Switch] Clicked, current:', checked, 'will become:', !checked);
       onCheckedChange(!checked);
     }
   };

packages/ui/vitest.config.ts

@@ -0,0 +1,19 @@
+import { defineConfig } from 'vitest/config';
+import react from '@vitejs/plugin-react';
+import path from 'path';
+
+export default defineConfig({
+  plugins: [react()],
+  test: {
+    environment: 'jsdom',
+    globals: true,
+    setupFiles: [path.resolve(__dirname, '../../tests/ts/setup.ts')],
+    include: ['src/**/*.test.{ts,tsx}'],
+    exclude: ['**/node_modules/**', 'dist/**'],
+  },
+  resolve: {
+    alias: {
+      '@': path.resolve(__dirname, './src'),
+    },
+  },
+});