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($secondListener->preInvoked);
5256
$this->assertTrue($secondListener->postInvoked);

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy