Compare serverConnectionId instead of Server instance as Event::getServer is deprecated

Author: GromNaN

tests/SpecTests/RetryableReads/Prose2_RetryOnMongosTest.php

@@ -65,7 +65,8 @@ public function testRetryOnDifferentMongos(): void
 
         // Step 4: Enable failed command event monitoring for client
         $subscriber = new class implements CommandSubscriber {
-            public $commandFailedServers = [];
+            /** @var int[] */
+            public array $commandFailedServers = [];
 
             public function commandStarted(CommandStartedEvent $event): void
             {
@@ -77,7 +78,7 @@ public function commandSucceeded(CommandSucceededEvent $event): void
 
             public function commandFailed(CommandFailedEvent $event): void
             {
-                $this->commandFailedServers[] = $event->getServer();
+                $this->commandFailedServers[] = $event->getServerConnectionId();
             }
         };
 
@@ -130,21 +131,21 @@ public function testRetryOnSameMongos(): void
 
         // Step 4: Enable succeeded and failed command event monitoring
         $subscriber = new class implements CommandSubscriber {
-            public Server $commandSucceededServer;
-            public Server $commandFailedServer;
+            public ?int $commandSucceededServer = null;
+            public ?int $commandFailedServer = null;
 
             public function commandStarted(CommandStartedEvent $event): void
             {
             }
 
             public function commandSucceeded(CommandSucceededEvent $event): void
             {
-                $this->commandSucceededServer = $event->getServer();
+                $this->commandSucceededServer = $event->getServerConnectionId();
             }
 
             public function commandFailed(CommandFailedEvent $event): void
             {
-                $this->commandFailedServer = $event->getServer();
+                $this->commandFailedServer = $event->getServerConnectionId();
             }
         };
 

tests/SpecTests/RetryableWrites/Prose4_RetryOnDifferentMongosTest.php

@@ -67,7 +67,8 @@ public function testRetryOnDifferentMongos(): void
 
         // Step 4: Enable failed command event monitoring for client
         $subscriber = new class implements CommandSubscriber {
-            public $commandFailedServers = [];
+            /** @var int[]  */
+            public array $commandFailedServers = [];
 
             public function commandStarted(CommandStartedEvent $event): void
             {
@@ -79,7 +80,7 @@ public function commandSucceeded(CommandSucceededEvent $event): void
 
             public function commandFailed(CommandFailedEvent $event): void
             {
-                $this->commandFailedServers[] = $event->getServer();
+                $this->commandFailedServers[] = $event->getServerConnectionId();
             }
         };
 

tests/SpecTests/RetryableWrites/Prose5_RetryOnSameMongosTest.php

@@ -50,21 +50,21 @@ public function testRetryOnSameMongos(): void
 
         // Step 4: Enable succeeded and failed command event monitoring
         $subscriber = new class implements CommandSubscriber {
-            public Server $commandSucceededServer;
-            public Server $commandFailedServer;
+            public ?int $commandSucceededServer = null;
+            public ?int $commandFailedServer = null;
 
             public function commandStarted(CommandStartedEvent $event): void
             {
             }
 
             public function commandSucceeded(CommandSucceededEvent $event): void
             {
-                $this->commandSucceededServer = $event->getServer();
+                $this->commandSucceededServer = $event->getServerConnectionId();
             }
 
             public function commandFailed(CommandFailedEvent $event): void
             {
-                $this->commandFailedServer = $event->getServer();
+                $this->commandFailedServer = $event->getServerConnectionId();
             }
         };
 
@@ -78,7 +78,7 @@ public function commandFailed(CommandFailedEvent $event): void
 
         /* Step 6: Assert that exactly one failed command event and one
          * succeeded command event occurred. Assert that both events occurred on
-         * the same mongos. */
+         * the same mongos connection. */
         $this->assertNotNull($subscriber->commandSucceededServer);
         $this->assertNotNull($subscriber->commandFailedServer);
         $this->assertEquals($subscriber->commandSucceededServer, $subscriber->commandFailedServer);