Fix phpstan warnings and enhance check (#38)

.travis.yml

@@ -3,7 +3,7 @@ php:
   - 7
   - 7.1
   - 7.2
-#  - nightly // phpcs not support 7.3 yet
+  - nightly
 install:
   - composer self-update
 before_script:
@@ -14,7 +14,7 @@ script:
   - ./bin/phpcs --standard=psr12 src
   - ./bin/phpmd src text cleancode,codesize,controversial,design,unusedcode
   - ./bin/phpcpd src
-  - ./bin/phpstan analyse -n --no-ansi --level=1 src tests
+  - ./bin/phpstan analyse -n --no-ansi --level=7 src tests -c phpstan.neon
 after_success:
   - ./bin/pdepend --summary-xml=/tmp/summary.xml --jdepend-chart=/tmp/jdepend.svg --overview-pyramid=/tmp/pyramid.svg src
   - ./bin/phploc src

composer.json

@@ -37,7 +37,8 @@
         "phpmd/phpmd" : "^2.6",
         "pdepend/pdepend": "^2.5",
         "phploc/phploc": "^4.0",
-        "phpstan/phpstan": "^0.9|^0.10"
+        "phpstan/phpstan": "^0.9|^0.10",
+        "phpstan/phpstan-phpunit": "^0.9|^0.10.0"
     },
     "config": {
         "bin-dir": "bin"

phpstan.neon

@@ -0,0 +1,6 @@
+includes:
+    - vendor/phpstan/phpstan-phpunit/extension.neon
+    - vendor/phpstan/phpstan-phpunit/rules.neon
+parameters:
+	excludes_analyse:
+		- src/DependencyInjection/Configuration.php

src/Entity/Search/Response.php

@@ -51,7 +51,7 @@ class Response
     private $timing;
 
     /**
-     * @var ArrayCollection<Printdeal\PandosearchBundle\Entity\Suggestion>
+     * @var ArrayCollection<\Printdeal\PandosearchBundle\Entity\Suggestion>
      * @Serializer\Type("ArrayCollection<Printdeal\PandosearchBundle\Entity\Suggestion>")
      */
     private $suggestions;

src/Entity/Suggestion/Response.php

@@ -14,7 +14,7 @@ class Response
     protected $hits = [];
 
     /**
-     * @var ArrayCollection<Printdeal\PandosearchBundle\Entity\Suggestion>
+     * @var ArrayCollection<\Printdeal\PandosearchBundle\Entity\Suggestion>
      * @Serializer\Type("ArrayCollection<Printdeal\PandosearchBundle\Entity\Suggestion>")
      */
     private $suggestions;

src/Locator/HttpClientLocator.php

@@ -55,8 +55,9 @@ public function getClient(string $localization = self::DEFAULT_GUZZLE_CLIENT): C
             return $this->clients[$localization];
         }
 
-        if (count($this->clients) === 1) {
-            return reset($this->clients);
+        $client = reset($this->clients);
+        if ($client instanceof ClientInterface) {
+            return $client;
         }
 
         throw new ClientNotFoundException($localization);

tests/Service/SearchServiceTest.php

@@ -7,7 +7,7 @@
 use JMS\Serializer\Exception\UnsupportedFormatException;
 use JMS\Serializer\SerializerInterface;
 use PHPUnit\Framework\TestCase;
-use \PHPUnit_Framework_MockObject_MockObject as Mock;
+use PHPUnit\Framework\MockObject\MockObject as Mock;
 use Printdeal\PandosearchBundle\Criteria\SearchCriteria;
 use Printdeal\PandosearchBundle\Criteria\SuggestCriteria;
 use Printdeal\PandosearchBundle\Exception\ClientNotFoundException;
@@ -24,31 +24,31 @@
 class SearchServiceTest extends TestCase
 {
     /**
-     * @param Mock|null $clientLocator
-     * @param Mock|null $queryBuilder
-     * @param Mock|null $serializer
+     * @param HttpClientLocator|Mock $clientLocator
+     * @param QueryBuilder|Mock $queryBuilder
+     * @param SerializerInterface|Mock $serializer
      * @return SearchService
      */
     private function getSearchServiceMock(
         Mock $clientLocator = null,
         Mock $queryBuilder = null,
         Mock $serializer = null
     ) {
-        if (!$clientLocator) {
+        if (is_null($clientLocator)) {
             /** @var HttpClientLocator $clientLocator */
             $clientLocator = $this->getMockBuilder(HttpClientLocator::class)
                 ->disableOriginalConstructor()
                 ->getMock();
         }
 
-        if (!$queryBuilder) {
+        if (is_null($queryBuilder)) {
             /** @var QueryBuilder $queryBuilder */
             $queryBuilder = $this->getMockBuilder(QueryBuilder::class)
                 ->disableOriginalConstructor()
                 ->getMock();
         }
 
-        if (!$serializer) {
+        if (is_null($serializer)) {
             /** @var SerializerInterface $serializer */
             $serializer = $this->getMockBuilder(SerializerInterface::class)
                 ->disableOriginalConstructor()