feat: update E2E binaries to use Environment entities

- Replace hardcoded instance names with Environment entity auto-generation
- Change CLI from --templates-dir to --environment parameter
- Update all three E2E binaries (config, provision, full) with environment-specific configuration
- Add environment-specific default names for isolation:
  - e2e-config for config tests
  - e2e-provision for provision tests
  - e2e-full for full tests
- Fix missing panic documentation in e2e-provision-tests
- All tests passing with new environment-based infrastructure

This enables multiple concurrent test environments without conflicts.

Author: josecelano

docs/issues/multi-environment-support.md

@@ -128,19 +128,32 @@ pub mod environment_name;
 // ... existing modules
 ```
 
-### Step 4: Update E2E Binaries
+### Step 4: Update E2E Binaries ✅ COMPLETED
 
 **Files to Modify**:
 
-- `src/bin/e2e_provision_tests.rs`
-- `src/bin/e2e_config_tests.rs`
-- `src/bin/e2e_tests_full.rs`
+- ✅ `src/bin/e2e_provision_tests.rs`
+- ✅ `src/bin/e2e_config_tests.rs`
+- ✅ `src/bin/e2e_tests_full.rs`
 
 **Requirements**:
 
-- Create Environment instances in each binary
-- Use environment-specific configurations
-- Maintain existing CLI argument compatibility where possible
+- ✅ Create Environment instances in each binary
+- ✅ Use environment-specific configurations
+- ✅ Replace hardcoded instance names with Environment entity
+- ✅ Update CLI arguments from `--templates-dir` to `--environment`
+- ✅ Update help text and documentation
+
+**Implementation Summary**:
+
+Each binary now:
+
+- Accepts `--environment` parameter instead of `--templates-dir`
+- Creates Environment entity with auto-generated paths and instance names
+- Uses different default environment names for isolation:
+  - `e2e-config` for config tests
+  - `e2e-provision` for provision tests
+  - `e2e-full` for full tests
 
 **Example for e2e_config_tests.rs**:
 

src/bin/e2e_config_tests.rs

@@ -16,10 +16,10 @@
 //! Run with custom options:
 //!
 //! ```bash
-//! # Use custom templates directory
-//! cargo run --bin e2e-config-tests -- --templates-dir ./custom/templates
+//! # Use specific environment name
+//! cargo run --bin e2e-config-tests -- --environment e2e-staging
 //!
-//! # Change logging format
+//! # Change logging format  
 //! cargo run --bin e2e-config-tests -- --log-format json
 //!
 //! # Show help
@@ -50,7 +50,7 @@ use std::time::Instant;
 use torrust_tracker_deploy::e2e::tasks::run_configure_command::run_configure_command;
 use tracing::{error, info};
 
