finished protocol V1 test. Updated Socket disconnect.

Author: stefanak-michal

Socket.php

@@ -166,8 +166,10 @@ private function printHex(string $str, bool $write = true)
      */
     public function disconnect()
     {
-        socket_shutdown($this->socket);
-        @socket_close($this->socket);
+        if (is_resource($this->socket)) {
+            @socket_shutdown($this->socket);
+            @socket_close($this->socket);
+        }
     }
 
 }

tests/ATest.php

@@ -0,0 +1,43 @@
+<?php
+
+namespace Bolt\tests;
+
+use \PHPUnit\Framework\TestCase;
+
+/**
+ * Class TestBase
+ * @author Michal Stefanak
+ * @link https://github.com/stefanak-michal/Bolt
+ * @package Bolt\tests
+ */
+abstract class ATest extends TestCase
+{
+
+    /**
+     * @var bool
+     */
+    private static $bypass = false;
+
+    /**
+     * Bypass Socket final keyword before autoload
+     */
+    public static function setUpBeforeClass(): void
+    {
+        if (!self::$bypass) {
+            $path = __DIR__;
+            while (!file_exists($path . DS . 'Socket.php')) {
+                $path = dirname($path);
+            }
+
+            $content = file_get_contents($path . DS . 'Socket.php');
+            if (strpos($content, "final class Socket") !== false) {
+                file_put_contents($path . DS . 'Socket.php', str_replace('final class Socket', 'class Socket', $content));
+                $socket = new \Bolt\Socket('localhost', 0, 0);
+                file_put_contents($path . DS . 'Socket.php', $content);
+            }
+        }
+
+        self::$bypass = true;
+    }
+
+}

tests/BoltTest.php

@@ -2,7 +2,6 @@
 
 namespace Bolt\tests;
 
-use PHPUnit\Framework\TestCase;
 use Bolt\Bolt;
 
 /**
@@ -19,13 +18,13 @@
  * @requires extension sockets
  * @requires extension mbstring
  */
-class BoltTest extends TestCase
+class BoltTest extends \Bolt\tests\ATest
 {
 
     /**
      * @return Bolt|null
      */
-    public function testHello(): Bolt
+    public function testHello(): ?Bolt
     {
         try {
             $bolt = new Bolt($GLOBALS['NEO_HOST'] ?? '127.0.0.1', $GLOBALS['NEO_PORT'] ?? 7687);

tests/PackStream/v1/PackerTest.php

@@ -3,7 +3,7 @@
 namespace Bolt\tests\PackStream\v1;
 
 use Bolt\PackStream\v1\Packer;
-use PHPUnit\Framework\TestCase;
+use Exception;
 
 /**
  * Class PackerTest
@@ -18,7 +18,7 @@
  * @requires extension mbstring
  * @requires extension json
  */
-class PackerTest extends TestCase
+class PackerTest extends \Bolt\tests\ATest
 {
 
     /**
@@ -37,7 +37,7 @@ public function test__construct(): Packer
      * @param string $bin
      * @param array $args
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
     public function testPack(string $bin, array $args, Packer $packer)
     {
@@ -62,7 +62,7 @@ public function packProvider(): array
      * @param string $bin
      * @param int $number
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
     public function testPackInteger(string $bin, int $number, Packer $packer)
     {
@@ -83,7 +83,7 @@ public function integerProvider(): array
     /**
      * @depends test__construct
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
     public function testPackFloat(Packer $packer)
     {
@@ -93,7 +93,7 @@ public function testPackFloat(Packer $packer)
     /**
      * @depends test__construct
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
     public function testPackNull(Packer $packer)
     {
@@ -103,7 +103,7 @@ public function testPackNull(Packer $packer)
     /**
      * @depends test__construct
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
     public function testPackBool(Packer $packer)
     {
@@ -117,7 +117,7 @@ public function testPackBool(Packer $packer)
      * @param string $bin
      * @param string $str
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
     public function testPackString(string $bin, string $str, Packer $packer)
     {
@@ -138,7 +138,7 @@ public function stringProvider(): array
      * @param string $bin
      * @param array $arr
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
     public function testPackArray(string $bin, array $arr, Packer $packer)
     {
@@ -162,9 +162,9 @@ public function arrayProvider(): array
      * @param string $bin
      * @param object $obj
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
-    public function testPackMap(string $bin, object $obj, Packer $packer)
+    public function testPackMap(string $bin, $obj, Packer $packer)
     {
         $this->assertEquals($bin, $this->getMethod($packer)->invoke($packer, $obj));
     }
@@ -184,12 +184,12 @@ public function mapProvider(): array
      * Test it on data type resource, which is not implemented
      * @depends test__construct
      * @param Packer $packer
-     * @throws \Exception
+     * @throws Exception
      */
     public function testException(Packer $packer)
     {
         $f = fopen(__FILE__, 'r');
-        $this->expectException(\Exception::class);
+        $this->expectException(Exception::class);
         $this->getMethod($packer)->invoke($packer, $f);
         fclose($f);
     }