refactor: add container test type and reorganize test structure

- Create new 't/container/' test type for Docker-based infrastructure tests
- Move SSH connection tests from 't/e2e/' to 't/container/'
- Move Docker fixtures from 't/e2e/fixtures/' to 't/container/fixtures/'
- Update test script to support new 'container' test type
- Update GitHub workflow to include container tests in CI
- Update Perl version to v5.38 for consistency across files
- Clean up e2e directory to only contain virtualization-based tests

Test hierarchy now: unit < integration < container < e2e
- unit: No external dependencies
- integration: Multiple components, no infrastructure
- container: Docker-based infrastructure testing (CI compatible)
- e2e: Full virtualization (local development only)

Author: josecelano

.github/copilot-instructions.md

@@ -10,8 +10,7 @@ Follow [perlstyle](https://perldoc.perl.org/5.42.0/perlstyle) conventions for co
 
 #### Core Requirements
 
-- **Always use strict and warnings**: `use strict; use warnings;` or `use v5.36;` (enables both)
-- **Modern Perl**: Use `use v5.40;` or higher for modern features (enables strict, warnings, and more)
+- **Modern Perl**: Use `use v5.38;` for modern features (enables strict, warnings, and more)
 
 #### Naming Conventions
 
@@ -144,7 +143,7 @@ Follow [perlstyle](https://perldoc.perl.org/5.42.0/perlstyle) conventions for co
 ALWAYS run the tests suite before committing any changes:
 
 ```bash
-./script/test unit integration
+./script/test unit integration container
 ```
 
 Note: E2E tests are excluded from pre-commit checks as they are slow and require virtualization. Run them manually when needed with `./script/test e2e` or `./script/test all`.

.github/workflows/testing.yml

@@ -83,4 +83,4 @@ jobs:
           # ./script/test e2e
           # or
           # ./script/test all
-          ./script/test unit integration
+          ./script/test unit integration container

README.md

@@ -165,17 +165,35 @@ The project includes two types of tests:
 Run unit tests that don't require virtualization:
 
 ```bash
-./script/test-unit
+./script/test unit
 ```
 
 These tests can run in CI environments and test individual components in isolation.
 
+### Integration Tests
+
+Run integration tests that require multiple components to work together:
+
+```bash
+./script/test integration
+```
+
+It does not require any special setup and can be run on any machine with the necessary dependencies installed.
+
+### Container Tests
+
+Run tests that require Docker:
+
+```bash
+./script/test container
+```
+
 ### E2E Tests
 
 Run end-to-end tests that require local virtualization support:
 
 ```bash
-./script/test-e2e
+./script/test e2e
 ```
 
 **Requirements for E2E tests:**

lib/TorrustDeploy/Infrastructure/SSH/Connection.pm

@@ -1,6 +1,6 @@
 package TorrustDeploy::Infrastructure::SSH::Connection;
 
-use v5.40;
+use v5.38;
 use Moo;
 use Net::SSH2;
 use Carp qw(croak);

script/test

@@ -1,29 +1,34 @@
 #!/bin/bash
 
 # Unified test script for running different test suites
-# Usage: ./script/test [e2e] [integration] [unit] [all]
+# Usage: ./script/test [e2e] [integration] [unit] [container] [all]
 # Note: E2E tests require virtualization (KVM/QEMU) to be available on the host
+# Note: Container tests require Docker to be available on the host
 # Examples:
 #   ./script/test all                    # Run all tests
 #   ./script/test unit integration       # Run unit and integration tests only
+#   ./script/test container              # Run container tests only (requires Docker)
 #   ./script/test e2e                    # Run e2e tests only (requires virtualization)
 
 set -e
 
 # Default to no tests selected
 run_unit=false
 run_integration=false
+run_container=false
 run_e2e=false
 
 # If no arguments provided, show usage
 if [ $# -eq 0 ]; then
-    echo "Usage: $0 [e2e] [integration] [unit] [all]"
+    echo "Usage: $0 [e2e] [integration] [unit] [container] [all]"
     echo ""
     echo "Note: E2E tests require virtualization (KVM/QEMU) to be available on the host"
+    echo "Note: Container tests require Docker to be available on the host"
     echo ""
     echo "Examples:"
     echo "  $0 all                    # Run all tests"
     echo "  $0 unit integration       # Run unit and integration tests only"
+    echo "  $0 container              # Run container tests only (requires Docker)"
     echo "  $0 e2e                    # Run e2e tests only (requires virtualization)"
     exit 1
 fi
@@ -34,6 +39,7 @@ for arg in "$@"; do
         all)
             run_unit=true
             run_integration=true
+            run_container=true
             run_e2e=true
             break  # If 'all' is specified, no need to check other args
             ;;
@@ -43,12 +49,15 @@ for arg in "$@"; do
         integration)
             run_integration=true
             ;;
+        container)
+            run_container=true
+            ;;
         e2e)
             run_e2e=true
             ;;
         *)
             echo "Error: Unknown argument '$arg'"
