Merge pull request #15 from JeroenVanOort/correctly-handle-ref

Correctly handle $ref

Author: luisdalmolin

src/ValidatesOpenApiSpec.php

@@ -41,9 +41,9 @@ public function getOpenApiValidatorBuilder(): ValidatorBuilder
             $specType = $this->getSpecFileType();
 
             if ($specType === 'json') {
-                $this->openApiValidatorBuilder = (new ValidatorBuilder())->fromJson($this->getOpenApiSpec());
+                $this->openApiValidatorBuilder = (new ValidatorBuilder())->fromJsonFile($this->getOpenApiSpecPath());
             } elseif ($specType === 'yaml') {
-                $this->openApiValidatorBuilder = (new ValidatorBuilder())->fromYaml($this->getOpenApiSpec());
+                $this->openApiValidatorBuilder = (new ValidatorBuilder())->fromYamlFile($this->getOpenApiSpecPath());
             } else {
                 throw new UnknownParserForFileTypeException("Unknown parser for file type {$specType}");
             }
@@ -185,16 +185,6 @@ protected function shouldSkipRequestValidation(): bool
         return false;
     }
 
-    /**
-     * Gets the open api spec (contents).
-     *
-     * @return string
-     */
-    protected function getOpenApiSpec(): string
-    {
-        return File::get($this->getOpenApiSpecPath());
-    }
-
     /**
      * Gets the openapi.yaml path.
      */

tests/fixtures/OpenAPI.yaml

@@ -73,12 +73,7 @@ paths:
         '200':
           description: OK
   /private:
-    get:
-      responses:
-        '200':
-          description: OK
-      security:
-        - Basic: []
+    $ref: 'sub-spec.yaml'
 components:
   securitySchemes:
     Basic:

tests/fixtures/sub-spec.yaml

@@ -0,0 +1,6 @@
+get:
+  responses:
+    '200':
+      description: OK
+  security:
+    - Basic: []