phpcs & code quality fixes

.gitignore

@@ -14,3 +14,4 @@ phpunit.xml
 .swp
 /nbproject
 .env
+.php_cs.cache

composer.json

@@ -42,7 +42,7 @@
         "phpmd": "./vendor/bin/phpmd src text controversial,design,./phpmd.xml",
         "phpcs": "./vendor/bin/phpcs -p src --runtime-set testVersion 7.3",
         "phpcbf": "./vendor/bin/phpcbf -p src",
-        "test": "composer run-script phpmd && composer run-script phpunit"
+        "test": "composer run-script phpcs && composer run-script phpmd && composer run-script phpunit"
     },
     "prefer-stable": true,
     "license": "MIT"

phpcs.xml.dist

@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ruleset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="vendor/squizlabs/php_codesniffer/phpcs.xsd">
+    <arg name="basepath" value="."/>
+    <arg name="cache" value=".php_cs.cache"/>
+    <arg name="colors"/>
+    <arg name="extensions" value="php"/>
+
+    <rule ref="PSR2"/>
+
+    <file>src/</file>
+    <file>tests/</file>
+</ruleset>

src/Builder/ContainerBuilder.php

@@ -116,16 +116,16 @@ class ContainerBuilder implements BuilderInterface
         $container = new Container();
 
         switch ($this->env) {
-        case Environment::PROD:
+            case Environment::PROD:
-            $this->setProdServices($container);
+                $this->setProdServices($container);
-            break;
+                break;
-        case Environment::DEV:
+            case Environment::DEV:
-        case Environment::TEST:
+            case Environment::TEST:
-            $this->setProdServices($container);
+                $this->setProdServices($container);
-            $this->setDevServices($container);
+                $this->setDevServices($container);
-            break;
+                break;
-        default:
+            default:
-            throw new RuntimeException(sprintf('Invalid environment type: %s', $this->env));
+                throw new RuntimeException(sprintf('Invalid environment type: %s', $this->env));
         }
 
         return $container;

src/Component/Psr7/Stream.php

@@ -119,9 +119,12 @@ class Stream implements StreamInterface
                 throw $exception;
             }
 
-            trigger_error(sprintf(
+            trigger_error(
-                '%s::__toString exception: %s',
+                sprintf(
-                self::class, (string) $exception),
+                    '%s::__toString exception: %s',
+                    self::class,
+                    (string) $exception
+                ),
                 E_USER_ERROR
             );
         }

src/Factory/SerializerFactory.php

@@ -66,11 +66,11 @@ class SerializerFactory implements FactoryInterface
         $container = $this->container;
 
         return SerializerBuilder::create()
-            ->configureHandlers(function(HandlerRegistry $registry) use ($container) {
+            ->configureHandlers(function (HandlerRegistry $registry) use ($container) {
-                $returnNull = function($visitor, $obj, array $type) {
+                $returnNull = function ($visitor, $obj, array $type) {
                     return null;
                 };
-                $returnSame = function($visitor, $obj, array $type) {
+                $returnSame = function ($visitor, $obj, array $type) {
                     return $obj;
                 };
 
@@ -78,7 +78,7 @@ class SerializerFactory implements FactoryInterface
                     GraphNavigatorInterface::DIRECTION_SERIALIZATION,
                     'RequestDtoInterface',
                     'json',
-                    function($visitor, $obj, array $type) use ($container) {
+                    function ($visitor, $obj, array $type) use ($container) {
                         /** @var SerializerInterface $serializer */
                         $serializer = $container->get(Constants::SERIALIZER);
 
@@ -95,7 +95,7 @@ class SerializerFactory implements FactoryInterface
                     GraphNavigatorInterface::DIRECTION_SERIALIZATION,
                     'RequestDtoInterface',
                     'xml',
-                    function($visitor, $obj, array $type) use ($container) {
+                    function ($visitor, $obj, array $type) use ($container) {
                         /** @var SerializerInterface $serializer */
                         $serializer = $container->get(Constants::SERIALIZER);
 

src/Model/Request/BaseRequest.php

@@ -136,7 +136,9 @@ abstract class BaseRequest
      *
      * @return void
      */
-    final public function setMethod(string $method): void {}
+    final public function setMethod(string $method): void
+    {
+    }
 
     /**
      * Should return method name for this request.