Expect choice for failed-jobs command

Author: codebarista

src/Console/Commands/FailedJobs.php

@@ -16,16 +16,21 @@ class FailedJobs extends Command implements Isolatable, PromptsForMissingInput
     /**
      * @var string
      */
-    protected $signature = 'codebarista:failed-jobs {action : The action: retry or delete}';
+    protected $signature = 'codebarista:failed-jobs';
 
     /**
      * @var string
      */
-    protected $description = 'Retry or delete all failed jobs.';
+    protected $description = 'Retry or delete failed jobs';
 
     public function handle(RedisJobRepository $repository): int
     {
-        return match ($this->argument('action')) {
+        $action = $this->choice('What action should be taken?', [
+            'delete',
+            'retry',
+        ]);
+
+        return match ($action) {
             'delete' => $this->deleteFailedJobs($repository),
             'retry' => $this->retryFailedJobs($repository),
             default => self::INVALID,

tests/Feature/FailedJobsTest.php

@@ -2,37 +2,52 @@
 
 it('returns invalid', function () {
     $this->artisan('codebarista:failed-jobs')
-        ->expectsQuestion('What is the action: retry or delete?', 'invalid')
+        ->expectsChoice('What action should be taken?', 'invalid', [
+            'delete',
+            'retry',
+        ])
         ->assertExitCode(2);
 });
 
 it('aborts retry failed jobs', function () {
     $this->artisan('codebarista:failed-jobs')
-        ->expectsQuestion('What is the action: retry or delete?', 'retry')
+        ->expectsChoice('What action should be taken?', 'retry', [
+            'delete',
+            'retry',
+        ])
         ->expectsConfirmation('Really retry all failed jobs?')
         ->expectsOutputToContain('Process terminated by user')
         ->assertExitCode(0);
 });
 
 it('aborts delete failed jobs', function () {
     $this->artisan('codebarista:failed-jobs')
-        ->expectsQuestion('What is the action: retry or delete?', 'delete')
+        ->expectsChoice('What action should be taken?', 'delete', [
+            'delete',
+            'retry',
+        ])
         ->expectsConfirmation('Really delete all failed jobs?')
         ->expectsOutputToContain('Process terminated by user')
         ->assertExitCode(0);
 });
 
 it('retries failed jobs', function () {
     $this->artisan('codebarista:failed-jobs')
-        ->expectsQuestion('What is the action: retry or delete?', 'retry')
+        ->expectsChoice('What action should be taken?', 'retry', [
+            'delete',
+            'retry',
+        ])
         ->expectsConfirmation('Really retry all failed jobs?', 'yes')
         ->expectsOutputToContain('jobs retried')
         ->assertExitCode(0);
 });
 
 it('deletes failed jobs', function () {
     $this->artisan('codebarista:failed-jobs')
-        ->expectsQuestion('What is the action: retry or delete?', 'delete')
+        ->expectsChoice('What action should be taken?', 'delete', [
+            'delete',
+            'retry',
+        ])
         ->expectsConfirmation('Really delete all failed jobs?', 'yes')
         ->expectsOutputToContain('jobs deleted')
         ->assertExitCode(0);