refactor: reorganize test structure into unit/integration/e2e directories

- Move unit tests to t/unit/ directory (app.t, command/provision.t, provision/opentofu.t)
- Move integration tests to t/integration/ directory (app.t)
- Keep e2e tests in t/e2e/ directory (provision.t, renamed from 001-provision.t)
- Create dedicated test scripts: test-unit, test-integration, test-e2e, test-all
- Update GitHub Actions workflow to run unit and integration tests separately
- Exclude e2e tests from CI as they require local virtualization
- Add Test::Exception dependency to cpanfile for e2e tests
- Fix cleanup in e2e tests to use OpenTofu destroy directly

All tests pass: unit (3 files, 11 tests), integration (1 file, 1 test), e2e (1 file, 4 tests)

Author: josecelano

.github/workflows/testing.yml

@@ -1,3 +1,4 @@
+---
 name: Testing
 
 on:
@@ -58,7 +59,12 @@ jobs:
           eval $(perl -I ~/perl5/lib/perl5/ -Mlocal::lib)
           carmel install
 
-      - name: Run tests
+      - name: Run unit tests
         run: |
           eval $(perl -I ~/perl5/lib/perl5/ -Mlocal::lib)
-          carmel exec -- prove -lr t/
+          ./script/test-unit
+
+      - name: Run integration tests
+        run: |
+          eval $(perl -I ~/perl5/lib/perl5/ -Mlocal::lib)
+          ./script/test-integration

README.md

@@ -111,8 +111,39 @@ cd build/tofu && tofu output
 
 ## Testing
 
-Run the test suite:
+The project includes two types of tests:
+
+### Unit Tests
+
+Run unit tests that don't require virtualization:
+
+```bash
+./script/test-unit
+```
+
+These tests can run in CI environments and test individual components in isolation.
+
+### E2E Tests
+
+Run end-to-end tests that require local virtualization support:
 
 ```bash
-carmel exec -- prove -l t/
+./script/test-e2e
 ```
+
+**Requirements for E2E tests:**
+
+- Local machine with KVM/libvirt support
+- OpenTofu installed
+- Required system tools: `qemu-system-x86_64`, `sshpass`
+- Cannot run in CI environments
+
+### All Tests
+
+Run both unit and E2E tests:
+
+```bash
+./script/test-all
+```
+
+E2E tests will be automatically skipped in CI environments or when virtualization tools are not available.

cpanfile

@@ -7,6 +7,7 @@ requires 'Path::Tiny';
 on 'test' => sub {
     requires 'Test2::Suite';
     requires 'Test::MockModule';
+    requires 'Test::Exception';
 };
 
 on 'develop' => sub {

project-words.txt

@@ -1,6 +1,7 @@
 autoport
 buildx
 cdrom
+chdir
 cloudinit
 commoninit
 containerd
@@ -10,6 +11,7 @@ cpanminus
 dialout
 dmacvicar
 dpkg
+Ilib
 installdeps
 keyrings
 libvirtd
@@ -30,6 +32,7 @@ shogo
 somaxconn
 sshpass
 subtest
+tfstate
 usermod
 vcpu
 virsh

script/test-all

@@ -0,0 +1,4 @@
+#!/bin/bash
+
+# Run all tests (unit, integration, and e2e)
+exec carmel exec -- prove -l t/unit/ t/integration/ t/e2e/ --ext .t --recurse

script/test-e2e

@@ -0,0 +1,4 @@
+#!/bin/bash
+
+# Run E2E tests only
+exec carmel exec -- prove -l t/e2e/ --ext .t --recurse

script/test-integration

@@ -0,0 +1,4 @@
+#!/bin/bash
+
+# Run integration tests only
+exec carmel exec -- prove -l t/integration/ --ext .t --recurse

script/test-unit

@@ -0,0 +1,4 @@
+#!/bin/bash
+
+# Run unit tests only
+exec carmel exec -- prove -l t/unit/ --ext .t --recurse

t/e2e/provision.t

@@ -0,0 +1,80 @@
+use v5.40;
+use Test::More;
+use Test::Exception;
+use Capture::Tiny qw(capture);
+use File::Temp qw(tempdir);
+use Path::Tiny qw(path);
+
+# Skip this test if we're in CI or if virtualization is not available
+BEGIN {
+    if ($ENV{CI} || $ENV{SKIP_E2E}) {
+        plan skip_all => 'E2E tests require local virtualization support';
+    }
+    
+    # Check if required tools are available
+    my $missing_tools = [];
+    for my $tool (qw(tofu libvirtd qemu-system-x86_64 sshpass)) {
+        system("which $tool >/dev/null 2>&1") != 0 and push @$missing_tools, $tool;
+    }
+    
+    if (@$missing_tools) {
+        plan skip_all => "Missing required tools: " . join(', ', @$missing_tools);
+    }
+}
+
+plan tests => 4;
+
+# Change to project root for the test
+my $original_cwd = path('.');
+my $project_root = path(__FILE__)->parent->parent->parent;
+chdir $project_root or die "Cannot change to project root: $!";
+
+# Ensure we have required templates
+ok(-f 'templates/provision/tofu/providers/libvirt/main.tf', 'Required OpenTofu template exists');
+ok(-f 'templates/provision/cloud-init.yml', 'Required cloud-init template exists');
+
+subtest 'provision command executes successfully' => sub {
+    plan tests => 3;
+    
+    # Capture command output
+    my ($stdout, $stderr, $exit_code) = capture {
+        system($^X, '-Ilib', 'bin/torrust-deploy', 'provision');
+    };
+    
+    # Command should complete successfully
+    is($exit_code, 0, 'provision command exits with status 0');
+    
+    # Output should contain success message
+    like($stdout, qr/Provisioning completed successfully!/, 'output contains success message');
+    
+    # Should not have critical errors in stderr
+    unlike($stderr, qr/(?:fatal|error|died)/i, 'no critical errors in stderr');
+};
+
+subtest 'provision creates expected infrastructure' => sub {
+    plan tests => 2;
+    
+    # Check if build directory was created
+    ok(-d 'build/tofu', 'build/tofu directory was created');
+    
+    # Check if OpenTofu state file exists
+    ok(-f 'build/tofu/terraform.tfstate', 'OpenTofu state file was created');
+};
+
+# Cleanup: destroy infrastructure after test
+END {
+    if (-f 'build/tofu/terraform.tfstate') {
+        note "Cleaning up test infrastructure...";
+        # Manual cleanup using OpenTofu since destroy command doesn't exist yet
+        if (-d 'build/tofu') {
+            chdir 'build/tofu';
+            system('tofu', 'destroy', '-auto-approve') if -f 'main.tf';
+            chdir '../..';
+        }
+    }
+    
+    # Restore original working directory
+    chdir $original_cwd if $original_cwd;
+}
+
+done_testing();

t/integration.t renamed to t/integration/app.t

@@ -1,6 +1,6 @@
 use Test2::V0;
 use FindBin qw($Bin);
-use lib "$Bin/../lib";
+use lib "$Bin/../../lib";
 
 use TorrustDeploy::App;