refactor: inline setup_ssh_key function into environment module

The setup_ssh_key function was only used in one place (src/e2e/environment.rs),
so it has been inlined into the setup_ssh_credentials method for simplicity.

Changes:
- Inline setup_ssh_key logic into setup_ssh_credentials method
- Remove unused import and add required anyhow::Context import
- Delete src/e2e/tasks/setup_ssh_key.rs module file
- Update src/e2e/tasks/mod.rs to remove module reference
- Maintain all original functionality including error handling and logging

All tests and linters pass after refactoring.

Author: josecelano

src/e2e/environment.rs

@@ -20,14 +20,13 @@
 //! The environment ensures each test runs in isolation with its own
 //! temporary resources and configuration.
 
+use anyhow::Context;
 use tempfile::TempDir;
 use tracing::{info, warn};
 
 use crate::config::{Config, InstanceName, SshCredentials};
 use crate::container::Services;
 
-use super::tasks::setup_ssh_key::setup_ssh_key;
-
 /// Errors that can occur during test environment creation and initialization
 #[derive(Debug, thiserror::Error)]
 pub enum TestEnvironmentError {
@@ -288,9 +287,41 @@ impl TestEnvironment {
         let temp_ssh_key = temp_dir.path().join(SSH_PRIVATE_KEY_FILENAME);
         let temp_ssh_pub_key = temp_dir.path().join(SSH_PUBLIC_KEY_FILENAME);
 
-        setup_ssh_key(project_root, temp_dir)
+        // Copy SSH private key from fixtures to temp directory
+        let fixtures_ssh_key = project_root.join("fixtures/testing_rsa");
+
+        std::fs::copy(&fixtures_ssh_key, &temp_ssh_key)
+            .context("Failed to copy SSH private key to temporary directory")
+            .map_err(|e| TestEnvironmentError::SshKeySetupError { source: e })?;
+
+        // Copy SSH public key from fixtures to temp directory
+        let fixtures_ssh_pub_key = project_root.join("fixtures/testing_rsa.pub");
+
+        std::fs::copy(&fixtures_ssh_pub_key, &temp_ssh_pub_key)
+            .context("Failed to copy SSH public key to temporary directory")
             .map_err(|e| TestEnvironmentError::SshKeySetupError { source: e })?;
 
+        // 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)
+                .context("Failed to get SSH key metadata")
+                .map_err(|e| TestEnvironmentError::SshKeySetupError { source: e })?
+                .permissions();
+            perms.set_mode(0o600);
+            std::fs::set_permissions(&temp_ssh_key, perms)
+                .context("Failed to set SSH key permissions")
+                .map_err(|e| TestEnvironmentError::SshKeySetupError { source: e })?;
+        }
+
+        info!(
+            operation = "ssh_key_setup",
+            private_location = %temp_ssh_key.display(),
+            public_location = %temp_ssh_pub_key.display(),
+            "SSH keys copied to temporary location"
+        );
+
         Ok(SshCredentials::new(
             temp_ssh_key,
             temp_ssh_pub_key,

src/e2e/tasks/mod.rs

@@ -11,7 +11,6 @@
 //! ### Infrastructure-agnostic tasks (can be used with both containers and VMs):
 //! - `run_configure_command` - Infrastructure configuration via Ansible and playbook execution
 //! - `run_deployment_validation` - Deployment validation and testing
-//! - `setup_ssh_key` - SSH key generation and setup
 //! - `run_test_command` - Deployment validation and testing
 //!
 //! ### Container-specific tasks (`container` submodule):
@@ -32,5 +31,4 @@ pub mod preflight_cleanup;
 pub mod run_configure_command;
 pub mod run_deployment_validation;
 pub mod run_test_command;
-pub mod setup_ssh_key;
 pub mod virtual_machine;