Skip to content

Commit 6c5b78e

Browse files
committed
do not mock event classes
1 parent c59bb66 commit 6c5b78e

File tree

3 files changed

+11
-18
lines changed

3 files changed

+11
-18
lines changed

Tests/EventListener/AddRequestFormatsListenerTest.php

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
use Symfony\Component\HttpFoundation\Request;
1717
use Symfony\Component\HttpKernel\Event\RequestEvent;
1818
use Symfony\Component\HttpKernel\EventListener\AddRequestFormatsListener;
19+
use Symfony\Component\HttpKernel\HttpKernelInterface;
1920
use Symfony\Component\HttpKernel\KernelEvents;
2021

2122
/**
@@ -54,22 +55,12 @@ public function testRegisteredEvent()
5455
public function testSetAdditionalFormats()
5556
{
5657
$request = $this->createMock(Request::class);
57-
$event = $this->getRequestEventMock($request);
58+
$event = new RequestEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MASTER_REQUEST);
5859

5960
$request->expects($this->once())
6061
->method('setFormat')
6162
->with('csv', ['text/csv', 'text/plain']);
6263

6364
$this->listener->onKernelRequest($event);
6465
}
65-
66-
protected function getRequestEventMock(Request $request)
67-
{
68-
$event = $this->createMock(RequestEvent::class);
69-
$event->expects($this->any())
70-
->method('getRequest')
71-
->willReturn($request);
72-
73-
return $event;
74-
}
7566
}

Tests/EventListener/SaveSessionListenerTest.php

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,17 @@ class SaveSessionListenerTest extends TestCase
2626
{
2727
public function testOnlyTriggeredOnMasterRequest()
2828
{
29+
$session = $this->createMock(SessionInterface::class);
30+
$session->expects($this->never())->method('save');
31+
$session->expects($this->any())->method('isStarted')->willReturn(true);
32+
33+
$request = new Request();
34+
$request->setSession($session);
35+
2936
$listener = new SaveSessionListener();
30-
$event = $this->createMock(ResponseEvent::class);
31-
$event->expects($this->once())->method('isMasterRequest')->willReturn(false);
32-
$event->expects($this->never())->method('getRequest');
3337

3438
// sub request
35-
$listener->onKernelResponse($event);
39+
$listener->onKernelResponse(new ResponseEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::SUB_REQUEST, new Response()));
3640
}
3741

3842
public function testSessionSaved()

Tests/EventListener/SessionListenerTest.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,9 +58,7 @@ public function testSessionIsSet()
5858
$request = new Request();
5959
$listener = new SessionListener($container);
6060

61-
$event = $this->createMock(RequestEvent::class);
62-
$event->expects($this->exactly(2))->method('isMasterRequest')->willReturn(true);
63-
$event->expects($this->once())->method('getRequest')->willReturn($request);
61+
$event = new RequestEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MASTER_REQUEST);
6462

6563
$listener->onKernelRequest($event);
6664

0 commit comments

Comments
 (0)