Fix codestyle and phpstan

Author: DEVizzent

src/Config/ExpectationsPath.php

@@ -3,6 +3,7 @@
 namespace DEVizzent\CodeceptionMockServerHelper\Config;
 
 use DEVizzent\CodeceptionMockServerHelper\MockServerHelper;
+use InvalidArgumentException;
 use RecursiveDirectoryIterator;
 use RecursiveIteratorIterator;
 use SplFileInfo;
@@ -13,24 +14,27 @@ class ExpectationsPath
 
     public function __construct(string $path = '')
     {
-        if (!empty($path)) {
+        if ('' === $path) {
             $this->set($path);
         }
     }
 
-    private function set(string $path)
+    private function set(string $path): void
     {
         $path = realpath($path);
-        if ($path) {
+        if (false !== $path) {
             $this->path = $path;
             return;
         }
-        throw new \InvalidArgumentException(sprintf('"%s" is not a valid path for "expectationsPath"', $path));
+        throw new InvalidArgumentException(sprintf('"%s" is not a valid path for "expectationsPath"', $path));
     }
 
+    /**
+     * @return array<string>
+     */
     public function getExpectationsFiles(): iterable
     {
-        if (empty($this->path)) {
+        if ('' !== $this->path) {
             return [];
         }
         if (!is_dir($this->path)) {
@@ -40,11 +44,11 @@ public function getExpectationsFiles(): iterable
         $files = [];
         /** @var SplFileInfo $file */
         foreach ($recursiveIterator as $file) {
-            if ($file->isDir()){
+            if ($file->isDir()) {
                 continue;
             }
             $files[] = $file->getPathname();
         }
         return $files;
     }
-}
+}

src/MockServerHelper.php

@@ -58,7 +58,9 @@ public function _initialize(): void
 
         try {
             $this->deactivateNotMatchedRequest();
-        } catch (AssertionFailedError $exception) {}
+        } catch (AssertionFailedError $exception) {
+            return;
+        }
     }
 
     public function _beforeSuite($settings = []): void
@@ -130,7 +132,7 @@ public function deactivateNotMatchedRequest(): void
         $this->mockserver->removeById(self::NOT_MATCHED_REQUEST_ID);
     }
 
-    public function createMockRequestFromJsonFile($expectationFile): void
+    public function createMockRequestFromJsonFile(string $expectationFile): void
     {
         $expectationJson = file_get_contents($expectationFile);
         Assert::assertIsString($expectationJson);