@@ -560,22 +560,27 @@ public function testUninitializedSessionWithoutInitializedSession()
560
560
public function testResponseHeadersMaxAgeAndExpiresNotBeOverridenIfSessionStarted ()
561
561
{
562
562
$ 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 );
564
567
565
568
$ container = new Container ();
566
- $ container ->set ('initialized_session ' , $ session );
569
+ $ container ->set ('session_factory ' , $ sessionFactory );
567
570
568
571
$ listener = new SessionListener ($ container );
569
572
$ kernel = $ this ->createMock (HttpKernelInterface::class);
570
573
571
574
$ request = new Request ();
572
575
$ listener ->onKernelRequest (new RequestEvent ($ kernel , $ request , HttpKernelInterface::MAIN_REQUEST ));
573
576
577
+ $ request ->getSession ();
578
+
574
579
$ response = new Response ();
575
580
$ response ->setPrivate ();
576
581
$ expiresHeader = gmdate ('D, d M Y H:i:s ' , time () + 600 ).' GMT ' ;
577
582
$ 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 ));
579
584
580
585
$ this ->assertTrue ($ response ->headers ->has ('expires ' ));
581
586
$ this ->assertSame ($ expiresHeader , $ response ->headers ->get ('expires ' ));
@@ -590,20 +595,20 @@ public function testResponseHeadersMaxAgeAndExpiresNotBeOverridenIfSessionStarte
590
595
public function testResponseHeadersMaxAgeAndExpiresDefaultValuesIfSessionStarted ()
591
596
{
592
597
$ 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 );
594
599
595
600
$ container = new Container ();
596
- $ container ->set ('initialized_session ' , $ session );
597
601
598
602
$ listener = new SessionListener ($ container );
599
603
$ kernel = $ this ->createMock (HttpKernelInterface::class);
600
604
601
605
$ request = new Request ();
606
+ $ request ->setSession ($ session );
602
607
$ listener ->onKernelRequest (new RequestEvent ($ kernel , $ request , HttpKernelInterface::MAIN_REQUEST ));
603
608
604
609
$ response = new Response ();
605
610
$ 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 ));
607
612
608
613
$ this ->assertTrue ($ response ->headers ->has ('expires ' ));
609
614
$ this ->assertSame ($ expiresHeader , $ response ->headers ->get ('expires ' ));
0 commit comments