psalm to level 2

.github/workflows/php.yml

@@ -41,4 +41,4 @@ jobs:
         run: XDEBUG_MODE=coverage bin/phpspec run -f progress -c phpspec-coverage.yml
 
       - name: Run behat tests
-        run: ./bin/behat
+        run: ./bin/behat --snippets-for

CHANGELOG.md

@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+### Removed
+* upgrade psalm error level to 2
+
 ### Removed
 * CircleCI configuration for CI
 

README.md

@@ -51,7 +51,7 @@ use PUGX\Poser\Render\SvgPlasticRender;
 use PUGX\Poser\Poser;
 
 $render = new SvgPlasticRender();
-$poser = new Poser($render);
+$poser = new Poser([$render]);
 
 echo $poser->generate('license', 'MIT', '428F7E', 'plastic');
 // or

composer.json

@@ -55,7 +55,7 @@
         "php-cs-fixer": "bin/php-cs-fixer fix -v --ansi",
         "phpspec": "bin/phpspec run --format=pretty --ansi",
         "phpspec-coverage": "bin/phpspec run -f progress -c phpspec-coverage.yml --ansi",
-        "behat": "bin/behat",
+        "behat": "bin/behat --snippets-for",
         "docker:build:php74": "docker build -t pugx/poser:php74 -f .docker/base/php74/Dockerfile .",
         "docker:push:php74": "docker push pugx/poser:php74",
         "docker:build:php80": "docker build -t pugx/poser:php80 -f .docker/base/php80/Dockerfile .",

features/bootstrap/FeatureContext.php

@@ -1,11 +1,11 @@
 <?php
 
-use Behat\Behat\Context\SnippetAcceptingContext;
+use Behat\Behat\Context\Context;
 
 /**
  * Behat context class.
  */
-class FeatureContext implements SnippetAcceptingContext
+class FeatureContext implements Context
 {
     private string $binFolder;
     private string $output;
@@ -80,7 +80,7 @@ private function assertEquals($given, $expected): void
         \similar_text($expected, $given, $perc);
 
         if ($perc < 94) {
-            throw new \Exception('String similarity:' . $perc . '%. String expected:' . $expected . PHP_EOL . ' given:' . $given);
+            throw new \Exception('String similarity:' . $perc . '%. String expected:' . $expected . \PHP_EOL . ' given:' . $given);
         }
     }
 }

psalm.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <psalm
-    errorLevel="3"
+    errorLevel="2"
     resolveFromConfigFile="true"
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xmlns="https://getpsalm.org/schema/config"
@@ -13,6 +13,7 @@
         <ignoreFiles>
             <directory name="vendor" />
             <directory name="spec" />
+            <directory name="features" />
         </ignoreFiles>
     </projectFiles>
 </psalm>

spec/PUGX/Poser/PoserSpec.php

@@ -89,7 +89,7 @@ public function getMatchers(): array
                 $regex = '/^<svg.*width="((.|\n)*)' . $subject . '((.|\n)*)' . $status . '((.|\n)*)<\/svg>$/';
                 $matches = [];
 
-                return \preg_match($regex, $object, $matches, PREG_OFFSET_CAPTURE, 0);
+                return \preg_match($regex, $object, $matches, \PREG_OFFSET_CAPTURE, 0);
             },
         ];
     }

spec/PUGX/Poser/Render/SvgFlatRenderSpec.php

@@ -29,7 +29,7 @@ public function getMatchers(): array
                 $regex = '/^<svg.*width="((.|\n)*)<\/svg>$/';
                 $matches = [];
 
-                return \preg_match($regex, (string) $subject, $matches, PREG_OFFSET_CAPTURE, 0);
+                return \preg_match($regex, (string) $subject, $matches, \PREG_OFFSET_CAPTURE, 0);
             },
         ];
     }

spec/PUGX/Poser/Render/SvgFlatSquareRenderSpec.php

@@ -29,7 +29,7 @@ public function getMatchers(): array
                 $regex = '/^<svg.*width="((.|\n)*)<\/svg>$/';
                 $matches = [];
 
-                return \preg_match($regex, (string) $subject, $matches, PREG_OFFSET_CAPTURE, 0);
+                return \preg_match($regex, (string) $subject, $matches, \PREG_OFFSET_CAPTURE, 0);
             },
         ];
     }

spec/PUGX/Poser/Render/SvgPlasticRenderSpec.php

@@ -45,7 +45,7 @@ public function getMatchers(): array
                 $regex = '/^<svg.*width="((.|\n)*)<\/svg>$/';
                 $matches = [];
 
-                return \preg_match($regex, (string) $subject, $matches, PREG_OFFSET_CAPTURE, 0);
+                return \preg_match($regex, (string) $subject, $matches, \PREG_OFFSET_CAPTURE, 0);
             },
         ];
     }

src/Image.php

@@ -34,7 +34,7 @@ private function __construct(string $content, string $style)
      */
     public function __toString(): string
     {
-        return (string) $this->content;
+        return $this->content;
     }
 
     /**

src/Poser.php

@@ -13,12 +13,9 @@ class Poser
      *
      * @param $renders
      */
-    public function __construct($renders)
+    public function __construct(array $renders)
     {
         $this->renders = [];
-        if (!\is_array($renders)) {
-            $renders = [$renders];
-        }
 
         foreach ($renders as $render) {
             $this->addStyleRender($render);

src/UI/Command.php

@@ -28,6 +28,18 @@ class Command extends BaseCommand
 
     protected string $header;
 
+    public function __construct(string $name = null)
+    {
+        parent::__construct($name);
+
+        $this->poser = new Poser([
+            new SvgPlasticRender(),
+            new SvgFlatRender(),
+            new SvgFlatSquareRender(),
+        ]);
+        $this->header = self::HEADER;
+    }
+
     private function init(): void
     {
         $this->poser = new Poser([

src/UI/SingleCommandApplication.php

@@ -51,7 +51,7 @@ public function __construct(Command $command, string $version = 'UNKNOWN')
 
         // Add the given command as single (accessible) command.
         $this->add($command);
-        $this->commandName = (string) $commandName;
+        $this->commandName = $commandName;
 
         // Override the Application's definition so that it does not
         // require a command name as first argument.