Skip to content

Commit 5529d29

Browse files
[HttpKernel] fix merge
1 parent f81630f commit 5529d29

File tree

1 file changed

+11
-6
lines changed

1 file changed

+11
-6
lines changed

Tests/EventListener/SessionListenerTest.php

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -560,22 +560,27 @@ public function testUninitializedSessionWithoutInitializedSession()
560560
public function testResponseHeadersMaxAgeAndExpiresNotBeOverridenIfSessionStarted()
561561
{
562562
$session = $this->createMock(Session::class);
563-
$session->expects($this->exactly(2))->method('getUsageIndex')->will($this->onConsecutiveCalls(0, 1));
563+
$session->expects($this->once())->method('getUsageIndex')->willReturn(1);
564+
$session->expects($this->once())->method('getName')->willReturn('foo');
565+
$sessionFactory = $this->createMock(SessionFactory::class);
566+
$sessionFactory->expects($this->once())->method('createSession')->willReturn($session);
564567

565568
$container = new Container();
566-
$container->set('initialized_session', $session);
569+
$container->set('session_factory', $sessionFactory);
567570

568571
$listener = new SessionListener($container);
569572
$kernel = $this->createMock(HttpKernelInterface::class);
570573

571574
$request = new Request();
572575
$listener->onKernelRequest(new RequestEvent($kernel, $request, HttpKernelInterface::MAIN_REQUEST));
573576

577+
$request->getSession();
578+
574579
$response = new Response();
575580
$response->setPrivate();
576581
$expiresHeader = gmdate('D, d M Y H:i:s', time() + 600).' GMT';
577582
$response->setMaxAge(600);
578-
$listener->onKernelResponse(new ResponseEvent($kernel, new Request(), HttpKernelInterface::MAIN_REQUEST, $response));
583+
$listener->onKernelResponse(new ResponseEvent($kernel, $request, HttpKernelInterface::MAIN_REQUEST, $response));
579584

580585
$this->assertTrue($response->headers->has('expires'));
581586
$this->assertSame($expiresHeader, $response->headers->get('expires'));
@@ -590,20 +595,20 @@ public function testResponseHeadersMaxAgeAndExpiresNotBeOverridenIfSessionStarte
590595
public function testResponseHeadersMaxAgeAndExpiresDefaultValuesIfSessionStarted()
591596
{
592597
$session = $this->createMock(Session::class);
593-
$session->expects($this->exactly(2))->method('getUsageIndex')->will($this->onConsecutiveCalls(0, 1));
598+
$session->expects($this->once())->method('getUsageIndex')->willReturn(1);
594599

595600
$container = new Container();
596-
$container->set('initialized_session', $session);
597601

598602
$listener = new SessionListener($container);
599603
$kernel = $this->createMock(HttpKernelInterface::class);
600604

601605
$request = new Request();
606+
$request->setSession($session);
602607
$listener->onKernelRequest(new RequestEvent($kernel, $request, HttpKernelInterface::MAIN_REQUEST));
603608

604609
$response = new Response();
605610
$expiresHeader = gmdate('D, d M Y H:i:s', time()).' GMT';
606-
$listener->onKernelResponse(new ResponseEvent($kernel, new Request(), HttpKernelInterface::MAIN_REQUEST, $response));
611+
$listener->onKernelResponse(new ResponseEvent($kernel, $request, HttpKernelInterface::MAIN_REQUEST, $response));
607612

608613
$this->assertTrue($response->headers->has('expires'));
609614
$this->assertSame($expiresHeader, $response->headers->get('expires'));

0 commit comments

Comments
 (0)