[Scheduler] Add AbstractTriggerDecorator

symfony · fabpot · Jul 31, 2023 · Jul 28, 2023 · Jul 28, 2023 · 3ca7eb3a5583d15ab1170c92362d76d0e03c56da
commit 3ca7eb3a5583d15ab1170c92362d76d0e03c56da
@@ -5,6 +5,7 @@ CHANGELOG
 ---
 
  * Allow setting timezone of next run date in CronExpressionTrigger
+ * Add `AbstractTriggerDecorator`
 
 6.3
 ---

@@ -0,0 +1,40 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Scheduler\Tests\Trigger;
+
+use PHPUnit\Framework\TestCase;
+use Symfony\Component\Scheduler\Trigger\ExcludeTimeTrigger;
+use Symfony\Component\Scheduler\Trigger\JitterTrigger;
+use Symfony\Component\Scheduler\Trigger\TriggerInterface;
+
+class AbstractDecoratedTriggerTest extends TestCase
+{
+    public function testCanGetInnerTrigger()
+    {
+        $trigger = new JitterTrigger($inner = $this->createMock(TriggerInterface::class));
+
+        $this->assertSame($inner, $trigger->inner());
+        $this->assertSame([$trigger], iterator_to_array($trigger->decorators()));
+    }
+
+    public function testCanGetNestedInnerTrigger()
+    {
+        $trigger = new ExcludeTimeTrigger(
+            $jitter = new JitterTrigger($inner = $this->createMock(TriggerInterface::class)),
+            new \DateTimeImmutable(),
+            new \DateTimeImmutable(),
+        );
+
+        $this->assertSame($inner, $trigger->inner());
+        $this->assertSame([$trigger, $jitter], iterator_to_array($trigger->decorators()));
+    }
+}
@@ -0,0 +1,49 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Scheduler\Trigger;
+
+/**
+ * @author Kevin Bond <kevinbond@gmail.com>
+ */
+abstract class AbstractDecoratedTrigger implements TriggerInterface
+{
+    public function __const
8000
ruct(private TriggerInterface $inner)
+    {
+    }
+
+    final public function inner(): TriggerInterface
+    {
+        $inner = $this->inner;
+
+        while ($inner instanceof self) {
+            $inner = $inner->inner;
+        }
+
+        return $inner;
+    }
+
+    /**
+     * @return \Traversable<self>
+     */
+    final public function decorators(): \Traversable
+    {
+        yield $this;
+
+        $inner = $this->inner;
+
+        while ($inner instanceof self) {
+            yield $inner;
+
+            $inner = $inner->inner;
+        }
+    }
+}
@@ -14,
8000
13 +14,14 @@
 /**
  * @experimental
  */
-final class ExcludeTimeTrigger implements TriggerInterface
+final class ExcludeTimeTrigger extends AbstractDecoratedTrigger
 {
     public function __construct(
         private readonly TriggerInterface $inner,
         private readonly \DateTimeImmutable $from,
         private readonly \DateTimeImmutable $until,
     ) {
+        parent::__construct($inner);
     }
 
     public function __toString(): string

@@ -14,13 +14,14 @@
 /**
  * @author Kevin Bond <kevinbond@gmail.com>
  */
-final class JitterTrigger implements TriggerInterface
+final class JitterTrigger extends AbstractDecoratedTrigger
 {
     /**
      * @param positive-int $maxSeconds
      */
     public function __construct(private readonly TriggerInterface $trigger, private readonly int $maxSeconds = 60)
     {
+        parent::__construct($trigger);
     }
 
     public function __toString(): string

@@ -16,7 +16,7 @@
 /**
  * @experimental
  */
-class PeriodicalTrigger implements TriggerInterface, \Stringable
+class PeriodicalTrigger implements TriggerInterface
 {
     private float $intervalInSeconds = 0.0;
     private \DateTimeImmutable $from;
-Original file line number
+Diff line change
@@ Expand Up / @@ -5,6 +5,7 @@ CHANGELOG @@
     ---
      * Allow setting timezone of next run date in CronExpressionTrigger
+     * Add `AbstractTriggerDecorator`
 .3
     ---
@@ Expand Down @@