Soothe SA

Added a property_exists function to the DonutCommandInterceptorTest class to ensure 'bindings' property exists before being accessed. In the ResourceStorage class, simplified the getVary method by directly using the 'X_VARY' server variable in the explode function instead of casting it to a string.

src/ResourceStorage.php

@@ -238,7 +238,7 @@ private function getUriKey(AbstractUri $uri, string $type): string
     private function getVary(): string
     {
         $xvary = $_SERVER['X_VARY'];
-        $varys = explode(',', (string) $xvary);
+        $varys = explode(',', $xvary);
         $varyString = '';
         foreach ($varys as $vary) {
             $phpVaryKey = sprintf('X_%s', strtoupper($vary));

tests/DonutCommandInterceptorTest.php

@@ -14,6 +14,7 @@
 use function array_map;
 use function assert;
 use function dirname;
+use function property_exists;
 
 class DonutCommandInterceptorTest extends TestCase
 {
@@ -82,6 +83,7 @@ public function testCacheableResponse(): void
         $ro = $this->resource->get('page://self/html/blog-posting-cache?id=0');
         $interceptors = array_map(static fn (object $object): string => $object::class, $ro->bindings['onGet']); // @phpstan-ignore-line
         $this->assertContains(DonutCacheInterceptor::class, $interceptors);
+        assert(property_exists($ro, 'bindings')); // @phpstan-ignore-line
         assert(isset($ro->bindings['onGet'][0]));
         assert(isset($ro->bindings['onDelete'][0]));
         $this->assertInstanceOf(DonutCacheInterceptor::class, $ro->bindings['onDelete'][0]);