Remove Every::SECOND

tempestphp · May 13, 2024 · bb31f81 · bb31f81
1 parent 4fb0505
commit bb31f81
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 15 deletions.
diff --git a/src/Scheduler/Every.php b/src/Scheduler/Every.php
@@ -6,7 +6,6 @@
 
 enum Every
 {
-    case SECOND;
     case MINUTE;
     case QUARTER;
     case HALF_HOUR;
@@ -20,7 +19,6 @@ enum Every
     public function toInterval(): Interval
     {
         return match ($this) {
-            self::SECOND => new Interval(seconds: 1),
             self::MINUTE => new Interval(minutes: 1),
             self::QUARTER => new Interval(minutes: 15),
             self::HALF_HOUR => new Interval(minutes: 30),

diff --git a/tests/Fixtures/ScheduledCommand.php b/tests/Fixtures/ScheduledCommand.php
@@ -16,14 +16,14 @@ public function __construct(
     ) {
     }
 
-    #[Schedule(Every::SECOND)]
+    #[Schedule(Every::MINUTE)]
     #[ConsoleCommand('scheduled')]
     public function command(): void
     {
         $this->console->writeln('A command got scheduled');
     }
 
-    #[Schedule(Every::SECOND)]
+    #[Schedule(Every::MINUTE)]
     public function method(): void
     {
         $this->console->writeln('A method got scheduled');

diff --git a/tests/Scheduler/EveryTest.php b/tests/Scheduler/EveryTest.php
@@ -13,15 +13,6 @@
  */
 final class EveryTest extends TestCase
 {
-    public function test_every_second_gets_transformed_to_interval()
-    {
-        $every = Every::SECOND;
-        $interval = $every->toInterval();
-
-        $this->assertSame(1, $interval->seconds);
-        $this->assertSame(1, $interval->inSeconds());
-    }
-
     public function test_every_minute_gets_transformed_to_interval()
     {
         $every = Every::MINUTE;

diff --git a/tests/Scheduler/GenericSchedulerTest.php b/tests/Scheduler/GenericSchedulerTest.php
@@ -41,7 +41,7 @@ public function test_scheduler_executes_handlers()
         $config = new SchedulerConfig();
         $config->addMethodInvocation(
             new ReflectionMethod($this, 'handler'),
-            new Schedule(Every::SECOND)
+            new Schedule(Every::MINUTE)
         );
 
         $scheduler = new GenericScheduler($config, $executor);
@@ -60,7 +60,7 @@ public function test_scheduler_executes_commands()
         $config->addCommandInvocation(
             new ReflectionMethod($this, 'command'),
             new ConsoleCommand('command'),
-            new Schedule(Every::SECOND)
+            new Schedule(Every::MINUTE)
         );
 
         $scheduler = new GenericScheduler($config, $executor);