more tests

src/Commands/SchedulerRunInvocation.php → src/Commands/SchedulerRunInvocationCommand.php

@@ -9,7 +9,7 @@
 use Tempest\Console\ConsoleOutput;
 use Tempest\Container\Container;
 
-final class SchedulerRunInvocation
+final class SchedulerRunInvocationCommand
 {
     public const string NAME = 'scheduler:invoke';
 

src/Scheduler/GenericScheduler.php

@@ -5,7 +5,7 @@
 namespace Tempest\Console\Scheduler;
 
 use DateTime;
-use Tempest\Console\Commands\SchedulerRunInvocation;
+use Tempest\Console\Commands\SchedulerRunInvocationCommand;
 
 final class GenericScheduler implements Scheduler
 {
@@ -38,7 +38,7 @@ private function execute(ScheduledInvocation $invocation): void
     private function compileInvocation(ScheduledInvocation $invocation): string
     {
         $commandName = $invocation->invocation instanceof HandlerInvocation ?
-            SchedulerRunInvocation::NAME . ' ' . $invocation->invocation->getName()
+            SchedulerRunInvocationCommand::NAME . ' ' . $invocation->invocation->getName()
             : $invocation->invocation->getName();
 
         return join(' ', [

tests/Scheduler/GenericSchedulerTest.php

@@ -20,7 +20,7 @@
  */
 final class GenericSchedulerTest extends TestCase
 {
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -89,6 +89,8 @@ public function test_scheduler_only_dispatches_the_command_in_desired_times()
         // command won't run twice in a row
         $scheduler->run($at);
 
+        // nor when it's called before the next minute
+        $scheduler->run($at->modify('+30 seconds'));
 
         $executor = $this->createMock(NullInvocationExecutor::class);
 
@@ -101,13 +103,13 @@ public function test_scheduler_only_dispatches_the_command_in_desired_times()
         $scheduler->run($at->modify('+1 minute'));
     }
 
-    public function handler()
+    // dummy handler for testing
+    public function handler(): void
     {
-
     }
 
-    public function command()
+    // dummy command for testing
+    public function command(): void
     {
-
     }
 }

tests/Scheduler/HandlerInvocationTest.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Tempest\Console\Scheduler;
+
+use PHPUnit\Framework\TestCase;
+use ReflectionMethod;
+use Tempest\Console\Scheduler\HandlerInvocation;
+
+/**
+ * @internal
+ * @small
+ */
+final class HandlerInvocationTest extends TestCase
+{
+    public function test_name_gets_constructed_properly(): void
+    {
+        $invocation = new HandlerInvocation(
+            new ReflectionMethod(
+                objectOrMethod: $this,
+                method: 'handler',
+            ),
+        );
+
+        $this->assertSame('Tests\\\Tempest\\\Console\\\Scheduler\\\HandlerInvocationTest::handler', $invocation->getName());
+    }
+
+    // dummy handler method
+    public function handler()
+    {
+
+    }
+}

tests/Scheduler/SchedulerInvokeCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Tempest\Console\Scheduler;
+
+use App\Console\ScheduledCommand;
+use Tempest\Console\Commands\SchedulerRunInvocationCommand;
+use Tests\Tempest\Console\TestCase;
+
+/**
+ * @internal
+ * @small
+ */
+final class SchedulerInvokeCommandTest extends TestCase
+{
+    public function test_scheduler_invoke_command_executes_handler()
+    {
+        $this->console->call(SchedulerRunInvocationCommand::NAME . ' ' . (ScheduledCommand::class . '::method'))
+            ->assertContains('method got scheduled');
+    }
+}