From 2703df41cd7d2261a4f27075aeeee550d136b871 Mon Sep 17 00:00:00 2001
From: Lewis Larsen <lewis@larsens.dev>
Date: Thu, 4 Jul 2024 14:37:15 +0100
Subject: [PATCH] fix: resolved linting errors, test failures and static
 analysis issues

---
 app/Models/BackupTask.php                     |  19 +-
 .../Backup/Tasks/AbstractBackupTask.php       |  14 +-
 config/app.php                                |   2 +-
 .../backup-tasks/index-table.blade.php        |   4 +-
 .../DeleteBackupTaskLogButtonTest.php         |   2 +-
 .../Livewire/UpdateBackupTaskFormTest.php     | 623 +++++++++---------
 tests/Pest.php                                |  24 +
 tests/Unit/Models/BackupTaskTest.php          |   5 +-
 tests/Unit/Services/Backup/BackupTest.php     |  19 +-
 .../Unit/Services/Backup/BackupTestClass.php  |  19 +
 .../Backup/Tasks/DatabaseBackupTaskTest.php   |  24 +-
 .../Backup/Tasks/FileBackupTaskTest.php       |  21 +-
 .../Tasks/Helpers/DBBackupTaskTestClass.php   |  21 +
 .../Tasks/Helpers/FileBackupTaskTestClass.php |  21 +
 .../Backup/Traits/BackupHelpersTest.php       |  49 +-
 .../Backup/Traits/BackupHelpersTestClass.php  |  51 ++
 16 files changed, 479 insertions(+), 439 deletions(-)
 create mode 100644 tests/Unit/Services/Backup/BackupTestClass.php
 create mode 100644 tests/Unit/Services/Backup/Tasks/Helpers/DBBackupTaskTestClass.php
 create mode 100644 tests/Unit/Services/Backup/Tasks/Helpers/FileBackupTaskTestClass.php
 create mode 100644 tests/Unit/Services/Backup/Traits/BackupHelpersTestClass.php

diff --git a/app/Models/BackupTask.php b/app/Models/BackupTask.php
index 485429f0..2a8a5faa 100644
--- a/app/Models/BackupTask.php
+++ b/app/Models/BackupTask.php
@@ -12,6 +12,7 @@
 use App\Traits\HasTags;
 use Cron\CronExpression;
 use Illuminate\Database\Eloquent\Builder;
+use Illuminate\Database\Eloquent\Collection;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
@@ -66,10 +67,10 @@ public static function logsCountPerMonthForLastSixMonths(int $userId): array
             ->get();
 
         return $results->mapWithKeys(function ($item) use ($locale) {
-            $carbonDate = Carbon::parse($item->month_date)->locale($locale);
+            $carbonDate = Carbon::parse($item->getAttribute('month_date'))->locale($locale);
             $localizedMonth = ucfirst($carbonDate->isoFormat('MMM YYYY'));
 
-            return [$localizedMonth => $item->count];
+            return [$localizedMonth => $item->getAttribute('count')];
         })->toArray();
     }
 
@@ -80,16 +81,18 @@ public static function logsCountPerMonthForLastSixMonths(int $userId): array
      */
     public static function backupTasksCountByType(int $userId): array
     {
-        return self::query()
+        /** @var Collection<int, Model> $results */
+        $results = self::query()
             ->where('user_id', $userId)
             ->selectRaw('type, COUNT(*) as count')
             ->groupBy('type')
-            ->get()
-            ->mapWithKeys(function ($item) {
-                $localizedType = __($item->type);
+            ->get();
+
+        return $results->mapWithKeys(function ($item) {
+            $localizedType = __($item->getAttribute('type'));
 
-                return [$localizedType => $item->count];
-            })
+            return [$localizedType => $item->getAttribute('count')];
+        })
             ->toArray();
     }
 
diff --git a/app/Services/Backup/Tasks/AbstractBackupTask.php b/app/Services/Backup/Tasks/AbstractBackupTask.php
index df89586d..6e1cc203 100644
--- a/app/Services/Backup/Tasks/AbstractBackupTask.php
+++ b/app/Services/Backup/Tasks/AbstractBackupTask.php
@@ -48,6 +48,13 @@ public function handle(): void
         }
     }
 