-use torrust_tracker_deploy::config::InstanceName;
+use torrust_tracker_deploy::domain::{Environment, EnvironmentName};
 use torrust_tracker_deploy::e2e::context::{TestContext, TestContextType};
 use torrust_tracker_deploy::e2e::tasks::{
     container::{
@@ -67,9 +67,9 @@ use torrust_tracker_deploy::shared::Username;
 #[command(name = "e2e-config-tests")]
 #[command(about = "E2E configuration tests for Torrust Tracker Deploy using Docker containers")]
 struct CliArgs {
-    /// Templates directory path (default: ./data/templates)
-    #[arg(long, default_value = "./data/templates")]
-    templates_dir: String,
+    /// Environment name for deployment testing (e.g., "e2e-config", "staging"). This determines the instance name and directory structure.
+    #[arg(long, default_value = "e2e-config")]
+    environment: String,
 
     /// Logging format to use
     #[arg(
@@ -106,25 +106,35 @@ pub async fn main() -> Result<()> {
     info!(
         application = "torrust_tracker_deploy",
         test_suite = "e2e_config_tests",
+        environment = %cli.environment,
         log_format = ?cli.log_format,
         "Starting E2E configuration tests with Docker containers"
     );
 
     let test_start = Instant::now();
 
-    let instance_name =
-        InstanceName::new("torrust-tracker-vm".to_string()).expect("Valid hardcoded instance name");
-
-    let ssh_user = Username::new("torrust").expect("Valid hardcoded username");
+    // Create Environment entity from CLI argument
+    let env_name = EnvironmentName::new(&cli.environment)
+        .map_err(|e| anyhow::anyhow!("Invalid environment name '{}': {}", cli.environment, e))?;
 
     let ssh_private_key_path = std::path::PathBuf::from("fixtures/testing_rsa");
     let ssh_public_key_path = std::path::PathBuf::from("fixtures/testing_rsa.pub");
 
+    let environment = Environment::new(
+        env_name,
+        ssh_private_key_path.clone(),
+        ssh_public_key_path.clone(),
+    );
+
+    let ssh_user = Username::new("torrust").expect("Valid hardcoded username");
+
+    // Extract values from Environment entity for TestContext
+    // TODO: Update TestContext to accept Environment entity directly
     let test_context = TestContext::initialized(
         false,
-        cli.templates_dir,
+        environment.templates_dir(),
         &ssh_user,
-        instance_name,
+        environment.instance_name.clone(),
         ssh_private_key_path,
         ssh_public_key_path,
         TestContextType::Container,

src/bin/e2e_provision_tests.rs

@@ -5,6 +5,30 @@
 //! is properly provisioned and ready for configuration, but does NOT attempt
 //! to configure or install software.
 //!
+//! ## Usage
+//!
+//! Run the E2E provisioning tests:
+//!
+//! ```bash
+//! cargo run --bin e2e-provision-tests
+//! ```
+//!
+//! Run with custom options:
+//!
+//! ```bash
+//! # Use specific environment name
+//! cargo run --bin e2e-provision-tests -- --environment e2e-staging
+//!
+//! # Keep test environment after completion (for debugging)
+//! cargo run --bin e2e-provision-tests -- --keep
+//!
+//! # Change logging format
+//! cargo run --bin e2e-provision-tests -- --log-format json
+//!
+//! # Show help
+//! cargo run --bin e2e-provision-tests -- --help
+//! ```
+//!
 //! ## Test Workflow
 //!
 //! 1. **Preflight cleanup** - Remove any lingering test resources
@@ -22,7 +46,7 @@ use std::time::Instant;
 use tracing::{error, info};
 
 // Import E2E testing infrastructure
-use torrust_tracker_deploy::config::InstanceName;
+use torrust_tracker_deploy::domain::{Environment, EnvironmentName};
 use torrust_tracker_deploy::e2e::context::{TestContext, TestContextType};
 use torrust_tracker_deploy::e2e::tasks::{
     preflight_cleanup::cleanup_lingering_resources,
@@ -42,9 +66,9 @@ struct Cli {
     #[arg(long)]
     keep: bool,
 
-    /// Templates directory path (default: ./data/templates)
-    #[arg(long, default_value = "./data/templates")]
-    templates_dir: String,
+    /// Environment name for deployment testing (e.g., "e2e-provision", "staging"). This determines the instance name and directory structure.
+    #[arg(long, default_value = "e2e-provision")]
+    environment: String,
 
     /// Logging format to use
     #[arg(
@@ -69,18 +93,16 @@ struct Cli {
 /// # Errors
 ///
 /// This function may return errors in the following cases:
+/// - Invalid environment name provided via CLI
 /// - Test environment setup fails
 /// - Pre-flight cleanup encounters issues
 /// - Infrastructure provisioning fails
 /// - Cleanup operations fail
 ///
 /// # Panics
 ///
-/// May panic if the hardcoded instance name "torrust-tracker-vm" is invalid,
+/// This function may panic if the hardcoded username "torrust" is invalid,
 /// which should never happen in normal operation.
-///
-/// May panic during the match statement if unexpected error combinations occur
-/// that are not handled by the current error handling logic.
 #[tokio::main]
 pub async fn main() -> Result<()> {
     let cli = Cli::parse();
@@ -95,20 +117,23 @@ pub async fn main() -> Result<()> {
         "Starting E2E provisioning tests"
     );
 
-    // Instance name for the test environment - not user configurable for now
-    let instance_name =
-        InstanceName::new("torrust-tracker-vm".to_string()).expect("Valid hardcoded instance name");
-
-    let ssh_user = Username::new("torrust").expect("Valid hardcoded username");
-
+    // Create environment entity from CLI input
+    let environment_name = EnvironmentName::new(cli.environment)?;
     let ssh_private_key_path = std::path::PathBuf::from("fixtures/testing_rsa");
     let ssh_public_key_path = std::path::PathBuf::from("fixtures/testing_rsa.pub");
+    let environment = Environment::new(
+        environment_name,
+        ssh_private_key_path.clone(),
+        ssh_public_key_path.clone(),
+    );
+
+    let ssh_user = Username::new("torrust").expect("Valid hardcoded username");
 
     let test_context = TestContext::initialized(
         cli.keep,
-        cli.templates_dir,
+        environment.data_dir.to_string_lossy().to_string(),
         &ssh_user,
-        instance_name,
+        environment.instance_name.clone(),
         ssh_private_key_path,
         ssh_public_key_path,
         TestContextType::VirtualMachine,

src/bin/e2e_tests_full.rs

@@ -9,6 +9,30 @@
 //! - `cargo run --bin e2e-provision-tests` - Infrastructure provisioning only
 //! - `cargo run --bin e2e-config-tests` - Configuration and software installation
 //!
+//! ## Usage
+//!
+//! Run the full E2E test suite:
+//!
+//! ```bash
+//! cargo run --bin e2e-tests-full
+//! ```
+//!
+//! Run with custom options:
+//!
+//! ```bash
+//! # Use specific environment name
+//! cargo run --bin e2e-tests-full -- --environment e2e-staging
+//!
+//! # Keep test environment after completion (for debugging)
+//! cargo run --bin e2e-tests-full -- --keep
+//!
+//! # Change logging format
+//! cargo run --bin e2e-tests-full -- --log-format json
+//!
+//! # Show help
+//! cargo run --bin e2e-tests-full -- --help
+//! ```
+//!
 //! ## Test Workflow
 //!
 //! 1. **Preflight cleanup** - Remove any lingering test resources
@@ -27,7 +51,7 @@ use std::time::Instant;
 use tracing::{error, info};
 
 // Import E2E testing infrastructure
-use torrust_tracker_deploy::config::InstanceName;
+use torrust_tracker_deploy::domain::{Environment, EnvironmentName};
 use torrust_tracker_deploy::e2e::context::{TestContext, TestContextType};
 use torrust_tracker_deploy::e2e::tasks::{
     preflight_cleanup::cleanup_lingering_resources,
@@ -49,9 +73,9 @@ struct Cli {
     #[arg(long)]
     keep: bool,
 
-    /// Templates directory path (default: ./data/templates)
-    #[arg(long, default_value = "./data/templates")]
-    templates_dir: String,
+    /// Environment name for deployment testing (e.g., "e2e-full", "staging"). This determines the instance name and directory structure.
+    #[arg(long, default_value = "e2e-full")]
+    environment: String,
 
     /// Logging format to use
     #[arg(
@@ -70,6 +94,7 @@ struct Cli {
 /// # Errors
 ///
 /// Returns an error if:
+/// - Invalid environment name provided via CLI
 /// - Pre-flight cleanup fails
 /// - Infrastructure provisioning fails  
 /// - Service configuration fails
@@ -94,20 +119,23 @@ pub async fn main() -> Result<()> {
         "Starting E2E tests"
     );
 
-    // Instance name for the test environment - not user configurable for now
-    let instance_name =
-        InstanceName::new("torrust-tracker-vm".to_string()).expect("Valid hardcoded instance name");
-
-    let ssh_user = Username::new("torrust").expect("Valid hardcoded username");
-
+    // Create environment entity from CLI input
+    let environment_name = EnvironmentName::new(cli.environment)?;
     let ssh_private_key_path = std::path::PathBuf::from("fixtures/testing_rsa");
     let ssh_public_key_path = std::path::PathBuf::from("fixtures/testing_rsa.pub");
+    let environment = Environment::new(
+        environment_name,
+        ssh_private_key_path.clone(),
+        ssh_public_key_path.clone(),
+    );
+
+    let ssh_user = Username::new("torrust").expect("Valid hardcoded username");
 
     let test_context = TestContext::initialized(
         cli.keep,
-        cli.templates_dir,
+        environment.data_dir.to_string_lossy().to_string(),
         &ssh_user,
-        instance_name,
+        environment.instance_name.clone(),
         ssh_private_key_path,
         ssh_public_key_path,
         TestContextType::VirtualMachine,