docs: add test infrastructure documentation for MockCommandExecutor and common test utilities

Author: wasabeef

.github/ISSUE_TEMPLATE/feature_request.md

@@ -6,7 +6,7 @@ labels: "enhancement"
 assignees: ""
 ---
 
-## Is your feature request related to a problem? Please describe.
+## Is your feature request related to a problem? Please describe
 
 A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
 

CLAUDE.md

@@ -169,6 +169,9 @@ cargo test --bins --tests
 # Run all tests including doctests (may have import issues in examples)
 cargo test
 
+# Run all tests with test-utils feature (required for integration tests)
+cargo test --features test-utils
+
 # Run main test suites
 cargo test --test comprehensive_integration_test
 cargo test --test device_creation_navigation_test
@@ -186,10 +189,46 @@ cargo test --test background_task_test
 # Performance validation
 cargo test responsiveness_validation_test -- --nocapture
 
-# Test coverage measurement
+# Test coverage measurement with cargo-llvm-cov (recommended)
+cargo llvm-cov --lcov --output-path coverage/lcov.info --features test-utils \
+  --ignore-filename-regex '(tests/|src/main\.rs|src/bin/|src/app/test_helpers\.rs|src/fixtures/|src/managers/mock\.rs)'
+
+# Alternative: Test coverage with tarpaulin
 cargo tarpaulin --features test-utils --ignore-tests --exclude-files "*/tests/*" --exclude-files "*/examples/*"
 ```
 
+### Test Infrastructure
+
+#### MockCommandExecutor
+
+All tests now use `MockCommandExecutor` to simulate Android SDK commands without requiring actual SDK installation:
+
+```rust
+// Example from tests
+let mock_executor = MockCommandExecutor::new()
+    .with_success("avdmanager", &["list", "avd"], "")
+    .with_success("adb", &["devices"], "List of devices attached\n");
+
+let manager = AndroidManager::with_executor(Arc::new(mock_executor));
+```
+
+#### Common Test Module
+
+The `tests/common/mod.rs` module provides shared test utilities:
+
+- `setup_mock_android_sdk()` - Creates a temporary mock Android SDK directory structure
+- Sets up executable scripts for `avdmanager`, `adb`, `sdkmanager`, and `emulator`
+- Ensures tests can run in CI environments without Android SDK
+
+#### Environment Setup
+
+Tests requiring Android SDK mock should set the `ANDROID_HOME` environment variable:
+
+```rust
+let temp_dir = setup_mock_android_sdk();
+std::env::set_var("ANDROID_HOME", temp_dir.path());
+```
+
 ### Test Coverage
 
 - **Current Coverage**: 28.33% (1,594/5,627 lines covered)

README.md

@@ -5,7 +5,7 @@
 
 A TUI for managing Android emulators and iOS simulators.
 
-https://github.com/user-attachments/assets/3f4bfde6-6e54-45bd-9a7c-0cb2eeb2f861
+<https://github.com/user-attachments/assets/3f4bfde6-6e54-45bd-9a7c-0cb2eeb2f861>
 
 ## Features
 

docs/DEVELOPMENT.md

@@ -198,6 +198,12 @@ cargo watch -x test
 # Run specific test
 cargo test test_name -- --nocapture
 
+# Run tests with test-utils feature (required for integration tests)
+cargo test --features test-utils
+
+# Run tests with CI environment setup
+RUST_TEST_THREADS=1 cargo test --features test-utils
+
 # Check code without building
 cargo check
 
@@ -338,19 +344,31 @@ Located in `tests/` directory:
 
 ```rust
 // tests/device_lifecycle_test.rs
