Merge pull request #1 from akondas/phpstan

Add phpstan

Author: akondas

.travis.yml

@@ -19,4 +19,5 @@ before_script:
 
 script:
     - composer check-cs
-    - vendor/bin/phpunit
+    - composer phpstan
+    - composer tests

composer.json

@@ -15,7 +15,8 @@
     },
     "require-dev": {
         "phpunit/phpunit": "^7.0",
-        "symplify/easy-coding-standard": "^3.2"
+        "symplify/easy-coding-standard": "^3.2",
+        "phpstan/phpstan": "^0.10.3"
     },
     "autoload": {
         "psr-4": {
@@ -34,6 +35,7 @@
         ],
         "fix-cs": "vendor/bin/ecs check src tests --fix --config=coding-standard.neon",
         "check-cs": "vendor/bin/ecs check src tests --config=coding-standard.neon",
+        "phpstan": "vendor/bin/phpstan analyse src tests --level=max",
         "tests": "vendor/bin/phpunit"
     }
 }

composer.lock

@@ -26,12 +26,12 @@ public function send(Message $message): void
 
     public function host(): string
     {
-        return parse_url($this->connection->getRemoteAddress(), PHP_URL_HOST);
+        return parse_url((string) $this->connection->getRemoteAddress(), PHP_URL_HOST);
     }
 
     public function port(): int
     {
-        return parse_url($this->connection->getRemoteAddress(), PHP_URL_PORT);
+        return parse_url((string) $this->connection->getRemoteAddress(), PHP_URL_PORT);
     }
 
     /**

src/Node/Peer.php

@@ -17,9 +17,11 @@ final class NodeTest extends TestCase
 
     public function setUp(): void
     {
+        /** @var P2pServer $p2pServer */
+        $p2pServer = $this->createMock(P2pServer::class);
         $this->node = new Node(
             new Miner(new Blockchain(Block::genesis()), new ZeroPrefix()),
-            $this->createMock(P2pServer::class)
+            $p2pServer
         );
     }