API Update API to reflect changes to CLI interaction

src/Extensions/QueuedJobDescriptorExtension.php

@@ -12,7 +12,7 @@
 class QueuedJobDescriptorExtension extends Extension
 {
     /**
-     * Called on dev/build by DatabaseAdmin
+     * Called by DbBuild
      */
     protected function onAfterBuild(): void
     {

src/Tasks/GarbageCollectionTask.php

@@ -3,31 +3,22 @@
 namespace SilverStripe\Tasks;
 
 use SilverStripe\Dev\BuildTask;
+use SilverStripe\PolyExecution\PolyOutput;
 use SilverStripe\SessionManager\Services\GarbageCollectionService;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
 
 class GarbageCollectionTask extends BuildTask
 {
-    /**
-     * @var string
-     */
-    private static $segment = 'LoginSessionGarbageCollectionTask';
+    protected static string $commandName = 'LoginSessionGarbageCollectionTask';
 
-    /**
-     * @var string
-     */
-    protected $title = 'Login Session Garbage Collection Task';
+    protected string $title = 'Login Session Garbage Collection Task';
 
-    /**
-     * @var string
-     */
-    protected $description = 'Removes expired login sessions and “remember me” hashes from the database';
+    protected static string $description = 'Removes expired login sessions and "remember me" hashes from the database';
 
-    /**
-     * @param HTTPRequest $request
-     */
-    public function run($request)
+    protected function execute(InputInterface $input, PolyOutput $output): int
     {
         GarbageCollectionService::singleton()->collect();
-        echo "Garbage collection completed successfully\n";
+        return Command::SUCCESS;
     }
 }

src/Tasks/InvalidateAllSessionsTask.php

@@ -2,32 +2,26 @@
 
 namespace SilverStripe\SessionManager\Tasks;
 
-use SilverStripe\Control\HTTPRequest;
 use SilverStripe\Dev\BuildTask;
+use SilverStripe\PolyExecution\PolyOutput;
 use SilverStripe\Security\RememberLoginHash;
 use SilverStripe\SessionManager\Models\LoginSession;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
 
 class InvalidateAllSessionsTask extends BuildTask
 {
-    private static string $segment = 'InvalidateAllSessions';
+    protected static string $commandName = 'InvalidateAllSessions';
 
-    /**
-     * @var string
-     */
-    protected $title = 'Invalidate All Login Sessions Task';
+    protected string $title = 'Invalidate All Login Sessions Task';
 
-    /**
-     * @var string
-     */
-    protected $description = 'Removes all login sessions and "remember me" hashes (including yours) from the database';
+    protected static string $description = 'Removes all login sessions and "remember me" hashes'
+                                            . ' (including yours) from the database';
 
-    /**
-     * @param HTTPRequest $request
-     */
-    public function run($request)
+    protected function execute(InputInterface $input, PolyOutput $output): int
     {
         LoginSession::get()->removeAll();
         RememberLoginHash::get()->removeAll();
-        echo "Session removal completed successfully\n";
+        return Command::SUCCESS;
     }
 }

tests/php/Tasks/InvalidateAllSessionsTaskTest.php

@@ -6,6 +6,7 @@
 use SilverStripe\Control\Tests\HttpRequestMockBuilder;
 use SilverStripe\Core\Injector\Injector;
 use SilverStripe\Dev\SapphireTest;
+use SilverStripe\PolyExecution\PolyOutput;
 use SilverStripe\ORM\FieldType\DBDatetime;
 use SilverStripe\Security\AuthenticationHandler;
 use SilverStripe\Security\Member;
@@ -15,6 +16,8 @@
 use SilverStripe\SessionManager\Middleware\LoginSessionMiddleware;
 use SilverStripe\SessionManager\Models\LoginSession;
 use SilverStripe\SessionManager\Tasks\InvalidateAllSessionsTask;
+use Symfony\Component\Console\Input\ArrayInput;
+use Symfony\Component\Console\Output\BufferedOutput;
 
 class InvalidateAllSessionsTaskTest extends SapphireTest
 {
@@ -46,7 +49,11 @@ public function testRemoveAllSessions()
 
         // Completely invalidate all current sessions and login hashes
         $task = new InvalidateAllSessionsTask();
-        $task->run($request);
+        $buffer = new BufferedOutput();
+        $output = new PolyOutput(PolyOutput::FORMAT_ANSI, wrappedOutput: $buffer);
+        $input = new ArrayInput([]);
+        $input->setInteractive(false);
+        $task->run($input, $output);
 
         // Assert all sessions are removed from the database
         $this->assertSame(0, LoginSession::get()->count(), 'There are no LoginSessions');