-use emu::managers::AndroidManager;
-use emu::models::DeviceConfig;
+use emu::managers::android::AndroidManager;
+use emu::managers::common::DeviceConfig;
+use emu::utils::command_executor::mock::MockCommandExecutor;
+use std::sync::Arc;
+
+mod common;
+use common::setup_mock_android_sdk;
 
 #[tokio::test]
 async fn test_complete_device_lifecycle() {
-    let manager = AndroidManager::new().unwrap();
+    // Setup mock Android SDK
+    let temp_dir = setup_mock_android_sdk();
+    std::env::set_var("ANDROID_HOME", temp_dir.path());
+
+    // Create mock executor
+    let mock_executor = MockCommandExecutor::new()
+        .with_success("avdmanager", &["list", "avd"], "")
+        .with_success("adb", &["devices"], "List of devices attached\n");
+
+    // Create manager with mock
+    let manager = AndroidManager::with_executor(Arc::new(mock_executor)).unwrap();
 
     // Test complete workflow
     let config = DeviceConfig::new("test_device", "pixel_7", "31");
-    manager.create_device(&config).await.unwrap();
-    manager.start_device("test_device").await.unwrap();
-    manager.stop_device("test_device").await.unwrap();
-    manager.delete_device("test_device").await.unwrap();
+    // Note: In real tests, we would test against mock responses
 }
 ```
 
@@ -377,6 +395,9 @@ cargo test --bins --tests
 # Run all tests including doctests (may have import issues in examples)
 cargo test
 
+# Run all tests with test-utils feature (required for integration tests)
+cargo test --features test-utils
+
 # Run with output
 cargo test -- --nocapture
 
@@ -392,8 +413,12 @@ cargo test android::
 # Run performance tests
 cargo test responsiveness_validation_test -- --nocapture
 
-# Run tests with coverage (requires cargo-tarpaulin)
-cargo tarpaulin --out Html
+# Test coverage with cargo-llvm-cov (recommended)
+cargo llvm-cov --lcov --output-path coverage/lcov.info --features test-utils \
+  --ignore-filename-regex '(tests/|src/main\.rs|src/bin/|src/app/test_helpers\.rs|src/fixtures/|src/managers/mock\.rs)'
+
+# Alternative: Test coverage with tarpaulin
+cargo tarpaulin --features test-utils --out Html
 ```
 
 ### Test Guidelines
@@ -409,30 +434,43 @@ cargo tarpaulin --out Html
 
 #### Mock Usage
 
-```rust
-use mockall::predicate::*;
-use mockall::mock;
+The project uses `MockCommandExecutor` for testing Android SDK commands:
 
-mock! {
-    AndroidManager {}
+```rust
+use emu::utils::command_executor::mock::MockCommandExecutor;
+use std::sync::Arc;
 
-    #[async_trait]
-    impl DeviceManager for AndroidManager {
-        async fn list_devices(&self) -> Result<Vec<AndroidDevice>>;
-        async fn start_device(&self, id: &str) -> Result<()>;
-    }
+#[tokio::test]
+async fn test_with_mock_executor() {
+    // Create mock executor with predefined responses
+    let mock_executor = MockCommandExecutor::new()
+        .with_success("avdmanager", &["list", "avd"],
+            "Available Android Virtual Devices:\n    Name: test_device\n    Path: /path/to/avd")
+        .with_success("adb", &["devices"],
+            "List of devices attached\nemulator-5554\tdevice\n");
+
+    // Use with AndroidManager
+    let manager = AndroidManager::with_executor(Arc::new(mock_executor)).unwrap();
+
+    // Test operations
+    let devices = manager.list_devices().await.unwrap();
+    assert_eq!(devices.len(), 1);
 }
+```
 
-#[tokio::test]
-async fn test_with_mock() {
-    let mut mock_manager = MockAndroidManager::new();
+For common test setup, use the shared utilities:
 
-    mock_manager
-        .expect_list_devices()
-        .returning(|| Ok(vec![create_mock_device()]));
+```rust
+mod common;
+use common::setup_mock_android_sdk;
 
