refactor: reorganize test structure and fix cloud-init header

- Mirror production directory structure in tests (t/command/provision.t)
- Update GitHub Actions workflow to use recursive test discovery (prove -lr)
- Configure yamllint to ignore cloud-init files for header format exception
- Add critical warning comment to prevent future cloud-init header modifications
- Update test regex to match corrected header format

Fixes VM provisioning issues caused by cloud-init header spacing.

Author: josecelano

.github/workflows/testing.yml

@@ -61,4 +61,4 @@ jobs:
       - name: Run tests
         run: |
           eval $(perl -I ~/perl5/lib/perl5/ -Mlocal::lib)
-          carmel exec -- prove -l t/
+          carmel exec -- prove -lr t/

.yamllint-ci.yml

@@ -8,3 +8,7 @@ rules:
   document-start: disable # Cloud-init files don't need --- start
   truthy:
     allowed-values: ["true", "false", "yes", "no", "on", "off"] # Allow cloud-init and GitHub Actions common values
+
+# Ignore cloud-init files for comment spacing since #cloud-config is a special directive
+ignore: |
+  **/cloud-init.yml

t/provision_command.t renamed to t/command/provision.t

@@ -1,6 +1,6 @@
 use Test2::V0;
 use FindBin qw($Bin);
-use lib "$Bin/../lib";
+use lib "$Bin/../../lib";
 use Path::Tiny qw(path);
 use File::Temp qw(tempdir);
 
@@ -30,7 +30,7 @@ subtest 'Provision command is discoverable by app' => sub {
 };
 
 subtest 'Template files exist and are readable' => sub {
-    my $project_root = path($Bin)->parent;
+    my $project_root = path($Bin)->parent->parent;
     my $templates_dir = $project_root->child('templates/provision');
 
     ok($templates_dir->exists, 'Templates/provision directory exists');
@@ -49,7 +49,7 @@ subtest 'Template files exist and are readable' => sub {
 };
 
 subtest 'Template files have expected content' => sub {
-    my $project_root = path($Bin)->parent;
+    my $project_root = path($Bin)->parent->parent;
     my $templates_dir = $project_root->child('templates/provision');
 
     my $main_tf_content = $templates_dir->child('tofu/providers/libvirt/main.tf')->slurp_utf8;
@@ -59,7 +59,7 @@ subtest 'Template files have expected content' => sub {
     like($main_tf_content, qr/torrust-tracker/, 'main.tf contains torrust-tracker VM name');
 
     my $cloud_init_content = $templates_dir->child('cloud-init.yml')->slurp_utf8;
-    like($cloud_init_content, qr/#cloud-config/, 'cloud-init.yml has cloud-config header');
+    like($cloud_init_content, qr/^#cloud-config/, 'cloud-init.yml has cloud-config header');
     like($cloud_init_content, qr/hostname:\s*torrust-tracker/, 'cloud-init.yml sets hostname');
     like($cloud_init_content, qr/name:\s*torrust/, 'cloud-init.yml creates torrust user');
 };
@@ -76,7 +76,7 @@ subtest 'Provision command template copying functionality' => sub {
     });
 
     # Test the _copy_templates method directly
-    my $project_root = path($Bin)->parent;
+    my $project_root = path($Bin)->parent->parent;
     my $templates_dir = $project_root->child('templates/provision');
 
     # Test that template directory exists

templates/provision/cloud-init.yml

@@ -1,6 +1,7 @@
 #cloud-config
-# cloud-config
-# Optimized cloud-init configuration based on manual testing
+# ⚠️  CRITICAL: The line above MUST be exactly "#cloud-config" (no space after #)
+# This is a special cloud-init directive that will break VM provisioning if modified.
+# Linting tools may warn about missing space, but this is intentional and required.
 
 # Basic system configuration
 hostname: torrust-tracker