-            echo "Valid arguments are: e2e, integration, unit, all"
+            echo "Valid arguments are: e2e, integration, unit, container, all"
             exit 1
             ;;
     esac
@@ -65,6 +74,10 @@ if [ "$run_integration" = true ]; then
     test_dirs+=("t/integration/")
 fi
 
+if [ "$run_container" = true ]; then
+    test_dirs+=("t/container/")
+fi
+
 if [ "$run_e2e" = true ]; then
     test_dirs+=("t/e2e/")
 fi

t/e2e/fixtures/Dockerfile.ssh-server renamed to t/container/fixtures/Dockerfile.ssh-server

@@ -1,6 +1,6 @@
 #!/usr/bin/env perl
 
-use v5.40;
+use v5.38;
 use Test2::V0;
 use File::Spec;
 use Cwd qw(getcwd);
@@ -13,7 +13,7 @@ my $SSH_HOST = 'localhost';
 my $SSH_PORT = 2222;
 my $SSH_USER = 'testuser';
 my $SSH_PASS = 'testpass123';
-my $TEST_KEY_PATH = File::Spec->catfile(getcwd(), 't', 'e2e', 'fixtures', 'test_key');
+my $TEST_KEY_PATH = File::Spec->catfile(getcwd(), 't', 'container', 'fixtures', 'test_key');
 
 # Skip tests if Docker is not available
 sub docker_available {
@@ -29,7 +29,7 @@ plan skip_all => 'Docker Compose not available' unless docker_compose_available(
 
 # Test environment setup and teardown
 sub start_ssh_server {
-    my $fixtures_dir = File::Spec->catdir(getcwd(), 't', 'e2e', 'fixtures');
+    my $fixtures_dir = File::Spec->catdir(getcwd(), 't', 'container', 'fixtures');
 
     # Stop any existing container
     system("cd '$fixtures_dir' && docker compose down >/dev/null 2>&1");
@@ -61,7 +61,7 @@ sub start_ssh_server {
 }
 
 sub stop_ssh_server {
-    my $fixtures_dir = File::Spec->catdir(getcwd(), 't', 'e2e', 'fixtures');
+    my $fixtures_dir = File::Spec->catdir(getcwd(), 't', 'container', 'fixtures');
     system("cd '$fixtures_dir' && docker compose down >/dev/null 2>&1");
 }
 
@@ -71,7 +71,7 @@ start_ssh_server();
 # Ensure cleanup happens even if tests fail
 END { stop_ssh_server(); }
 
-subtest 'E2E SSH Connection Tests' => sub {
+subtest 'Container SSH Connection Tests' => sub {
     subtest 'Connection and Authentication' => sub {
         my $ssh = TorrustDeploy::Infrastructure::SSH::Connection->new(
             host => "$SSH_HOST:$SSH_PORT",