refactor: move Ansible and OpenTofu packages to parent directory

- Move TorrustDeploy::Provision::Ansible to TorrustDeploy::Ansible
- Move TorrustDeploy::Provision::OpenTofu to TorrustDeploy::OpenTofu
- Remove empty TorrustDeploy::Provision directory
- Update all imports and package declarations
- Move corresponding test files from t/unit/provision/ to t/unit/
- Update all test references to use new package names
- All tests and linting pass after refactor

Author: josecelano

lib/TorrustDeploy/Provision/Ansible.pm renamed to lib/TorrustDeploy/Ansible.pm

@@ -1,12 +1,12 @@
-package TorrustDeploy::Provision::Ansible;
+package TorrustDeploy::Ansible;
 
 use v5.38;
 
 use Path::Tiny qw(path);
 
 =head1 NAME
 
-TorrustDeploy::Provision::Ansible - Ansible command wrapper for Torrust deployment
+TorrustDeploy::Ansible - Ansible command wrapper for Torrust deployment
 
 =head1 DESCRIPTION
 

lib/TorrustDeploy/App/Command/Provision.pm

@@ -3,8 +3,8 @@ package TorrustDeploy::App::Command::Provision;
 use v5.38;
 
 use TorrustDeploy::App -command;
-use TorrustDeploy::Provision::OpenTofu;
-use TorrustDeploy::Provision::Ansible;
+use TorrustDeploy::OpenTofu;
+use TorrustDeploy::Ansible;
 use Path::Tiny qw(path);
 use File::Spec;
 use Time::HiRes qw(sleep);
