8000 minor #52085 [Scheduler] postRun event test is missing (alli83) · symfony/symfony@3fb0e11 · GitHub
[go: up one dir, main page]

Skip to content

Commit 3fb0e11

Browse files
committed
minor #52085 [Scheduler] postRun event test is missing (alli83)
This PR was merged into the 6.4 branch. Discussion ---------- [Scheduler] postRun event test is missing | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | no | Deprecations? | | Tickets | | License | MIT The PostRun event test is missing in `DispatchSchedulerEventListenerTest` and there is an error to be rectified Commits ------- c263424 [Scheduler] postRun event test is missing
2 parents 934aea0 + c263424 commit 3fb0e11

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

src/Symfony/Component/Scheduler/Tests/EventListener/DispatchSchedulerEventListenerTest.php

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,9 @@
1515
use Psr\Container\ContainerInterface;
1616
use Symfony\Component\EventDispatcher\EventDispatcher;
1717
use Symfony\Component\Messenger\Envelope;
18+
use Symfony\Component\Messenger\Event\WorkerMessageHandledEvent;
1819
use Symfony\Component\Messenger\Event\WorkerMessageReceivedEvent;
20+
use Symfony\Component\Scheduler\Event\PostRunEvent;
1921
use Symfony\Component\Scheduler\Event\PreRunEvent;
2022
use Symfony\Component\Scheduler\EventListener\DispatchSchedulerEventListener;
2123
use Symfony\Component\Scheduler\Generator\MessageContext;
@@ -33,20 +35,22 @@ public function testDispatchSchedulerEvents()
3335

3436
$schedulerProvider = new SomeScheduleProvider([$defaultRecurringMessage]);
3537
$scheduleProviderLocator = $this->createMock(ContainerInterface::class);
36-
$scheduleProviderLocator->expects($this->once())->method('has')->willReturn(true);
37-
$scheduleProviderLocator->expects($this->once())->method('get')->willReturn($schedulerProvider);
38+
$scheduleProviderLocator->expects($this->any())->method('has')->willReturn(true);
39+
$scheduleProviderLocator->expects($this->any())->method('get')->willReturn($schedulerProvider);
3840

3941
$context = new MessageContext('default', 'default', $trigger, $this->createMock(\DateTimeImmutable::class));
4042
$envelope = (new Envelope(new \stdClass()))->with(new ScheduledStamp($context));
4143

4244
/** @var ContainerInterface $scheduleProviderLocator */
4345
$listener = new DispatchSchedulerEventListener($scheduleProviderLocator, $eventDispatcher = new EventDispatcher());
4446
$workerReceivedEvent = new WorkerMessageReceivedEvent($envelope, 'default');
47+
$workerHandledEvent = new WorkerMessageHandledEvent($envelope, 'default');
4548
$secondListener = new TestEventListener();
4649

4750
$eventDispatcher->addListener(PreRunEvent::class, [$secondListener, 'preRun']);
48-
$eventDispatcher->addListener(PreRunEvent::class, [$secondListener, 'postRun']);
51+
$eventDispatcher->addListener(PostRunEvent::class, [$secondListener, 'postRun']);
4952
$listener->onMessageReceived($workerReceivedEvent);
53+
$listener->onMessageHandled($workerHandledEvent);
5054

5155
$this->assertTrue< 3EDC /span>($secondListener->preInvoked);
5256
$this->assertTrue($secondListener->postInvoked);

0 commit comments

Comments
 (0)
0