Merge pull request #156 from neo4j-php/default-protocol-versions

Default protocol versions update

Author: stefanak-michal

src/Bolt.php

@@ -33,7 +33,7 @@ public function __construct(private IConnection $connection)
             $this->track();
         }
 
-        $this->setProtocolVersions(5.6, 5.4, 5, 4.4);
+        $this->setProtocolVersions('5.8.8', '4.4.4');
     }
 
     private function track(): void
@@ -198,7 +198,7 @@ private function handshake(): string
     /**
      * Read and compose selected protocol version
      */
-    private function unpackProtocolVersion(string $bytes): ?string
+    private function unpackProtocolVersion(string $bytes): string
     {
         $result = [];
 
@@ -210,8 +210,7 @@ private function unpackProtocolVersion(string $bytes): ?string
             array_shift($result);
         }
 
-        $version = implode('.', array_reverse($result));
-        return in_array($version, $this->protocolVersions) ? $version : null;
+        return implode('.', array_reverse($result));
     }
 
     /**

tests/structures/v5/StructuresTest.php

@@ -3,7 +3,7 @@
 namespace Bolt\tests\structures\v5;
 
 use Bolt\Bolt;
-use Bolt\protocol\{AProtocol, V4_3, V4_4, V5, V5_1};
+use Bolt\protocol\AProtocol;
 use Bolt\tests\structures\v1\DateTimeTrait;
 use Bolt\tests\structures\v1\DateTimeZoneIdTrait;
 use Bolt\protocol\v5\structures\{
@@ -24,15 +24,14 @@
  */
 class StructuresTest extends \Bolt\tests\structures\StructureLayer
 {
-    public function testInit(): AProtocol|V4_3|V4_4|V5|V5_1
+    public function testInit(): AProtocol
     {
         $conn = new \Bolt\connection\StreamSocket($GLOBALS['NEO_HOST'] ?? '127.0.0.1', $GLOBALS['NEO_PORT'] ?? 7687);
         $this->assertInstanceOf(\Bolt\connection\StreamSocket::class, $conn);
 
         $bolt = new Bolt($conn);
         $this->assertInstanceOf(Bolt::class, $bolt);
 
-        /** @var AProtocol|V4_3|V4_4|V5|V5_1 $protocol */
         $protocol = $bolt->build();
         $this->assertInstanceOf(AProtocol::class, $protocol);
 
@@ -54,7 +53,7 @@ public function testInit(): AProtocol|V4_3|V4_4|V5|V5_1
     /**
      * @depends testInit
      */
-    public function testNode(AProtocol|V4_3|V4_4|V5|V5_1 $protocol)
+    public function testNode(AProtocol $protocol)
     {
         $protocol->begin()->getResponse();
 
@@ -81,7 +80,7 @@ public function testNode(AProtocol|V4_3|V4_4|V5|V5_1 $protocol)
     /**
      * @depends testInit
      */
-    public function testPath(AProtocol|V4_3|V4_4|V5|V5_1 $protocol)
+    public function testPath(AProtocol $protocol)
     {
         $protocol->begin()->getResponse();
 
@@ -112,7 +111,7 @@ public function testPath(AProtocol|V4_3|V4_4|V5|V5_1 $protocol)
     /**
      * @depends testInit
      */
-    public function testRelationship(AProtocol|V4_3|V4_4|V5|V5_1 $protocol)
+    public function testRelationship(AProtocol $protocol)
     {
         $protocol->begin()->getResponse();