@@ -39,7 +39,7 @@ sub execute {
 
     # Set up OpenTofu working directory and copy resolved templates
     my $tofu_dir = $work_dir->child('tofu');
-    my $tofu = TorrustDeploy::Provision::OpenTofu->new();
+    my $tofu = TorrustDeploy::OpenTofu->new();
     $tofu->copy_templates($tofu_dir);
 
     # Initialize OpenTofu
@@ -54,7 +54,7 @@ sub execute {
 
     # Set up Ansible working directory and copy resolved templates
     my $ansible_dir = $work_dir->child('ansible');
-    my $ansible = TorrustDeploy::Provision::Ansible->new();
+    my $ansible = TorrustDeploy::Ansible->new();
     $ansible->copy_templates_and_generate_inventory($vm_ip, $ansible_dir);
 
     # Wait for cloud-init completion using Ansible

lib/TorrustDeploy/Provision/OpenTofu.pm renamed to lib/TorrustDeploy/OpenTofu.pm

@@ -1,4 +1,4 @@
-package TorrustDeploy::Provision::OpenTofu;
+package TorrustDeploy::OpenTofu;
 
 use v5.38;
 
@@ -7,7 +7,7 @@ use Path::Tiny qw(path);
 
 =head1 NAME
 
-TorrustDeploy::Provision::OpenTofu - OpenTofu command wrapper for Torrust deployment
+TorrustDeploy::OpenTofu - OpenTofu command wrapper for Torrust deployment
 
 =head1 DESCRIPTION
 

t/unit/provision/ansible.t renamed to t/unit/ansible.t

@@ -4,20 +4,20 @@ use File::Temp qw(tempdir);
 use Path::Tiny qw(path);
 
 # Test the Ansible wrapper module
-use_ok('TorrustDeploy::Provision::Ansible');
+use_ok('TorrustDeploy::Ansible');
 
 subtest 'constructor' => sub {
     plan tests => 2;
 
-    my $ansible = TorrustDeploy::Provision::Ansible->new();
+    my $ansible = TorrustDeploy::Ansible->new();
     ok(defined $ansible, 'constructor returns defined object');
-    isa_ok($ansible, 'TorrustDeploy::Provision::Ansible', 'object has correct class');
+    isa_ok($ansible, 'TorrustDeploy::Ansible', 'object has correct class');
 };
 
 subtest 'methods exist' => sub {
     plan tests => 2;
 
-    my $ansible = TorrustDeploy::Provision::Ansible->new();
+    my $ansible = TorrustDeploy::Ansible->new();
     can_ok($ansible, 'copy_templates_and_generate_inventory');
     can_ok($ansible, 'run_verification');
 };
@@ -26,8 +26,8 @@ subtest 'basic functionality test' => sub {
     plan tests => 1;
 
     # This is a simple smoke test - just verify the module loads and can be instantiated
-    my $ansible = TorrustDeploy::Provision::Ansible->new();
-    ok(ref($ansible) eq 'TorrustDeploy::Provision::Ansible', 'module works correctly');
+    my $ansible = TorrustDeploy::Ansible->new();
+    ok(ref($ansible) eq 'TorrustDeploy::Ansible', 'module works correctly');
 };
 
 done_testing();

t/unit/command/provision.t

@@ -87,8 +87,8 @@ subtest 'Provision command template copying functionality' => sub {
     ok($test_tofu_dir->exists, 'Test tofu directory created');
 
     # Test the OpenTofu copy_templates method since that's where it moved
-    require TorrustDeploy::Provision::OpenTofu;
-    my $tofu = TorrustDeploy::Provision::OpenTofu->new();
+    require TorrustDeploy::OpenTofu;
+    my $tofu = TorrustDeploy::OpenTofu->new();
 
     eval {
         $tofu->copy_templates($test_tofu_dir);
@@ -117,8 +117,8 @@ subtest 'Provision command internal methods' => sub {
     });
 
     # Test that OpenTofu functionality is available via the OpenTofu package
-    require TorrustDeploy::Provision::OpenTofu;
-    my $tofu = TorrustDeploy::Provision::OpenTofu->new();
+    require TorrustDeploy::OpenTofu;
+    my $tofu = TorrustDeploy::OpenTofu->new();
     ok($tofu->can('copy_templates'), 'OpenTofu package has copy_templates method');
     ok($tofu->can('init'), 'OpenTofu package has init method');
     ok($tofu->can('apply'), 'OpenTofu package has apply method');

t/unit/provision/opentofu.t renamed to t/unit/opentofu.t

@@ -4,24 +4,24 @@ use lib "$Bin/../../../lib";
 use Path::Tiny qw(path);
 use File::Temp qw(tempdir);
 
-use TorrustDeploy::Provision::OpenTofu;
+use TorrustDeploy::OpenTofu;
 
 subtest 'OpenTofu module loads correctly' => sub {
-    ok(TorrustDeploy::Provision::OpenTofu->can('new'), 'OpenTofu has new method');
-    ok(TorrustDeploy::Provision::OpenTofu->can('copy_templates'), 'OpenTofu has copy_templates method');
-    ok(TorrustDeploy::Provision::OpenTofu->can('init'), 'OpenTofu has init method');
-    ok(TorrustDeploy::Provision::OpenTofu->can('apply'), 'OpenTofu has apply method');
-    ok(TorrustDeploy::Provision::OpenTofu->can('get_vm_ip'), 'OpenTofu has get_vm_ip method');
+    ok(TorrustDeploy::OpenTofu->can('new'), 'OpenTofu has new method');
+    ok(TorrustDeploy::OpenTofu->can('copy_templates'), 'OpenTofu has copy_templates method');
+    ok(TorrustDeploy::OpenTofu->can('init'), 'OpenTofu has init method');
+    ok(TorrustDeploy::OpenTofu->can('apply'), 'OpenTofu has apply method');
+    ok(TorrustDeploy::OpenTofu->can('get_vm_ip'), 'OpenTofu has get_vm_ip method');
 };
 
 subtest 'OpenTofu instance creation' => sub {
-    my $tofu = TorrustDeploy::Provision::OpenTofu->new();
+    my $tofu = TorrustDeploy::OpenTofu->new();
     ok($tofu, 'OpenTofu instance created successfully');
-    isa_ok($tofu, 'TorrustDeploy::Provision::OpenTofu');
+    isa_ok($tofu, 'TorrustDeploy::OpenTofu');
 };
 
 subtest 'OpenTofu template copying functionality' => sub {
-    my $tofu = TorrustDeploy::Provision::OpenTofu->new();
+    my $tofu = TorrustDeploy::OpenTofu->new();
 
     # Create temporary directory for testing
     my $temp_dir = tempdir(CLEANUP => 1);