+    public function generateBackupFileName(string $extension): string
+    {
+        $prefix = $this->backupTask->hasFileNameAppended() ? $this->backupTask->appended_file_name . '_' : '';
+
+        return "{$prefix}backup_{$this->backupTask->id}_" . date('YmdHis') . ".{$extension}";
+    }
+
     abstract protected function performBackup(): void;
 
     /**
@@ -114,11 +121,4 @@ protected function logMessage(string $message): void
     {
         $this->logWithTimestamp($message, $this->backupTask->user->timezone);
     }
-
-    protected function generateBackupFileName(string $extension): string
-    {
-        $prefix = $this->backupTask->hasFileNameAppended() ? $this->backupTask->appended_file_name . '_' : '';
-
-        return "{$prefix}backup_{$this->backupTask->id}_" . date('YmdHis') . ".{$extension}";
-    }
 }
diff --git a/config/app.php b/config/app.php
index a8b11075..5a6f121b 100644
--- a/config/app.php
+++ b/config/app.php
@@ -45,6 +45,6 @@
         'en' => 'English (English)',
         'da' => 'Danish (Dansk)',
         'zh' => 'Chinese (中文)',
-        'ru' => 'Russian (Русский)'
+        'ru' => 'Russian (Русский)',
     ],
 ];
diff --git a/resources/views/livewire/backup-tasks/index-table.blade.php b/resources/views/livewire/backup-tasks/index-table.blade.php
index afa61592..4c025d78 100644
--- a/resources/views/livewire/backup-tasks/index-table.blade.php
+++ b/resources/views/livewire/backup-tasks/index-table.blade.php
@@ -60,8 +60,8 @@
     </div>
     <script>
         document.addEventListener('livewire:navigated', function () {
-            let filesType = @json(__('Files Task'), JSON_THROW_ON_ERROR);
-            let databaseType = @json(__('Database Task'), JSON_THROW_ON_ERROR);
+            let filesType = {!! __('Files Task') !!};
+            let databaseType = {!! __('Database Task') !!};
 
             tippy('#files-type', {
                 content: filesType,
diff --git a/tests/Feature/BackupTasks/Livewire/DeleteBackupTaskLogButtonTest.php b/tests/Feature/BackupTasks/Livewire/DeleteBackupTaskLogButtonTest.php
index 296ffccc..4c423403 100644
--- a/tests/Feature/BackupTasks/Livewire/DeleteBackupTaskLogButtonTest.php
+++ b/tests/Feature/BackupTasks/Livewire/DeleteBackupTaskLogButtonTest.php
@@ -14,7 +14,7 @@
 });
 
 test('the authorized user can delete a task log', function () {
-    \Masmerise\Toaster\Toaster::fake();
+    Toaster::fake();
 
     $user = User::factory()->create();
     $task = BackupTask::factory()->create(['user_id' => $user->id]);
diff --git a/tests/Feature/BackupTasks/Livewire/UpdateBackupTaskFormTest.php b/tests/Feature/BackupTasks/Livewire/UpdateBackupTaskFormTest.php
index 740906ac..ba4f0c44 100644
--- a/tests/Feature/BackupTasks/Livewire/UpdateBackupTaskFormTest.php
+++ b/tests/Feature/BackupTasks/Livewire/UpdateBackupTaskFormTest.php
@@ -8,26 +8,6 @@
 use App\Models\User;
 use Livewire\Livewire;
 
-function createUserWithBackupTaskAndDependencies(): array
-{
-    $user = User::factory()->create();
-    $remoteServer = RemoteServer::factory()->create(['user_id' => $user->id]);
-    $backupTask = BackupTask::factory()->create([
-        'user_id' => $user->id,
-        'remote_server_id' => $remoteServer->id,
-        'maximum_backups_to_keep' => 5,
-        'source_path' => null,
-    ]);
-    $backupDestination = BackupDestination::factory()->create(['user_id' => $user->id]);
-
-    return [
-        'user' => $user,
-        'remoteServer' => $remoteServer,
-        'backupTask' => $backupTask,
-        'backupDestination' => $backupDestination,
-    ];
-}
-
 beforeEach(function () {
     $this->data = createUserWithBackupTaskAndDependencies();
     $this->actingAs($this->data['user']);
@@ -40,354 +20,357 @@ function createUserWithBackupTaskAndDependencies(): array
     ])->assertOk();
 });
 
-test('backup task can be updated by the owner', function () {
-
-    $tag1 = Tag::factory()->create(['label' => 'Tag 1', 'user_id' => $this->data['user']->id]);
-    $tag2 = Tag::factory()->create(['label' => 'Tag 2', 'user_id' => $this->data['user']->id]);
-    $tagIds = [$tag1->id, $tag2->id];
-
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => $this->data['user']->remoteServers,
-        'availableTags' => $this->data['user']->tags,
-    ]);
-
-    $livewire->set('label', 'Updated Label')
-        ->set('description', 'Updated Description')
-        ->set('remoteServerId', $this->data['remoteServer']->id)
-        ->set('backupDestinationId', $this->data['backupDestination']->id)
-        ->set('frequency', BackupTask::FREQUENCY_WEEKLY)
-        ->set('timeToRun', '12:00')
-        ->set('backupsToKeep', 10)
-        ->set('backupType', BackupTask::TYPE_DATABASE)
-        ->set('databaseName', 'database_name')
-        ->set('appendedFileName', 'appended_file_name')
-        ->set('useCustomCron', false)
-        ->set('notifyEmail', 'alerts@email.com')
-        ->set('notifyDiscordWebhook', 'https://discord.com/api/webhooks/1234567890/ABC123')
-        ->set('notifySlackWebhook', 'https://hooks.slack.com/services/T00000000/B00000000/XXXXXXXXXXXXXXXX')
-        ->set('storePath', '/my-cool-backups')
-        ->set('excludedDatabaseTables', 'table1,table2')
-        ->set('selectedTags', $tagIds)
-        ->call('submit')
-        ->assertHasNoErrors();
-
-    $this->assertDatabaseHas('backup_tasks', [
-        'label' => 'Updated Label',
-        'description' => 'Updated Description',
-        'remote_server_id' => $this->data['remoteServer']->id,
-        'backup_destination_id' => $this->data['backupDestination']->id,
-        'frequency' => BackupTask::FREQUENCY_WEEKLY,
-        'time_to_run_at' => '12:00',
-        'maximum_backups_to_keep' => 10,
-        'type' => BackupTask::TYPE_DATABASE,
-        'database_name' => 'database_name',
-        'appended_file_name' => 'appended_file_name',
-        'notify_email' => 'alerts@email.com',
-        'notify_discord_webhook' => 'https://discord.com/api/webhooks/1234567890/ABC123',
-        'notify_slack_webhook' => 'https://hooks.slack.com/services/T00000000/B00000000/XXXXXXXXXXXXXXXX',
-        'store_path' => '/my-cool-backups',
-        'excluded_database_tables' => 'table1,table2',
-    ]);
-
-    $this->assertDatabaseHas('taggables', [
-        'tag_id' => $tag1->id,
-        'taggable_id' => $this->data['backupTask']->id,
-        'taggable_type' => 'App\Models\BackupTask',
-    ]);
-
-    $this->assertDatabaseHas('taggables', [
-        'tag_id' => $tag2->id,
-        'taggable_id' => $this->data['backupTask']->id,
-        'taggable_type' => 'App\Models\BackupTask',
-    ]);
-});
+describe('backup task update', function () {
+    test('can be updated by the owner', function () {
+        $tag1 = Tag::factory()->create(['label' => 'Tag 1', 'user_id' => $this->data['user']->id]);
+        $tag2 = Tag::factory()->create(['label' => 'Tag 2', 'user_id' => $this->data['user']->id]);
+        $tagIds = [$tag1->id, $tag2->id];
 
-test('backup task can be updated by the owner with custom cron', function () {
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => $this->data['user']->remoteServers,
-    ]);
-
-    $livewire->set('label', 'Updated Label')
-        ->set('description', 'Updated Description')
-        ->set('remoteServerId', $this->data['remoteServer']->id)
-        ->set('backupDestinationId', $this->data['backupDestination']->id)
-        ->set('cronExpression', '0 0 * * *')
-        ->set('backupsToKeep', 10)
-        ->set('backupType', BackupTask::TYPE_DATABASE)
-        ->set('databaseName', 'database_name')
-        ->set('appendedFileName', 'appended_file_name')
-        ->call('submit')
-        ->assertHasNoErrors();
-
-    $this->assertDatabaseHas('backup_tasks', [
-        'label' => 'Updated Label',
-        'description' => 'Updated Description',
-        'remote_server_id' => $this->data['remoteServer']->id,
-        'backup_destination_id' => $this->data['backupDestination']->id,
-        'custom_cron_expression' => '0 0 * * *',
-        'maximum_backups_to_keep' => 10,
-        'type' => BackupTask::TYPE_DATABASE,
-        'database_name' => 'database_name',
-        'appended_file_name' => 'appended_file_name',
-    ]);
-});
-
-test('backup task cannot be updated by another user', function () {
-    $anotherUser = User::factory()->create();
-
-    $this->actingAs($anotherUser);
-
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => $anotherUser->remoteServers,
-    ]);
-
-    $livewire->set('label', 'Updated Label')
-        ->set('description', 'Updated Description')
-        ->set('remoteServerId', $this->data['remoteServer']->id)
-        ->set('backupDestinationId', BackupDestination::factory()->create()->id)
-        ->set('frequency', BackupTask::FREQUENCY_WEEKLY)
-        ->set('timeToRun', '12:00')
-        ->set('backupsToKeep', 10)
-        ->set('backupType', BackupTask::TYPE_DATABASE)
-        ->set('databaseName', 'database_name')
-        ->set('appendedFileName', 'appended_file_name')
-        ->call('submit')
-        ->assertForbidden();
-});
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => $this->data['user']->remoteServers,
+            'availableTags' => $this->data['user']->tags,
+        ]);
 
-test('backup task has required validation rules', function () {
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => RemoteServer::all(),
-    ]);
-
-    $livewire->set('label', '')
-        ->set('backupDestinationId', '')
-        ->set('backupType', '')
-        ->call('submit')
-        ->assertHasErrors([
-            'label' => 'required',
-            'backupDestinationId' => 'required',
-            'backupType' => 'required',
+        $livewire->set('label', 'Updated Label')
+            ->set('description', 'Updated Description')
+            ->set('remoteServerId', $this->data['remoteServer']->id)
+            ->set('backupDestinationId', $this->data['backupDestination']->id)
+            ->set('frequency', BackupTask::FREQUENCY_WEEKLY)
+            ->set('timeToRun', '12:00')
+            ->set('backupsToKeep', 10)
+            ->set('backupType', BackupTask::TYPE_DATABASE)
+            ->set('databaseName', 'database_name')
+            ->set('appendedFileName', 'appended_file_name')
+            ->set('useCustomCron', false)
+            ->set('notifyEmail', 'alerts@email.com')
+            ->set('notifyDiscordWebhook', 'https://discord.com/api/webhooks/1234567890/ABC123')
+            ->set('notifySlackWebhook', 'https://hooks.slack.com/services/T00000000/B00000000/XXXXXXXXXXXXXXXX')
+            ->set('storePath', '/my-cool-backups')
+            ->set('excludedDatabaseTables', 'table1,table2')
+            ->set('selectedTags', $tagIds)
+            ->call('submit')
+            ->assertHasNoErrors();
+
+        $this->assertDatabaseHas('backup_tasks', [
+            'label' => 'Updated Label',
+            'description' => 'Updated Description',
+            'remote_server_id' => $this->data['remoteServer']->id,
+            'backup_destination_id' => $this->data['backupDestination']->id,
+            'frequency' => BackupTask::FREQUENCY_WEEKLY,
+            'time_to_run_at' => '12:00',
+            'maximum_backups_to_keep' => 10,
+            'type' => BackupTask::TYPE_DATABASE,
+            'database_name' => 'database_name',
+            'appended_file_name' => 'appended_file_name',
+            'notify_email' => 'alerts@email.com',
+            'notify_discord_webhook' => 'https://discord.com/api/webhooks/1234567890/ABC123',
+            'notify_slack_webhook' => 'https://hooks.slack.com/services/T00000000/B00000000/XXXXXXXXXXXXXXXX',
+            'store_path' => '/my-cool-backups',
+            'excluded_database_tables' => 'table1,table2',
         ]);
-});
 
-test('discord webhook url must be valid', function () {
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => RemoteServer::all(),
-    ]);
+        $this->assertDatabaseHas('taggables', [
+            'tag_id' => $tag1->id,
+            'taggable_id' => $this->data['backupTask']->id,
+            'taggable_type' => 'App\Models\BackupTask',
+        ]);
 
-    $livewire->set('notifyDiscordWebhook', 'invalid-discord-url')
-        ->call('submit')
-        ->assertHasErrors([
-            'notifyDiscordWebhook' => 'starts_with:https://discord.com/api/webhooks/',
+        $this->assertDatabaseHas('taggables', [
+            'tag_id' => $tag2->id,
+            'taggable_id' => $this->data['backupTask']->id,
+            'taggable_type' => 'App\Models\BackupTask',
         ]);
-});
+    });
 
-test('slack webhook url must be valid', function () {
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => RemoteServer::all(),
-    ]);
+    test('can be updated by the owner with custom cron', function () {
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => $this->data['user']->remoteServers,
+        ]);
 
-    $livewire->set('notifySlackWebhook', 'invalid-slack-url')
-        ->call('submit')
-        ->assertHasErrors([
-            'notifySlackWebhook' => 'starts_with:https://hooks.slack.com/services/',
+        $livewire->set('label', 'Updated Label')
+            ->set('description', 'Updated Description')
+            ->set('remoteServerId', $this->data['remoteServer']->id)
+            ->set('backupDestinationId', $this->data['backupDestination']->id)
+            ->set('cronExpression', '0 0 * * *')
+            ->set('backupsToKeep', 10)
+            ->set('backupType', BackupTask::TYPE_DATABASE)
+            ->set('databaseName', 'database_name')
+            ->set('appendedFileName', 'appended_file_name')
+            ->call('submit')
+            ->assertHasNoErrors();
+
+        $this->assertDatabaseHas('backup_tasks', [
+            'label' => 'Updated Label',
+            'description' => 'Updated Description',
+            'remote_server_id' => $this->data['remoteServer']->id,
+            'backup_destination_id' => $this->data['backupDestination']->id,
+            'custom_cron_expression' => '0 0 * * *',
+            'maximum_backups_to_keep' => 10,
+            'type' => BackupTask::TYPE_DATABASE,
+            'database_name' => 'database_name',
+            'appended_file_name' => 'appended_file_name',
         ]);
-});
+    });
 
-test('the time to run at is converted from the users timezone to UTC', function () {
+    test('cannot be updated by another user', function () {
+        $anotherUser = User::factory()->create();
 
-    $this->data['user']->update(['timezone' => 'America/New_York']);
+        $this->actingAs($anotherUser);
 
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => RemoteServer::all(),
-    ]);
-
-    $livewire->set('timeToRun', '12:00') // 12:00 PM in America/New_York
-        ->set('description', '')
-        ->set('sourcePath', '/var/www/html')
-        ->call('submit')
-        ->assertHasNoErrors();
-
-    $this->assertDatabaseHas('backup_tasks', [
-        'time_to_run_at' => '16:00', // 4:00 PM in UTC
-    ]);
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => $anotherUser->remoteServers,
+        ]);
+
+        $livewire->set('label', 'Updated Label')
+            ->set('description', 'Updated Description')
+            ->set('remoteServerId', $this->data['remoteServer']->id)
+            ->set('backupDestinationId', BackupDestination::factory()->create()->id)
+            ->set('frequency', BackupTask::FREQUENCY_WEEKLY)
+            ->set('timeToRun', '12:00')
+            ->set('backupsToKeep', 10)
+            ->set('backupType', BackupTask::TYPE_DATABASE)
+            ->set('databaseName', 'database_name')
+            ->set('appendedFileName', 'appended_file_name')
+            ->call('submit')
+            ->assertForbidden();
+    });
 });
 
-test('the store path needs to be a valid unix path', function () {
+describe('validation rules', function () {
+    test('backup task has required validation rules', function () {
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => RemoteServer::all(),
+        ]);
 
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => RemoteServer::all(),
-    ]);
+        $livewire->set('label', '')
+            ->set('backupDestinationId', '')
+            ->set('backupType', '')
+            ->call('submit')
+            ->assertHasErrors([
+                'label' => 'required',
+                'backupDestinationId' => 'required',
+                'backupType' => 'required',
+            ]);
+    });
+
+    test('discord webhook url must be valid', function () {
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => RemoteServer::all(),
+        ]);
 
-    $livewire->set('sourcePath', 'C:\var\www\html')
-        ->call('submit')
-        ->assertHasErrors([
-            'sourcePath' => 'regex:/^\/[a-zA-Z0-9_\/]+$/', // Unix path
+        $livewire->set('notifyDiscordWebhook', 'invalid-discord-url')
+            ->call('submit')
+            ->assertHasErrors([
+                'notifyDiscordWebhook' => 'starts_with:https://discord.com/api/webhooks/',
+            ]);
+    });
+
+    test('slack webhook url must be valid', function () {
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => RemoteServer::all(),
         ]);
-});
 
-test('the excluded database tables must be a valid comma separated list', function () {
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => RemoteServer::all(),
-    ]);
-
-    $livewire->set('excludedDatabaseTables', 'table1, table2, table3')
-        ->set('sourcePath', '/var/www/html')
-        ->set('description', '')
-        ->call('submit')
-        ->assertHasErrors([
-            'excludedDatabaseTables' => 'regex:/^([a-zA-Z0-9_]+,? ?)+$/', // Comma separated list
+        $livewire->set('notifySlackWebhook', 'invalid-slack-url')
+            ->call('submit')
+            ->assertHasErrors([
+                'notifySlackWebhook' => 'starts_with:https://hooks.slack.com/services/',
+            ]);
+    });
+
+    test('the store path needs to be a valid unix path', function () {
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => RemoteServer::all(),
+        ]);
+
+        $livewire->set('sourcePath', 'C:\var\www\html')
+            ->call('submit')
+            ->assertHasErrors([
+                'sourcePath' => 'regex:/^\/[a-zA-Z0-9_\/]+$/', // Unix path
+            ]);
+    });
+
+    test('the excluded database tables must be a valid comma separated list', function () {
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => RemoteServer::all(),
         ]);
+
+        $livewire->set('excludedDatabaseTables', 'table1, table2, table3')
+            ->set('sourcePath', '/var/www/html')
+            ->set('description', '')
+            ->call('submit')
+            ->assertHasErrors([
+                'excludedDatabaseTables' => 'regex:/^([a-zA-Z0-9_]+,? ?)+$/', // Comma separated list
+            ]);
+    });
 });
 
-test('a task cannot share the same time as another task on the same server', function () {
-    $this->withoutExceptionHandling();
-    $user = User::factory()->create();
+describe('time and timezone handling', function () {
+    test('the time to run at is converted from the users timezone to UTC', function () {
+        $this->data['user']->update(['timezone' => 'America/New_York']);
 
-    $remoteServer = RemoteServer::factory()->create([
-        'user_id' => $user->id,
-    ]);
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => RemoteServer::all(),
+        ]);
 
-    BackupTask::factory()->create([
-        'remote_server_id' => $remoteServer->id,
-        'time_to_run_at' => '12:00',
-        'user_id' => $user->id,
-    ]);
+        $livewire->set('timeToRun', '12:00') // 12:00 PM in America/New_York
+            ->set('description', '')
+            ->set('sourcePath', '/var/www/html')
+            ->call('submit')
+            ->assertHasNoErrors();
 
-    $backupTaskTwo = BackupTask::factory()->create([
-        'remote_server_id' => $remoteServer->id,
-        'time_to_run_at' => '13:00',
-        'user_id' => $user->id,
-    ]);
+        $this->assertDatabaseHas('backup_tasks', [
+            'time_to_run_at' => '16:00', // 4:00 PM in UTC
+        ]);
+    });
 
-    $this->assertDatabaseHas('backup_tasks', [
-        'time_to_run_at' => '12:00',
-    ]);
+    test('a task cannot share the same time as another task on the same server', function () {
+        $this->withoutExceptionHandling();
+        $user = User::factory()->create();
 
-    $this->actingAs($user);
+        $remoteServer = RemoteServer::factory()->create([
+            'user_id' => $user->id,
+        ]);
 
-    Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $backupTaskTwo,
-        'remoteServers' => RemoteServer::all(),
-    ])
-        ->set('timeToRun', '12:00')
-        ->call('submit')
-        ->assertHasErrors('timeToRun');
-});
+        BackupTask::factory()->create([
+            'remote_server_id' => $remoteServer->id,
+            'time_to_run_at' => '12:00',
+            'user_id' => $user->id,
+        ]);
 
-test('a task retains its set time without validation errors', function () {
+        $backupTaskTwo = BackupTask::factory()->create([
+            'remote_server_id' => $remoteServer->id,
+            'time_to_run_at' => '13:00',
+            'user_id' => $user->id,
+        ]);
 
-    $user = User::factory()->create();
+        $this->assertDatabaseHas('backup_tasks', [
+            'time_to_run_at' => '12:00',
+        ]);
 
-    $remoteServer = RemoteServer::factory()->create([
-        'user_id' => $user->id,
-    ]);
+        $this->actingAs($user);
 
-    $backupTask = BackupTask::factory()->create([
-        'remote_server_id' => $remoteServer->id,
-        'time_to_run_at' => '12:00',
-        'user_id' => $user->id,
-    ]);
+        Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $backupTaskTwo,
+            'remoteServers' => RemoteServer::all(),
+        ])
+            ->set('timeToRun', '12:00')
+            ->call('submit')
+            ->assertHasErrors('timeToRun');
+    });
 
-    $this->assertDatabaseHas('backup_tasks', [
-        'time_to_run_at' => '12:00',
-    ]);
+    test('a task retains its set time without validation errors', function () {
+        $user = User::factory()->create();
 
-    $this->actingAs($user);
+        $remoteServer = RemoteServer::factory()->create([
+            'user_id' => $user->id,
+        ]);
 
-    Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $backupTask,
-        'remoteServers' => RemoteServer::all(),
-    ])
-        ->set('timeToRun', '12:00')
-        ->set('sourcePath', '/var/www/html')
-        ->set('description', '')
-        ->call('submit')
-        ->assertHasNoErrors();
-});
+        $backupTask = BackupTask::factory()->create([
+            'remote_server_id' => $remoteServer->id,
+            'time_to_run_at' => '12:00',
+            'user_id' => $user->id,
+        ]);
 
-test('users cannot set tags that do not belong them', function () {
+        $this->assertDatabaseHas('backup_tasks', [
+            'time_to_run_at' => '12:00',
+        ]);
 
-    $tag = Tag::factory()->create();
+        $this->actingAs($user);
+
+        Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $backupTask,
+            'remoteServers' => RemoteServer::all(),
+        ])
+            ->set('timeToRun', '12:00')
+            ->set('sourcePath', '/var/www/html')
+            ->set('description', '')
+            ->call('submit')
+            ->assertHasNoErrors();
+    });
+});
 
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => RemoteServer::all(),
-    ]);
+describe('tag handling', function () {
+    test('users cannot set tags that do not belong them', function () {
+        $tag = Tag::factory()->create();
 
-    $livewire->set('selectedTags', [$tag->id])
-        ->call('submit')
-        ->assertHasErrors([
-            'selectedTags' => 'exists',
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => RemoteServer::all(),
         ]);
-});
 
-test('users cannot set tags that do not exist', function () {
+        $livewire->set('selectedTags', [$tag->id])
+            ->call('submit')
+            ->assertHasErrors([
+                'selectedTags' => 'exists',
+            ]);
+    });
 
-    $livewire = Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $this->data['backupTask'],
-        'remoteServers' => RemoteServer::all(),
-    ]);
+    test('users cannot set tags that do not exist', function () {
 
-    $livewire->set('selectedTags', [999])
-        ->call('submit')
-        ->assertHasErrors([
-            'selectedTags' => 'exists',
+        $livewire = Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $this->data['backupTask'],
+            'remoteServers' => RemoteServer::all(),
         ]);
-});
 
-test('a user can update their already existing tags', function () {
+        $livewire->set('selectedTags', [999])
+            ->call('submit')
+            ->assertHasErrors([
+                'selectedTags' => 'exists',
+            ]);
+    });
 
-    $user = User::factory()->create();
+    test('a user can update their already existing tags', function () {
 
-    $tag1 = Tag::factory()->create(['label' => 'Tag 1', 'user_id' => $user->id]);
-    $tag2 = Tag::factory()->create(['label' => 'Tag 2', 'user_id' => $user->id]);
-    $tag3 = Tag::factory()->create(['label' => 'Tag 3', 'user_id' => $user->id]);
+        $user = User::factory()->create();
 
-    $backupTask = BackupTask::factory()->create([
-        'user_id' => $user->id,
-    ]);
+        $tag1 = Tag::factory()->create(['label' => 'Tag 1', 'user_id' => $user->id]);
+        $tag2 = Tag::factory()->create(['label' => 'Tag 2', 'user_id' => $user->id]);
+        $tag3 = Tag::factory()->create(['label' => 'Tag 3', 'user_id' => $user->id]);
 
-    $backupTask->tags()->attach([$tag1->id, $tag2->id]);
+        $backupTask = BackupTask::factory()->create([
+            'user_id' => $user->id,
+        ]);
 
-    $this->actingAs($user);
+        $backupTask->tags()->attach([$tag1->id, $tag2->id]);
+
+        $this->actingAs($user);
+
+        Livewire::test(UpdateBackupTaskForm::class, [
+            'backupTask' => $backupTask,
+            'remoteServers' => RemoteServer::all(),
+            'availableTags' => $user->tags,
+        ])
+            ->set('selectedTags', [$tag3->id])
+            ->set('sourcePath', '/var/www/html')
+            ->set('description', '')
+            ->call('submit')
+            ->assertHasNoErrors();
+
+        $this->assertDatabaseHas('taggables', [
+            'tag_id' => $tag3->id,
+            'taggable_id' => $backupTask->id,
+            'taggable_type' => 'App\Models\BackupTask',
+        ]);
 
-    Livewire::test(UpdateBackupTaskForm::class, [
-        'backupTask' => $backupTask,
-        'remoteServers' => RemoteServer::all(),
-        'availableTags' => $user->tags,
-    ])
-        ->set('selectedTags', [$tag3->id])
-        ->set('sourcePath', '/var/www/html')
-        ->set('description', '')
-        ->call('submit')
-        ->assertHasNoErrors();
-
-    $this->assertDatabaseHas('taggables', [
-        'tag_id' => $tag3->id,
-        'taggable_id' => $backupTask->id,
-        'taggable_type' => 'App\Models\BackupTask',
-    ]);
-
-    $this->assertDatabaseMissing('taggables', [
-        'tag_id' => $tag1->id,
-        'taggable_id' => $backupTask->id,
-        'taggable_type' => 'App\Models\BackupTask',
-    ]);
-
-    $this->assertDatabaseMissing('taggables', [
-        'tag_id' => $tag2->id,
-        'taggable_id' => $backupTask->id,
-        'taggable_type' => 'App\Models\BackupTask',
-    ]);
+        $this->assertDatabaseMissing('taggables', [
+            'tag_id' => $tag1->id,
+            'taggable_id' => $backupTask->id,
+            'taggable_type' => 'App\Models\BackupTask',
+        ]);
+
+        $this->assertDatabaseMissing('taggables', [
+            'tag_id' => $tag2->id,
+            'taggable_id' => $backupTask->id,
+            'taggable_type' => 'App\Models\BackupTask',
+        ]);
+    });
 });
diff --git a/tests/Pest.php b/tests/Pest.php
index 3e2c5b1a..3d11f565 100644
--- a/tests/Pest.php
+++ b/tests/Pest.php
@@ -1,6 +1,10 @@
 <?php
 
 use App\Console\Commands\GenerateSSHKeyCommand;
+use App\Models\BackupDestination;
+use App\Models\BackupTask;
+use App\Models\RemoteServer;
+use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Tests\TestCase;
 
@@ -90,3 +94,23 @@ function test_restore_keys(): void
     }
 }
 
+function createUserWithBackupTaskAndDependencies(): array
+{
+    $user = User::factory()->create();
+    $remoteServer = RemoteServer::factory()->create(['user_id' => $user->id]);
+    $backupTask = BackupTask::factory()->create([
+        'user_id' => $user->id,
+        'remote_server_id' => $remoteServer->id,
+        'maximum_backups_to_keep' => 5,
+        'source_path' => null,
+    ]);
+    $backupDestination = BackupDestination::factory()->create(['user_id' => $user->id]);
+
+    return [
+        'user' => $user,
+        'remoteServer' => $remoteServer,
+        'backupTask' => $backupTask,
+        'backupDestination' => $backupDestination,
+    ];
+}
+
diff --git a/tests/Unit/Models/BackupTaskTest.php b/tests/Unit/Models/BackupTaskTest.php
index 7cf40bca..5bee6768 100644
--- a/tests/Unit/Models/BackupTaskTest.php
+++ b/tests/Unit/Models/BackupTaskTest.php
@@ -9,6 +9,7 @@
 use App\Models\BackupTaskData;
 use App\Models\BackupTaskLog;
 use App\Models\RemoteServer;
+use App\Models\Tag;
 use App\Models\User;
 use Illuminate\Support\Carbon;
 
@@ -780,8 +781,8 @@
 it('returns the attached tags as a string', function () {
 
     $task = BackupTask::factory()->create();
-    $tag1 = \App\Models\Tag::factory()->create(['label' => 'Tag 1']);
-    $tag2 = \App\Models\Tag::factory()->create(['label' => 'Tag 2']);
+    $tag1 = Tag::factory()->create(['label' => 'Tag 1']);
+    $tag2 = Tag::factory()->create(['label' => 'Tag 2']);
 
     $task->tags()->attach([$tag1->id, $tag2->id]);
 
diff --git a/tests/Unit/Services/Backup/BackupTest.php b/tests/Unit/Services/Backup/BackupTest.php
index 26c86bce..e473a4cb 100644
--- a/tests/Unit/Services/Backup/BackupTest.php
+++ b/tests/Unit/Services/Backup/BackupTest.php
@@ -6,27 +6,14 @@
 use App\Models\BackupTask;
 use App\Models\BackupTaskLog;
 use App\Models\RemoteServer;
-use App\Services\Backup\Backup;
 use App\Services\Backup\Contracts\SFTPInterface;
 use App\Services\Backup\Destinations\Contracts\BackupDestinationInterface;
 use App\Services\Backup\Destinations\S3;
 use Illuminate\Support\Facades\Config;
-
-class BackupTest extends Backup
-{
-    public function publicValidateConfiguration(): void
-    {
-        $this->validateConfiguration();
-    }
-
-    public function publicCreateSFTP($host, $port, $timeout): SFTPInterface
-    {
-        return $this->createSFTP($host, $port, $timeout);
-    }
-}
+use Tests\Unit\Services\Backup\BackupTestClass;
 
 beforeEach(function () {
-    $this->backup = Mockery::mock(BackupTest::class)
+    $this->backup = Mockery::mock(BackupTestClass::class)
         ->makePartial()
         ->shouldAllowMockingProtectedMethods();
     $this->mockSftp = Mockery::mock(SFTPInterface::class);
@@ -113,6 +100,7 @@ public function publicCreateSFTP($host, $port, $timeout): SFTPInterface
 });
 
 it('establishes SFTP connection', function () {
+    test_create_keys();
     $remoteServer = RemoteServer::factory()->create([
         'ip_address' => '192.168.1.1',
         'port' => 22,
@@ -138,6 +126,7 @@ public function publicCreateSFTP($host, $port, $timeout): SFTPInterface
 
     expect($sftp)->toBe($mockSftp)
         ->and($remoteServer->fresh()->connectivity_status)->toBe('online');
+    test_restore_keys();
 });
 
 it('zips remote directory successfully', function () {
diff --git a/tests/Unit/Services/Backup/BackupTestClass.php b/tests/Unit/Services/Backup/BackupTestClass.php
new file mode 100644
index 00000000..a85a8eb6
--- /dev/null
+++ b/tests/Unit/Services/Backup/BackupTestClass.php
@@ -0,0 +1,19 @@
+<?php
+
+namespace Tests\Unit\Services\Backup;
+
+use App\Services\Backup\Backup;
+use App\Services\Backup\Contracts\SFTPInterface;
+
+class BackupTestClass extends Backup
+{
+    public function publicValidateConfiguration(): void
+    {
+        $this->validateConfiguration();
+    }
+
+    public function publicCreateSFTP($host, $port, $timeout): SFTPInterface
+    {
+        return $this->createSFTP($host, $port, $timeout);
+    }
+}
diff --git a/tests/Unit/Services/Backup/Tasks/DatabaseBackupTaskTest.php b/tests/Unit/Services/Backup/Tasks/DatabaseBackupTaskTest.php
index 787b02d3..c933870b 100644
--- a/tests/Unit/Services/Backup/Tasks/DatabaseBackupTaskTest.php
+++ b/tests/Unit/Services/Backup/Tasks/DatabaseBackupTaskTest.php
@@ -9,27 +9,13 @@
 use App\Services\Backup\Adapters\SFTPAdapter;
 use App\Services\Backup\BackupConstants;
 use App\Services\Backup\Destinations\S3;
-use App\Services\Backup\Tasks\DatabaseBackupTask;
 use Illuminate\Foundation\Testing\RefreshDatabase;
+use Tests\Unit\Services\Backup\Tasks\Helpers\DBBackupTaskTestClass;
 
 uses(RefreshDatabase::class);
 
-class MockDatabaseBackupTask extends DatabaseBackupTask
-{
-    public function __construct($backupTaskId)
-    {
-        $this->backupTask = BackupTask::findOrFail($backupTaskId);
-        $this->scriptRunTime = microtime(true);
-        $this->logOutput = '';
-    }
-
-    public function validateConfiguration(): void
-    {
-        // Do nothing
-    }
-}
-
 beforeEach(function () {
+    Event::fake();
     $this->remoteServer = RemoteServer::factory()->create([
         'database_password' => encrypt('testpassword'),
     ]);
@@ -50,7 +36,7 @@ public function validateConfiguration(): void
     $this->sftpMock->shouldReceive('exec')->andReturn(''); // Default behavior
     $this->s3Mock = Mockery::mock(S3::class);
 
-    $this->databaseBackupTask = Mockery::mock(MockDatabaseBackupTask::class, [$this->backupTask->id])
+    $this->databaseBackupTask = Mockery::mock(DBBackupTaskTestClass::class, [$this->backupTask->id])
         ->makePartial()
         ->shouldAllowMockingProtectedMethods();
 
@@ -256,7 +242,7 @@ public function validateConfiguration(): void
     $this->backupTask->update(['appended_file_name' => 'custom']);
 
     // Create a new mock instance for this specific scenario
-    $mockTaskWithAppend = Mockery::mock(MockDatabaseBackupTask::class, [$this->backupTask->id])
+    $mockTaskWithAppend = Mockery::mock(DBBackupTaskTestClass::class, [$this->backupTask->id])
         ->makePartial()
         ->shouldAllowMockingProtectedMethods();
 
@@ -267,7 +253,7 @@ public function validateConfiguration(): void
 });
 
 test('handle unexpected exception', function () {
-    $this->databaseBackupTask->shouldReceive('performBackup')->andThrow(new \Exception('Unexpected error'));
+    $this->databaseBackupTask->shouldReceive('performBackup')->andThrow(new Exception('Unexpected error'));
     $this->databaseBackupTask->shouldReceive('handleBackupFailure')->once();
 
     expect(fn () => $this->databaseBackupTask->handle())
diff --git a/tests/Unit/Services/Backup/Tasks/FileBackupTaskTest.php b/tests/Unit/Services/Backup/Tasks/FileBackupTaskTest.php
index 7ee01f01..53334452 100644
--- a/tests/Unit/Services/Backup/Tasks/FileBackupTaskTest.php
+++ b/tests/Unit/Services/Backup/Tasks/FileBackupTaskTest.php
@@ -10,30 +10,17 @@
 use App\Services\Backup\Adapters\SFTPAdapter;
 use App\Services\Backup\BackupConstants;
 use App\Services\Backup\Destinations\S3;
-use App\Services\Backup\Tasks\FileBackupTask;
 use Exception;
 use Illuminate\Foundation\Testing\RefreshDatabase;
+use Illuminate\Support\Facades\Event;
 use Mockery;
 use RuntimeException;
+use Tests\Unit\Services\Backup\Tasks\Helpers\FileBackupTaskTestClass;
 
 uses(RefreshDatabase::class);
 
-class MockFileBackupTask extends FileBackupTask
-{
-    public function __construct($backupTaskId)
-    {
-        $this->backupTask = BackupTask::findOrFail($backupTaskId);
-        $this->scriptRunTime = microtime(true);
-        $this->logOutput = '';
-    }
-
-    public function validateConfiguration(): void
-    {
-        // Do nothing
-    }
-}
-
 beforeEach(function () {
+    Event::fake();
     $this->remoteServer = RemoteServer::factory()->create();
     $this->backupDestination = BackupDestination::factory()->create([
         'type' => BackupConstants::DRIVER_S3,
@@ -51,7 +38,7 @@ public function validateConfiguration(): void
     $this->sftpMock->shouldReceive('delete')->andReturn(true)->byDefault();
     $this->s3Mock = Mockery::mock(S3::class);
 
-    $this->fileBackupTask = Mockery::mock(MockFileBackupTask::class, [$this->backupTask->id])
+    $this->fileBackupTask = Mockery::mock(FileBackupTaskTestClass::class, [$this->backupTask->id])
         ->makePartial()
         ->shouldAllowMockingProtectedMethods();
 
diff --git a/tests/Unit/Services/Backup/Tasks/Helpers/DBBackupTaskTestClass.php b/tests/Unit/Services/Backup/Tasks/Helpers/DBBackupTaskTestClass.php
new file mode 100644
index 00000000..76e723f0
--- /dev/null
+++ b/tests/Unit/Services/Backup/Tasks/Helpers/DBBackupTaskTestClass.php
@@ -0,0 +1,21 @@
+<?php
+
+namespace Tests\Unit\Services\Backup\Tasks\Helpers;
+
+use App\Models\BackupTask;
+use App\Services\Backup\Tasks\DatabaseBackupTask;
+
+class DBBackupTaskTestClass extends DatabaseBackupTask
+{
+    public function __construct($backupTaskId)
+    {
+        $this->backupTask = BackupTask::findOrFail($backupTaskId);
+        $this->scriptRunTime = microtime(true);
+        $this->logOutput = '';
+    }
+
+    public function validateConfiguration(): void
+    {
+        // Do nothing
+    }
+}
diff --git a/tests/Unit/Services/Backup/Tasks/Helpers/FileBackupTaskTestClass.php b/tests/Unit/Services/Backup/Tasks/Helpers/FileBackupTaskTestClass.php
new file mode 100644
index 00000000..fac6fdb1
--- /dev/null
+++ b/tests/Unit/Services/Backup/Tasks/Helpers/FileBackupTaskTestClass.php
@@ -0,0 +1,21 @@
+<?php
+
+namespace Tests\Unit\Services\Backup\Tasks\Helpers;
+
+use App\Models\BackupTask;
+use App\Services\Backup\Tasks\FileBackupTask;
+
+class FileBackupTaskTestClass extends FileBackupTask
+{
+    public function __construct($backupTaskId)
+    {
+        $this->backupTask = BackupTask::findOrFail($backupTaskId);
+        $this->scriptRunTime = microtime(true);
+        $this->logOutput = '';
+    }
+
+    public function validateConfiguration(): void
+    {
+        // Do nothing
+    }
+}
diff --git a/tests/Unit/Services/Backup/Traits/BackupHelpersTest.php b/tests/Unit/Services/Backup/Traits/BackupHelpersTest.php
index 0c27f363..4797fa80 100644
--- a/tests/Unit/Services/Backup/Traits/BackupHelpersTest.php
+++ b/tests/Unit/Services/Backup/Traits/BackupHelpersTest.php
@@ -1,56 +1,11 @@
 <?php
 
 use App\Services\Backup\Contracts\SFTPInterface;
-use App\Services\Backup\Traits\BackupHelpers;
 use Illuminate\Support\Facades\Log;
-
-class BackupHelpersTest
-{
-    use BackupHelpers;
-
-    public function publicDownloadFileViaSFTP(SFTPInterface $sftp, string $remoteZipPath): string
-    {
-        return $this->downloadFileViaSFTP($sftp, $remoteZipPath);
-    }
-
-    public function publicOpenFileAsStream(string $tempFile): mixed
-    {
-        return $this->openFileAsStream($tempFile);
-    }
-
-    public function publicCleanUpTempFile(string $tempFile): void
-    {
-        $this->cleanUpTempFile($tempFile);
-    }
-
-    public function publicLogInfo(string $message, array $context = []): void
-    {
-        $this->logInfo($message, $context);
-    }
-
-    public function publicLogDebug(string $message, array $context = []): void
-    {
-        $this->logDebug($message, $context);
-    }
-
-    public function publicLogWarning(string $message, array $context = []): void
-    {
-        $this->logWarning($message, $context);
-    }
-
-    public function publicLogError(string $message, array $context = []): void
-    {
-        $this->logError($message, $context);
-    }
-
-    public function publicLogCritical(string $message, array $context = []): void
-    {
-        $this->logCritical($message, $context);
-    }
-}
+use Tests\Unit\Services\Backup\Traits\BackupHelpersTestClass;
 
 beforeEach(function () {
-    $this->helperClass = new BackupHelpersTest;
+    $this->helperClass = new BackupHelpersTestClass;
 });
 
 it('retries command until success', function () {
diff --git a/tests/Unit/Services/Backup/Traits/BackupHelpersTestClass.php b/tests/Unit/Services/Backup/Traits/BackupHelpersTestClass.php
new file mode 100644
index 00000000..20f84b4f
--- /dev/null
+++ b/tests/Unit/Services/Backup/Traits/BackupHelpersTestClass.php
@@ -0,0 +1,51 @@
+<?php
+
+namespace Tests\Unit\Services\Backup\Traits;
+
+use App\Services\Backup\Contracts\SFTPInterface;
+use App\Services\Backup\Traits\BackupHelpers;
+
+class BackupHelpersTestClass
+{
+    use BackupHelpers;
+
+    public function publicDownloadFileViaSFTP(SFTPInterface $sftp, string $remoteZipPath): string
+    {
+        return $this->downloadFileViaSFTP($sftp, $remoteZipPath);
+    }
+
+    public function publicOpenFileAsStream(string $tempFile): mixed
+    {
+        return $this->openFileAsStream($tempFile);
+    }
+
+    public function publicCleanUpTempFile(string $tempFile): void
+    {
+        $this->cleanUpTempFile($tempFile);
+    }
+
+    public function publicLogInfo(string $message, array $context = []): void
+    {
+        $this->logInfo($message, $context);
+    }
+
+    public function publicLogDebug(string $message, array $context = []): void
+    {
+        $this->logDebug($message, $context);
+    }
+
+    public function publicLogWarning(string $message, array $context = []): void
+    {
+        $this->logWarning($message, $context);
+    }
+
+    public function publicLogError(string $message, array $context = []): void
+    {
+        $this->logError($message, $context);
+    }
+
+    public function publicLogCritical(string $message, array $context = []): void
+    {
+        $this->logCritical($message, $context);
+    }
+}