No effects for no store

alexander-schranz · alexander-schranz · commit 252be508d628 · 2025-05-28T09:44:19.000+02:00
diff --git a/src/Symfony/Component/HttpKernel/Tests/EventListener/CacheAttributeListenerTest.php b/src/Symfony/Component/HttpKernel/Tests/EventListener/CacheAttributeListenerTest.php
@@ -102,18 +102,18 @@ public function testResponseIsPublicIfConfigurationIsPublicTrueNoStoreFalse()
         $this->assertFalse($this->response->headers->hasCacheControlDirective('no-store'));
     }
 
-    public function testResponseIsPrivateIfConfigurationIsPublicTrueNoStoreTrue()
+    public function testResponseKeepPublicIfConfigurationIsPublicTrueNoStoreTrue()
     {
         $request = $this->createRequest(new Cache(public: true, noStore: true));
 
         $this->listener->onKernelResponse($this->createEventMock($request, $this->response));
 
-        $this->assertFalse($this->response->headers->hasCacheControlDirective('public'));
-        $this->assertTrue($this->response->headers->hasCacheControlDirective('private'));
+        $this->assertTrue($this->response->headers->hasCacheControlDirective('public'));
+        $this->assertFalse($this->response->headers->hasCacheControlDirective('private'));
         $this->assertTrue($this->response->headers->hasCacheControlDirective('no-store'));
     }
 
-    public function testResponseIsPrivateNoStoreIfConfigurationIsNoStoreTrue()
+    public function testResponseHasNothingNoStoreIfConfigurationIsNoStoreTrue()
     {
         $request = $this->createRequest(new Cache(noStore: true));
 
@@ -124,14 +124,14 @@ public function testResponseIsPrivateNoStoreIfConfigurationIsNoStoreTrue()
         $this->assertTrue($this->response->headers->hasCacheControlDirective('no-store'));
     }
 
-    public function testResponseIsPrivateIfSharedMaxAgeSetAndNoStoreIsTrue()
+    public function testResponseIsPublicIfSharedMaxAgeSetAndNoStoreIsTrue()
     {
         $request = $this->createRequest(new Cache(smaxage: 1, noStore: true));
 
         $this->listener->onKernelResponse($this->createEventMock($request, $this->response));
 
-        $this->assertFalse($this->response->headers->hasCacheControlDirective('public'));
-        $this->assertTrue($this->response->headers->hasCacheControlDirective('private'));
+        $this->assertTrue($this->response->headers->hasCacheControlDirective('public'));
+        $this->assertFalse($this->response->headers->hasCacheControlDirective('private'));
         $this->assertTrue($this->response->headers->hasCacheControlDirective('no-store'));
     }
 

Original file line number	Diff line number	Diff line change
`@@ -102,18 +102,18 @@ public function testResponseIsPublicIfConfigurationIsPublicTrueNoStoreFalse()`
`102`	`102`	`$this->assertFalse($this->response->headers->hasCacheControlDirective('no-store'));`
`103`	`103`	`}`
`104`	`104`
`105`		`- public function testResponseIsPrivateIfConfigurationIsPublicTrueNoStoreTrue()`
	`105`	`+ public function testResponseKeepPublicIfConfigurationIsPublicTrueNoStoreTrue()`
`106`	`106`	`{`
`107`	`107`	`$request = $this->createRequest(new Cache(public: true, noStore: true));`
`108`	`108`
`109`	`109`	`$this->listener->onKernelResponse($this->createEventMock($request, $this->response));`
`110`	`110`
`111`		`- $this->assertFalse($this->response->headers->hasCacheControlDirective('public'));`
`112`		`- $this->assertTrue($this->response->headers->hasCacheControlDirective('private'));`
	`111`	`+ $this->assertTrue($this->response->headers->hasCacheControlDirective('public'));`
	`112`	`+ $this->assertFalse($this->response->headers->hasCacheControlDirective('private'));`
`113`	`113`	`$this->assertTrue($this->response->headers->hasCacheControlDirective('no-store'));`
`114`	`114`	`}`
`115`	`115`
`116`		`- public function testResponseIsPrivateNoStoreIfConfigurationIsNoStoreTrue()`
	`116`	`+ public function testResponseHasNothingNoStoreIfConfigurationIsNoStoreTrue()`
`117`	`117`	`{`
`118`	`118`	`$request = $this->createRequest(new Cache(noStore: true));`
`119`	`119`
`@@ -124,14 +124,14 @@ public function testResponseIsPrivateNoStoreIfConfigurationIsNoStoreTrue()`
`124`	`124`	`$this->assertTrue($this->response->headers->hasCacheControlDirective('no-store'));`
`125`	`125`	`}`
`126`	`126`
`127`		`- public function testResponseIsPrivateIfSharedMaxAgeSetAndNoStoreIsTrue()`
	`127`	`+ public function testResponseIsPublicIfSharedMaxAgeSetAndNoStoreIsTrue()`
`128`	`128`	`{`
`129`	`129`	`$request = $this->createRequest(new Cache(smaxage: 1, noStore: true));`
`130`	`130`
`131`	`131`	`$this->listener->onKernelResponse($this->createEventMock($request, $this->response));`
`132`	`132`
`133`		`- $this->assertFalse($this->response->headers->hasCacheControlDirective('public'));`
`134`		`- $this->assertTrue($this->response->headers->hasCacheControlDirective('private'));`
	`133`	`+ $this->assertTrue($this->response->headers->hasCacheControlDirective('public'));`
	`134`	`+ $this->assertFalse($this->response->headers->hasCacheControlDirective('private'));`
`135`	`135`	`$this->assertTrue($this->response->headers->hasCacheControlDirective('no-store'));`
`136`	`136`	`}`
`137`	`137`