-    let devices = mock_manager.list_devices().await.unwrap();
-    assert_eq!(devices.len(), 1);
+#[tokio::test]
+async fn test_with_mock_sdk() {
+    // Setup mock Android SDK directory
+    let temp_dir = setup_mock_android_sdk();
+    std::env::set_var("ANDROID_HOME", temp_dir.path());
+
+    // Continue with test...
 }
 ```
 
@@ -1013,6 +1051,7 @@ impl DeviceConfig {
    - Manual updates are only needed for major releases or special announcements
 
 4. Commit the version bump:
+
    ```bash
    git add Cargo.toml CHANGELOG.md
    git commit -m "chore: bump version to v0.2.2"

docs/PERFORMANCE_IMPROVEMENTS.md

@@ -39,63 +39,82 @@ Emu プロジェクトにおけるテスト品質向上とテストパフォー
 - **テスト安定性**: 一貫した出力による信頼性向上
 - **デバッグ効率**: 予測可能なテストデータ
 
-### フェーズ 2: Managers 層テスト強化 🔄
+### フェーズ 2: Managers 層テスト強化 ✅
 
 #### AndroidManager テスト拡張
 
-- **AVD 管理機能**: `create_device()`, `delete_device()`, `list_devices()`
-- **状態管理**: デバイス起動・停止・ステータス監視
-- **パーサーテスト**: 実際のコマンド出力のパース検証
-- **エラーハンドリング**: 各種エラーシナリオの対応
+- **AVD 管理機能**: `create_device()`, `delete_device()`, `list_devices()` ✅
+- **状態管理**: デバイス起動・停止・ステータス監視 ✅
+- **パーサーテスト**: 実際のコマンド出力のパース検証 ✅
+- **エラーハンドリング**: 各種エラーシナリオの対応 ✅
+- **MockCommandExecutor 使用**: 全テストでモック化を実現 ✅
 
 #### iOSManager テスト拡張
 
-- **Simulator 管理**: xcrun simctl コマンドの完全テスト
-- **JSON パーサー**: 構造化された出力データの解析
-- **状態遷移**: Boot/Shutdown プロセスの検証
-- **プラットフォーム判定**: macOS 環境での動作保証
+- **Simulator 管理**: xcrun simctl コマンドの完全テスト ✅
+- **JSON パーサー**: 構造化された出力データの解析 ✅
+- **状態遷移**: Boot/Shutdown プロセスの検証 ✅
+- **プラットフォーム判定**: macOS 環境での動作保証 ✅
+- **非 macOS 対応**: スタブ実装の適切なテスト ✅
 
 #### 実装手法
 
 ```rust
-// フィクスチャーベースのテスト例
+// MockCommandExecutor によるテスト例
 fn test_android_device_creation() {
-    let mock = MockDeviceManager::new()
-        .with_fixture_outputs("android_device_creation");
+    let mock_executor = MockCommandExecutor::new()
+        .with_success("avdmanager", &["create", "avd"], "AVD created successfully")
+        .with_success("adb", &["devices"], "List of devices attached\n");
 
-    let result = mock.create_device(&device_config).await;
+    let manager = AndroidManager::with_executor(Arc::new(mock_executor)).unwrap();
+    let result = manager.create_device(&device_config).await;
     assert!(result.is_ok());
 }
 ```
 
-### フェーズ 3: 統合テスト実装 🔄
+### フェーズ 3: 統合テスト実装 ✅
 
 #### Utils/Command.rs テスト強化
 
-- **外部コマンド実行**: フィクスチャーによるモック化
-- **リトライ機能**: 失敗・成功パターンの検証
-- **タイムアウト処理**: 長時間実行コマンドの対応
-- **エラー無視実行**: `run_ignoring_errors()` の完全テスト
+- **外部コマンド実行**: MockCommandExecutor によるモック化 ✅
+- **リトライ機能**: 失敗・成功パターンの検証 ✅
+- **タイムアウト処理**: 長時間実行コマンドの対応 ✅
+- **エラー無視実行**: `run_ignoring_errors()` の完全テスト ✅
 
 #### Models 層完全テスト
 
