refactor: extract SSH key setup method in TestEnvironment

Extract SSH key copying and permission setting logic into a dedicated
setup_ssh_key method to improve code organization and maintainability.

- Add setup_ssh_key method with proper error handling
- Remove inline SSH key setup from new method
- Consolidate verbose logging within the extracted method
- Maintain exact same functionality while improving structure

Author: josecelano

src/bin/e2e_tests.rs

@@ -50,21 +50,9 @@ impl TestEnvironment {
         // Create temporary directory for SSH keys
         let temp_dir = TempDir::new().context("Failed to create temporary directory")?;
 
-        // Copy SSH private key from fixtures to temp directory
-        let fixtures_ssh_key = project_root.join("fixtures/testing_rsa");
+        // Setup SSH key
         let temp_ssh_key = temp_dir.path().join("testing_rsa");
-
-        std::fs::copy(&fixtures_ssh_key, &temp_ssh_key)
-            .context("Failed to copy SSH private key to temporary directory")?;
-
-        // Set proper permissions on the SSH key (600)
-        #[cfg(unix)]
-        {
-            use std::os::unix::fs::PermissionsExt;
-            let mut perms = std::fs::metadata(&temp_ssh_key)?.permissions();
-            perms.set_mode(0o600);
-            std::fs::set_permissions(&temp_ssh_key, perms)?;
-        }
+        Self::setup_ssh_key(&project_root, &temp_dir, verbose, &temp_ssh_key)?;
 
         // Create configuration
         let config = Config::new(
@@ -86,10 +74,6 @@ impl TestEnvironment {
         Self::clean_and_prepare_templates(&services, verbose)?;
 
         if verbose {
-            println!(
-                "🔑 SSH key copied to temporary location: {}",
-                config.ssh_key_path.display()
-            );
             println!("📁 Temporary directory: {}", temp_dir.path().display());
             println!(
                 "📄 Templates directory: {}",
@@ -104,6 +88,39 @@ impl TestEnvironment {
         })
     }
 
+    /// Setup SSH key by copying from fixtures to temporary directory with proper permissions
+    fn setup_ssh_key(
+        project_root: &std::path::Path,
+        temp_dir: &TempDir,
+        verbose: bool,
+        ssh_key_path: &std::path::Path,
+    ) -> Result<()> {
+        // Copy SSH private key from fixtures to temp directory
+        let fixtures_ssh_key = project_root.join("fixtures/testing_rsa");
+        let temp_ssh_key = temp_dir.path().join("testing_rsa");
+
+        std::fs::copy(&fixtures_ssh_key, &temp_ssh_key)
+            .context("Failed to copy SSH private key to temporary directory")?;
+
+        // Set proper permissions on the SSH key (600)
+        #[cfg(unix)]
+        {
+            use std::os::unix::fs::PermissionsExt;
+            let mut perms = std::fs::metadata(&temp_ssh_key)?.permissions();
+            perms.set_mode(0o600);
+            std::fs::set_permissions(&temp_ssh_key, perms)?;
+        }
+
+        if verbose {
+            println!(
+                "🔑 SSH key copied to temporary location: {}",
+                ssh_key_path.display()
+            );
+        }
+
+        Ok(())
+    }
+
     /// Clean and prepare templates directory to ensure fresh embedded templates
     fn clean_and_prepare_templates(services: &Services, verbose: bool) -> Result<()> {
         // Clean templates directory to ensure we use fresh templates from embedded resources