Add missing handler for resource subscribe and unsubscribe

Author: bigdevlarry

src/Server/Session/Psr16StoreSession.php

@@ -114,7 +114,7 @@ public function getAllSessionIds(): array
             if (\count($validSessionIds) !== \count($sessionIdsData)) {
                 $this->cache->set(
                     self::SESSION_IDS_KEY,
-                    array_map(fn (Uuid $id) => $id->toRfc4122(), $validSessionIds),
+                    array_map(static fn (Uuid $id) => $id->toRfc4122(), $validSessionIds),
                     null
                 );
             }
@@ -156,7 +156,7 @@ private function removeSessionId(Uuid $id): void
             $idString = $id->toRfc4122();
             $sessionIds = array_values(array_filter(
                 $sessionIds,
-                fn ($sid) => $sid !== $idString
+                static fn ($sid) => $sid !== $idString
             ));
 
             $this->cache->set(self::SESSION_IDS_KEY, $sessionIds, null);

tests/Unit/Server/SessionSubscriptionManagerTest.php

@@ -76,7 +76,7 @@ public function testSubscribeAndSendsNotification(): void
 
         $this->sessionStore->expects($this->exactly(2))
             ->method('read')
-            ->willReturnCallback(function ($id) use ($uuid1, $uuid2, $sessionData1, $sessionData2) {
+            ->willReturnCallback(static function ($id) use ($uuid1, $uuid2, $sessionData1, $sessionData2) {
                 if ($id === $uuid1) {
                     return $sessionData1;
                 }
@@ -89,7 +89,7 @@ public function testSubscribeAndSendsNotification(): void
 
         $this->sessionFactory->expects($this->exactly(2))
             ->method('createWithId')
-            ->willReturnCallback(function ($id) use ($uuid1, $uuid2, $session1, $session2) {
+            ->willReturnCallback(static function ($id) use ($uuid1, $uuid2, $session1, $session2) {
                 if ($id === $uuid1) {
                     return $session1;
                 }
@@ -102,7 +102,7 @@ public function testSubscribeAndSendsNotification(): void
 
         $this->protocol->expects($this->exactly(2))
             ->method('sendNotification')
-            ->with($this->callback(function ($notification) use ($uri) {
+            ->with($this->callback(static function ($notification) use ($uri) {
                 return $notification instanceof ResourceUpdatedNotification
                     && $notification->uri === $uri;
             }));
@@ -123,7 +123,7 @@ public function testUnsubscribeRemovesOnlyTargetSession(): void
 
         $callCount = 0;
         $session1->expects($this->exactly(2))->method('get')->with('resource_subscriptions', [])
-            ->willReturnCallback(function () use (&$callCount, $uri) {
+            ->willReturnCallback(static function () use (&$callCount, $uri) {
                 return 0 === $callCount++ ? [] : [$uri => true];
             });
         $session1->expects($this->exactly(2))->method('set')
@@ -159,7 +159,7 @@ public function testUnsubscribeRemovesOnlyTargetSession(): void
 
         $this->protocol->expects($this->exactly(1))
             ->method('sendNotification')
-            ->with($this->callback(fn ($notification) => $notification instanceof ResourceUpdatedNotification && $notification->uri === $uri
+            ->with($this->callback(static fn ($notification) => $notification instanceof ResourceUpdatedNotification && $notification->uri === $uri
             ));
 
         $this->subscriptionManager->notifyResourceChanged($this->protocol, $uri);
@@ -180,7 +180,7 @@ public function testUnsubscribeDoesNotSendNotifications(): void
 
         $callCount = 0;
         $session->expects($this->exactly(2))->method('get')->with('resource_subscriptions', [])
-            ->willReturnCallback(function () use (&$callCount, $uri) {
+            ->willReturnCallback(static function () use (&$callCount, $uri) {
                 return 0 === $callCount++ ? [] : [$uri => true];
             });
         $session->expects($this->exactly(2))->method('set')