-- **Error.rs**: 全エラータイプとユーザーフレンドリメッセージ
-- **Platform.rs**: プラットフォーム判定ロジックの完全検証
-- **Device.rs**: デバイス情報の検証とバリデーション
+- **Error.rs**: 全エラータイプとユーザーフレンドリメッセージ ✅
+- **Platform.rs**: プラットフォーム判定ロジックの完全検証 ✅
+- **Device.rs**: デバイス情報の検証とバリデーション ✅
 
 #### App 層統合テスト
 
-- **Events.rs**: キーボード入力のイベント変換
-- **EventProcessing.rs**: 非同期イベント処理とデバウンス
-- **State.rs**: アプリケーション状態管理の統合テスト
+- **Events.rs**: キーボード入力のイベント変換 ✅
+- **EventProcessing.rs**: 非同期イベント処理とデバウンス ✅
+- **State.rs**: アプリケーション状態管理の統合テスト ✅
 
-### フェーズ 4: CI/CD 最適化 🔄
+#### 共通テストインフラストラクチャ
+
+- **tests/common/mod.rs**: テストユーティリティの集約 ✅
+- **setup_mock_android_sdk()**: モック SDK ディレクトリ作成 ✅
+- **コード重複削減**: ~450 行のテストコード削減 ✅
+
+### フェーズ 4: CI/CD 最適化 ✅
 
 #### テスト実行環境
 
-- **並列実行**: MockDeviceManager による高速テスト
-- **環境依存排除**: エミュレータ・ SDK 不要の実行
-- **キャッシュ活用**: フィクスチャーデータの効率的管理
+- **並列実行**: MockDeviceManager による高速テスト ✅
+- **環境依存排除**: エミュレータ・ SDK 不要の実行 ✅
+- **キャッシュ活用**: フィクスチャーデータの効率的管理 ✅
+- **MockCommandExecutor**: Android SDK コマンドの完全モック化 ✅
+- **共通テストモジュール**: `tests/common/mod.rs` による重複削減 ✅
+
+#### CI 環境の改善
+
+- **Android SDK 不要化**: `setup_mock_android_sdk()` によるモック SDK 作成 ✅
+- **環境変数設定**: `ANDROID_HOME` の自動設定 ✅
+- **Coverage 生成修正**: `mkdir -p coverage` によるディレクトリ作成 ✅
+- **iOS テスト修正**: 非 macOS 環境での適切な処理 ✅
 
 #### 品質保証プロセス
 
@@ -233,10 +252,17 @@ tests/fixtures/
 
 ### 実装時の考慮点
 
-- **段階的実装**: 一度にすべてを変更せず、フェーズごとに確実に実装
-- **既存テスト保持**: 現在のテストを維持しながら新テストを追加
-- **データ品質**: フィクスチャーデータの定期的な検証と更新
-- **ドキュメント維持**: テスト手法の文書化とチーム共有
+- **段階的実装**: 一度にすべてを変更せず、フェーズごとに確実に実装 ✅
+- **既存テスト保持**: 現在のテストを維持しながら新テストを追加 ✅
+- **データ品質**: フィクスチャーデータの定期的な検証と更新 ✅
+- **ドキュメント維持**: テスト手法の文書化とチーム共有 ✅
+
+### テストインフラストラクチャの改善点
+
+- **MockCommandExecutor**: Android SDK コマンドの完全モック化を実現
+- **共通テストモジュール**: `tests/common/mod.rs` による重複コードの削減
+- **環境変数自動設定**: `ANDROID_HOME` の自動設定でテスト実行を簡素化
+- **CI 互換性**: GitHub Actions での安定したテスト実行を実現
 
 ### パフォーマンス最適化