byjg / php-swagger-test

src/AbstractRequester.php

Doc Comments +3 added lines

@@ -119,6 +119,9 @@
         return $this;
     }
 
+    /**
+     * @param integer $code
+     */
     public function assertResponseCode($code)
     {
         $this->statusExpected = $code;

Spacing +3 added lines, -3 removed lines

@@ -163,7 +163,7 @@ 
         // Preparing Parameters
         $paramInQuery = null;
         if (!empty($this->query)) {
-            $paramInQuery = '?' . http_build_query($this->query);
+            $paramInQuery = '?'.http_build_query($this->query);
         }
 
         // Preparing Header
@@ -187,11 +187,11 @@ 
 
         $statusReturned = null;
         // Run the request
-        $response = $this->handleRequest($pathName . $paramInQuery, $header);
+        $response = $this->handleRequest($pathName.$paramInQuery, $header);
 
         // Get the response
         $responseHeader = $response->getHeaders();
-        $responseBody = json_decode((string) $response->getBody(), true);
+        $responseBody = json_decode((string)$response->getBody(), true);
         $statusReturned = $response->getStatusCode();
 
         // Assert results

src/Base/Body.php

Doc Comments +9 added lines, -6 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)
@@ -102,7 +102,7 @@ 
      * @param $name
      * @param $body
      * @param $type
-     * @return bool
+     * @return null|boolean
      * @throws NotMatchedException
      */
     protected function matchNumber($name, $body, $type)
@@ -122,7 +122,7 @@ 
      * @param $name
      * @param $body
      * @param $type
-     * @return bool
+     * @return null|boolean
      * @throws NotMatchedException
      */
     protected function matchBool($name, $body, $type)
@@ -143,7 +143,7 @@ 
      * @param $schema
      * @param $body
      * @param $type
-     * @return bool
+     * @return null|boolean
      * @throws DefinitionNotFoundException
      * @throws GenericSwaggerException
      * @throws InvalidDefinitionException
@@ -189,6 +189,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $name
+     */
     protected function matchTypes($name, $schema, $body)
     {
         if (!isset($schema['type'])) {
@@ -241,7 +244,7 @@ 
     }
 
     /**
-     * @param $name
+     * @param string $name
      * @param $schema
      * @param $body
      * @return bool|null
@@ -357,7 +360,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);
@@ -199,32 +199,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->matchDate($name, $schema, $body, $type);
             }
@@ -324,7 +324,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

Indentation +1 added lines, -1 removed lines

@@ -8,5 +8,5 @@
 
 abstract class ApiTestCase extends TestCase
 {
-   use AssertRequestAgainstSchema;
+    use AssertRequestAgainstSchema;
 }

src/ApiRequester.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         // Make the request
         $request = new Request(
             $this->method,
-            $this->schema->getServerUrl() . $path,
+            $this->schema->getServerUrl().$path,
             $headers,
             json_encode($this->requestBody)
         );