byjg / php-swagger-test

src/Swagger/SwaggerSchema.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
             throw new InvalidArgumentException('schema must be given as array or JSON string');
         }
         $this->jsonFile = $data;
-        $this->allowNullValues = (bool) $allowNullValues;
+        $this->allowNullValues = (bool)$allowNullValues;
     }
 
     public function getHttpSchema()
@@ -121,6 +121,6 @@ 
      */
     public function setAllowNullValues($value)
     {
-        $this->allowNullValues = (bool) $value;
+        $this->allowNullValues = (bool)$value;
     }
 }

src/AbstractRequester.php

Doc Comments +7 added lines

@@ -157,6 +157,9 @@ 
         return $this;
     }
 
+    /**
+     * @param integer $code
+     */
     public function assertResponseCode($code)
     {
         $this->statusExpected = $code;
@@ -266,6 +269,10 @@ 
         return $response;
     }
 
+    /**
+     * @param string $contentType
+     * @param string $body
+     */
     protected function parseMultiPartForm($contentType, $body)
     {
         $matchRequest = [];

Braces +3 added lines, -2 removed lines

@@ -285,8 +285,9 @@
 
         // loop data blocks
         foreach ($blocks as $id => $block) {
-            if (empty($block))
-                continue;
+            if (empty($block)) {
+                            continue;
+            }
 
             if (strpos($block, 'application/octet-stream') !== false) {
                 preg_match("/name=\"([^\"]*)\".*stream[\n|\r]+([^\n\r].*)?$/s", $block, $matches);

Spacing +4 added lines, -4 removed lines

@@ -211,10 +211,10 @@ 
             ->withScheme($uriSchema->getScheme())
             ->withHost($uriSchema->getHost())
             ->withPort($uriSchema->getPort())
-            ->withPath($uriSchema->getPath() . $this->psr7Request->getUri()->getPath());
+            ->withPath($uriSchema->getPath().$this->psr7Request->getUri()->getPath());
 
-        if (!preg_match("~^{$this->schema->getBasePath()}~",  $uri->getPath())) {
-            $uri = $uri->withPath($this->schema->getBasePath() . $uri->getPath());
+        if (!preg_match("~^{$this->schema->getBasePath()}~", $uri->getPath())) {
+            $uri = $uri->withPath($this->schema->getBasePath().$uri->getPath());
         }
 
         $this->psr7Request = $this->psr7Request->withUri($uri);
@@ -241,7 +241,7 @@ 
         // Handle Request
         $response = $this->handleRequest($this->psr7Request);
         $responseHeader = $response->getHeaders();
-        $responseBodyStr = (string) $response->getBody();
+        $responseBodyStr = (string)$response->getBody();
         $responseBody = json_decode($responseBodyStr, true);
         $statusReturned = $response->getStatusCode();
 

src/Base/Body.php

Doc Comments +7 added lines, -7 removed lines

@@ -82,7 +82,7 @@ 
      * @param $schema
      * @param $body
      * @param $type
-     * @return bool
+     * @return null|boolean
      * @throws NotMatchedException
      */
     protected function matchString($name, $schema, $body, $type)
@@ -103,7 +103,7 @@ 
      * @param $schema
      * @param $body
      * @param $type
-     * @return bool
+     * @return null|boolean
      * @throws NotMatchedException
      */
     protected function matchFile($name, $schema, $body, $type)
@@ -119,7 +119,7 @@ 
      * @param $name
      * @param $body
      * @param $type
-     * @return bool
+     * @return null|boolean
      * @throws NotMatchedException
      */
     protected function matchNumber($name, $body, $type)
@@ -139,7 +139,7 @@ 
      * @param $name
      * @param $body
      * @param $type
-     * @return bool
+     * @return null|boolean
      * @throws NotMatchedException
      */
     protected function matchBool($name, $body, $type)
@@ -160,7 +160,7 @@ 
      * @param $schema
      * @param $body
      * @param $type
-     * @return bool
+     * @return null|boolean
      * @throws DefinitionNotFoundException
      * @throws GenericSwaggerException
      * @throws InvalidDefinitionException
@@ -234,7 +234,7 @@ 
     }
 
     /**
-     * @param $name
+     * @param string $name
      * @param $schema
      * @param $body
      * @return bool|null
@@ -350,7 +350,7 @@ 
      * @param $body
      * @param $type
      * @param $nullable
-     * @return bool
+     * @return null|boolean
      * @throws NotMatchedException
      */
     protected function matchNull($name, $body, $type, $nullable)

Spacing +10 added lines, -10 removed lines

@@ -15,8 +15,8 @@ 
 
 abstract class Body
 {
-    const SWAGGER_PROPERTIES="properties";
-    const SWAGGER_REQUIRED="required";
+    const SWAGGER_PROPERTIES = "properties";
+    const SWAGGER_REQUIRED = "required";
 
     /**
      * @var Schema
@@ -72,7 +72,7 @@ 
             return new OpenApiResponseBody($schema, $name, $structure, $allowNullValues);
         }
 
-        throw new GenericSwaggerException("Cannot get instance SwaggerBody or SchemaBody from " . get_class($schema));
+        throw new GenericSwaggerException("Cannot get instance SwaggerBody or SchemaBody from ".get_class($schema));
     }
 
     abstract public function match($body);
@@ -192,32 +192,32 @@ 
         $nullable = isset($schema['nullable']) ? (bool)$schema['nullable'] : $this->schema->isAllowNullValues();
 
         $validators = [
-            function () use ($name, $body, $type, $nullable)
+            function() use ($name, $body, $type, $nullable)
             {
                 return $this->matchNull($name, $body, $type, $nullable);
             },
 
-            function () use ($name, $schema, $body, $type)
+            function() use ($name, $schema, $body, $type)
             {
                 return $this->matchString($name, $schema, $body, $type);
             },
 
-            function () use ($name, $body, $type)
+            function() use ($name, $body, $type)
             {
                 return $this->matchNumber($name, $body, $type);
             },
 
-            function () use ($name, $body, $type)
+            function() use ($name, $body, $type)
             {
                 return $this->matchBool($name, $body, $type);
             },
 
-            function () use ($name, $schema, $body, $type)
+            function() use ($name, $schema, $body, $type)
             {
                 return $this->matchArray($name, $schema, $body, $type);
             },
 
-            function () use ($name, $schema, $body, $type)
+            function() use ($name, $schema, $body, $type)
             {
                 return $this->matchFile($name, $schema, $body, $type);
             },
@@ -317,7 +317,7 @@ 
             return true;
         }
 
-        if(!isset($schema['$ref']) && isset($schema['content'])) {
+        if (!isset($schema['$ref']) && isset($schema['content'])) {
             $schema['$ref'] = $schema['content'][key($schema['content'])]['schema']['$ref'];
         }
 

src/ApiTestCase.php

Doc Comments +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
      * @param array|null $query
      * @param array|null $requestBody
      * @param array $requestHeader
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws DefinitionNotFoundException
      * @throws GenericSwaggerException
      * @throws HttpMethodNotFoundException

src/Swagger/SwaggerResponseBody.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     {
         if (!isset($this->structure['schema'])) {
             if (!empty($body)) {
-                throw new NotMatchedException("Expected empty body for " . $this->name);
+                throw new NotMatchedException("Expected empty body for ".$this->name);
             }
             return true;
         }

src/Exception/BaseException.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     {
         $this->body = $body;
         if (!empty($body)) {
-            $message = $message . " ->\n" . json_encode($body, JSON_PRETTY_PRINT) . "\n";
+            $message = $message." ->\n".json_encode($body, JSON_PRETTY_PRINT)."\n";
         }
         parent::__construct($message, $code, $previous);
     }

src/Base/Schema.php

Spacing +3 added lines, -3 removed lines

@@ -78,7 +78,7 @@ 
     {
         $method = strtolower($method);
 
-        $path = preg_replace('~^' . $this->getBasePath() . '~', '', $path);
+        $path = preg_replace('~^'.$this->getBasePath().'~', '', $path);
 
         $uri = new Uri($path);
 
@@ -96,7 +96,7 @@ 
                 continue;
             }
 
-            $pathItemPattern = '~^' . preg_replace('~{(.*?)}~', '(?<\1>[^/]+)', $pathItem) . '$~';
+            $pathItemPattern = '~^'.preg_replace('~{(.*?)}~', '(?<\1>[^/]+)', $pathItem).'$~';
 
             $matches = [];
             if (preg_match($pathItemPattern, $uri->getPath(), $matches)) {
@@ -122,7 +122,7 @@ 
             }
         }
 
-        throw new PathNotFoundException('Path "' . $path . '" not found');
+        throw new PathNotFoundException('Path "'.$path.'" not found');
     }
 
     /**

src/OpenApi/OpenApiResponseBody.php

Spacing +2 added lines, -2 removed lines

@@ -24,12 +24,12 @@
     {
         if (empty($this->structure['content']) && !isset($this->structure['$ref'])) {
             if (!empty($body)) {
-                throw new NotMatchedException("Expected empty body for " . $this->name);
+                throw new NotMatchedException("Expected empty body for ".$this->name);
             }
             return true;
         }
         
-        if(!isset($this->structure['content']) && isset($this->structure['$ref'])){
+        if (!isset($this->structure['content']) && isset($this->structure['$ref'])) {
             $defintion = $this->schema->getDefinition($this->structure['$ref']);
             return $this->matchSchema($this->name, $defintion, $body);
         }