Add more tests

Author: NeoIsRecursive

src/Console/Commands/RestoreCommand.php

@@ -5,7 +5,6 @@
 namespace Itiden\Backup\Console\Commands;
 
 use Illuminate\Console\Command;
-use Illuminate\Contracts\Console\PromptsForMissingInput;
 use Itiden\Backup\Contracts\Repositories\BackupRepository;
 use Itiden\Backup\DataTransferObjects\BackupDto;
 use Itiden\Backup\Facades\Restorer;
@@ -15,7 +14,7 @@
 /**
  * Restore content from a directory / backup
  */
-class RestoreCommand extends Command implements PromptsForMissingInput
+final class RestoreCommand extends Command
 {
     protected $signature = 'statamic:backup:restore {--path=} {--force}';
 
@@ -31,15 +30,17 @@ public function handle(BackupRepository $repo)
                 scroll: 10,
                 options: $repo->all()->flatMap(
                     fn (BackupDto $backup) => [$backup->path => $backup->path]
-                )
+                ),
+                required: true
             )),
         };
 
         if (
             $this->option('force')
             || confirm(
                 label: "Are you sure you want to restore your content?",
-                hint: "This will overwrite your current content with state from {$backup->created_at->format('Y-m-d H:i:s')}"
+                hint: "This will overwrite your current content with state from {$backup->created_at->format('Y-m-d H:i:s')}",
+                required: true
             )
         ) {
             spin(fn () => Restorer::restore($backup), 'Restoring backup');

tests/Feature/RestoreCommandTest.php

@@ -0,0 +1,35 @@
+<?php
+
+use Illuminate\Support\Facades\Storage;
+use Itiden\Backup\Contracts\Repositories\BackupRepository;
+use Itiden\Backup\Facades\Backuper;
+
+use function Pest\Laravel\artisan;
+
+uses()->group('restore-command');
+
+it('shows all available backups', function () {
+    app(BackupRepository::class)->empty();
+
+    Backuper::backup();
+
+    $backups = app(BackupRepository::class)->all();
+
+    artisan('statamic:backup:restore')
+        ->expectsQuestion(
+            question: 'Which backup do you want to restore to?',
+            answer: $backups->first()->path
+        )
+        ->expectsConfirmation('Are you sure you want to restore your content?')
+        ->assertFailed();
+});
+
+it('can restore from a specific path', function () {
+    app(BackupRepository::class)->empty();
+
+    $backup = Backuper::backup();
+
+    artisan('statamic:backup:restore', ['--path' => Storage::disk(config('backup.destination.disk'))->path($backup->path)])
+        ->expectsConfirmation('Are you sure you want to restore your content?')
+        ->assertFailed();
+});

tests/Unit/PipeTest.php

@@ -43,7 +43,7 @@
     Assets::class,
 ]);
 
-test('can skip a pipe', function () {
+test('can skip a pipe with users', function () {
     /** @var Users::class $pipe */
     $pipe = app()->make(Users::class);
 
@@ -63,3 +63,28 @@
 
     $zipper->close();
 });
+
+test('can skip a pipe with content', function () {
+    /** @var Users::class $pipe */
+    $pipe = app()->make(Content::class);
+
+    $callable = function ($z) {
+        return $z;
+    };
+
+    File::copyDirectory(config('backup.content_path'), config('backup.content_path') . '_backup');
+    File::deleteDirectory(config('backup.content_path'));
+
+    $zipper = Zipper::open(config('backup.temp_path') . '/backup.zip');
+
+    $pipe->backup(zip: $zipper, next: $callable);
+
+
+    expect($zipper->getMeta())->toHaveKey(Content::class);
+    expect($zipper->getMeta()[Content::class])->toHaveKey('skipped', 'Content directory didn\'t exist, is it configured correctly?');
+
+    $zipper->close();
+
+    File::copyDirectory(config('backup.content_path') . '_backup', config('backup.content_path'));
+    File::deleteDirectory(config('backup.content_path') . '_backup');
+});