diff --git a/Tests/EventListener/DispatchSchedulerEventListenerTest.php b/Tests/EventListener/DispatchSchedulerEventListenerTest.php index b0a1d3b..443b0ed 100644 --- a/Tests/EventListener/DispatchSchedulerEventListenerTest.php +++ b/Tests/EventListener/DispatchSchedulerEventListenerTest.php @@ -33,14 +33,14 @@ class DispatchSchedulerEventListenerTest extends TestCase { public function testDispatchSchedulerEvents() { - $trigger = $this->createMock(TriggerInterface::class); + $trigger = $this->createStub(TriggerInterface::class); $defaultRecurringMessage = RecurringMessage::trigger($trigger, (object) ['id' => 'default']); $schedulerProvider = new SomeScheduleProvider([$defaultRecurringMessage]); $scheduleProviderLocator = new Container(); $scheduleProviderLocator->set('default', $schedulerProvider); - $context = new MessageContext('default', 'default', $trigger, $this->createMock(\DateTimeImmutable::class)); + $context = new MessageContext('default', 'default', $trigger, new \DateTimeImmutable()); $envelope = (new Envelope(new \stdClass()))->with(new ScheduledStamp($context)); $listener = new DispatchSchedulerEventListener($scheduleProviderLocator, $eventDispatcher = new EventDispatcher()); diff --git a/Tests/Generator/MessageGeneratorTest.php b/Tests/Generator/MessageGeneratorTest.php index f8aa06f..8b243e3 100644 --- a/Tests/Generator/MessageGeneratorTest.php +++ b/Tests/Generator/MessageGeneratorTest.php @@ -366,7 +366,7 @@ class MessageGeneratorTest extends TestCase sort($runs); $ticks = [self::makeDateTime(''), 0]; - $trigger = $this->createMock(TriggerInterface::class); + $trigger = $this->createStub(TriggerInterface::class); $trigger ->method('getNextRunDate') ->willReturnCallback(function (\DateTimeImmutable $lastTick) use ($runs, &$ticks): \DateTimeImmutable { diff --git a/Tests/Messenger/SchedulerTransportFactoryTest.php b/Tests/Messenger/SchedulerTransportFactoryTest.php index 9a8132b..63c562e 100644 --- a/Tests/Messenger/SchedulerTransportFactoryTest.php +++ b/Tests/Messenger/SchedulerTransportFactoryTest.php @@ -12,8 +12,8 @@ namespace Symfony\Component\Scheduler\Tests\Messenger; use PHPUnit\Framework\TestCase; -use Psr\Clock\ClockInterface; use Psr\Container\ContainerInterface; +use Symfony\Component\Clock\MockClock; use Symfony\Component\Messenger\Transport\Serialization\SerializerInterface; use Symfony\Component\Scheduler\Exception\InvalidArgumentException; use Symfony\Component\Scheduler\Generator\MessageGenerator; @@ -29,9 +29,9 @@ class SchedulerTransportFactoryTest extends TestCase { public function testCreateTransport() { - $trigger = $this->createMock(TriggerInterface::class); - $serializer = $this->createMock(SerializerInterface::class); - $clock = $this->createMock(ClockInterface::class); + $trigger = $this->createStub(TriggerInterface::class); + $serializer = $this->createStub(SerializerInterface::class); + $clock = new MockClock(); $defaultRecurringMessage = RecurringMessage::trigger($trigger, (object) ['id' => 'default']); $customRecurringMessage = RecurringMessage::trigger($trigger, (object) ['id' => 'custom']); @@ -58,7 +58,7 @@ class SchedulerTransportFactoryTest extends TestCase $this->expectException(InvalidArgumentException::class); $this->expectExceptionMessage('The given Schedule DSN "schedule://#wrong" is invalid.'); - $factory->createTransport('schedule://#wrong', [], $this->createMock(SerializerInterface::class)); + $factory->createTransport('schedule://#wrong', [], $this->createStub(SerializerInterface::class)); } public function testNoName() @@ -68,7 +68,7 @@ class SchedulerTransportFactoryTest extends TestCase $this->expectException(InvalidArgumentException::class); $this->expectExceptionMessage('The Schedule DSN must contains a name, e.g. "schedule://default".'); - $factory->createTransport('schedule://', [], $this->createMock(SerializerInterface::class)); + $factory->createTransport('schedule://', [], $this->createStub(SerializerInterface::class)); } public function testNotFound() @@ -78,7 +78,7 @@ class SchedulerTransportFactoryTest extends TestCase $this->expectException(InvalidArgumentException::class); $this->expectExceptionMessage('The schedule "not-exists" is not found.'); - $factory->createTransport('schedule://not-exists', [], $this->createMock(SerializerInterface::class)); + $factory->createTransport('schedule://not-exists', [], $this->createStub(SerializerInterface::class)); } public function testSupports() @@ -95,9 +95,9 @@ class SchedulerTransportFactoryTest extends TestCase { return new SchedulerTransportFactory( new Container([ - 'default' => fn () => $this->createMock(ScheduleProviderInterface::class), + 'default' => fn () => $this->createStub(ScheduleProviderInterface::class), ]), - $this->createMock(ClockInterface::class), + new MockClock(), ); } } diff --git a/Tests/Messenger/SchedulerTransportTest.php b/Tests/Messenger/SchedulerTransportTest.php index d13e5bc..d3981ae 100644 --- a/Tests/Messenger/SchedulerTransportTest.php +++ b/Tests/Messenger/SchedulerTransportTest.php @@ -29,9 +29,9 @@ class SchedulerTransportTest extends TestCase (object) ['id' => 'first'], (object) ['id' => 'second'], ]; - $generator = $this->createMock(MessageGeneratorInterface::class); + $generator = $this->createStub(MessageGeneratorInterface::class); $generator->method('getMessages')->willReturnCallback(function () use ($messages): \Generator { - $trigger = $this->createMock(TriggerInterface::class); + $trigger = $this->createStub(TriggerInterface::class); $triggerAt = new \DateTimeImmutable('2020-02-20T02:00:00', new \DateTimeZone('UTC')); yield (new MessageContext('default', 'id1', $trigger, $triggerAt)) => $messages[0]; yield (new MessageContext('default', 'id2', $trigger, $triggerAt)) => $messages[1]; @@ -51,9 +51,9 @@ class SchedulerTransportTest extends TestCase public function testAddsStampToInnerRedispatchMessageEnvelope() { - $generator = $this->createMock(MessageGeneratorInterface::class); + $generator = $this->createStub(MessageGeneratorInterface::class); $generator->method('getMessages')->willReturnCallback(function (): \Generator { - yield new MessageContext('default', 'id', $this->createMock(TriggerInterface::class), new \DateTimeImmutable()) => new RedispatchMessage(new \stdClass(), ['transport']); + yield new MessageContext('default', 'id', $this->createStub(TriggerInterface::class), new \DateTimeImmutable()) => new RedispatchMessage(new \stdClass(), ['transport']); }); $envelopes = iterator_to_array((new SchedulerTransport($generator))->get()); @@ -66,7 +66,7 @@ class SchedulerTransportTest extends TestCase public function testAckIgnored() { - $transport = new SchedulerTransport($this->createMock(MessageGeneratorInterface::class)); + $transport = new SchedulerTransport($this->createStub(MessageGeneratorInterface::class)); $this->expectNotToPerformAssertions(); $transport->ack(new Envelope(new \stdClass())); @@ -74,7 +74,7 @@ class SchedulerTransportTest extends TestCase public function testRejectException() { - $transport = new SchedulerTransport($this->createMock(MessageGeneratorInterface::class)); + $transport = new SchedulerTransport($this->createStub(MessageGeneratorInterface::class)); $this->expectNotToPerformAssertions(); $transport->reject(new Envelope(new \stdClass())); @@ -82,7 +82,7 @@ class SchedulerTransportTest extends TestCase public function testSendException() { - $transport = new SchedulerTransport($this->createMock(MessageGeneratorInterface::class)); + $transport = new SchedulerTransport($this->createStub(MessageGeneratorInterface::class)); $this->expectException(LogicException::class); $transport->send(new Envelope(new \stdClass())); diff --git a/Tests/Trigger/AbstractDecoratedTriggerTest.php b/Tests/Trigger/AbstractDecoratedTriggerTest.php index 331b43b..7e42dfb 100644 --- a/Tests/Trigger/AbstractDecoratedTriggerTest.php +++ b/Tests/Trigger/AbstractDecoratedTriggerTest.php @@ -20,7 +20,7 @@ class AbstractDecoratedTriggerTest extends TestCase { public function testCanGetInnerTrigger() { - $trigger = new JitterTrigger($inner = $this->createMock(TriggerInterface::class)); + $trigger = new JitterTrigger($inner = $this->createStub(TriggerInterface::class)); $this->assertSame($inner, $trigger->inner()); $this->assertSame([$trigger], iterator_to_array($trigger->decorators())); @@ -29,7 +29,7 @@ class AbstractDecoratedTriggerTest extends TestCase public function testCanGetNestedInnerTrigger() { $trigger = new ExcludeTimeTrigger( - $jitter = new JitterTrigger($inner = $this->createMock(TriggerInterface::class)), + $jitter = new JitterTrigger($inner = $this->createStub(TriggerInterface::class)), new \DateTimeImmutable(), new \DateTimeImmutable(), ); diff --git a/Tests/Trigger/CallbackMessageProviderTest.php b/Tests/Trigger/CallbackMessageProviderTest.php index 7f7af0e..42e123c 100644 --- a/Tests/Trigger/CallbackMessageProviderTest.php +++ b/Tests/Trigger/CallbackMessageProviderTest.php @@ -20,7 +20,7 @@ class CallbackMessageProviderTest extends TestCase { public function testToString() { - $context = new MessageContext('test', 'test', $this->createMock(TriggerInterface::class), $this->createMock(\DateTimeImmutable::class)); + $context = new MessageContext('test', 'test', $this->createStub(TriggerInterface::class), new \DateTimeImmutable()); $messageProvider = new CallbackMessageProvider(static fn () => []); $this->assertEquals([], $messageProvider->getMessages($context)); $this->assertEquals('', $messageProvider->getId()); diff --git a/Tests/Trigger/ExcludeTimeTriggerTest.php b/Tests/Trigger/ExcludeTimeTriggerTest.php index 4245806..e03aa45 100644 --- a/Tests/Trigger/ExcludeTimeTriggerTest.php +++ b/Tests/Trigger/ExcludeTimeTriggerTest.php @@ -19,7 +19,7 @@ class ExcludeTimeTriggerTest extends TestCase { public function testGetNextRun() { - $inner = $this->createMock(TriggerInterface::class); + $inner = $this->createStub(TriggerInterface::class); $inner ->method('getNextRunDate') ->willReturnCallback(static fn (\DateTimeImmutable $d) => $d->modify('+1 sec')); diff --git a/Tests/Trigger/JitterTriggerTest.php b/Tests/Trigger/JitterTriggerTest.php index e794bd0..3af9076 100644 --- a/Tests/Trigger/JitterTriggerTest.php +++ b/Tests/Trigger/JitterTriggerTest.php @@ -20,7 +20,7 @@ class JitterTriggerTest extends TestCase public function testCanAddJitter() { $time = new \DateTimeImmutable(); - $inner = $this->createMock(TriggerInterface::class); + $inner = $this->createStub(TriggerInterface::class); $inner->method('getNextRunDate')->willReturn($time); $trigger = new JitterTrigger($inner);