fix: Use original Swagger object and fix references (#70)

welovecoding · Aug 8, 2019 · 6867544 · 6867544
1 parent 41292d5
commit 6867544
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/generators/MethodGenerator.ts b/src/generators/MethodGenerator.ts
@@ -223,7 +223,7 @@ export class MethodGenerator {
         console.warn(`No reference found for "${schema.$ref}".`);
         return TypeScriptType.EMPTY_OBJECT;
       }
-      const definition = schema.$ref.replace('#/definitions', '');
+      const definition = schema.$ref.replace('#/definitions/', '');
       properties = this.spec.definitions[definition].properties;
       requiredProperties = this.spec.definitions[definition].required;
       schemaType = this.spec.definitions[definition].type as string;

diff --git a/src/validator/SwaggerValidator.ts b/src/validator/SwaggerValidator.ts
@@ -2,5 +2,8 @@ import {OpenAPIV2} from 'openapi-types';
 import SwaggerParser from 'swagger-parser';
 
 export async function validateConfig(swaggerJson: OpenAPIV2.Document): Promise<void> {
-  await SwaggerParser.validate(swaggerJson);
+  // let's create a copy because swagger-parser modifies the object,
+  // see https://github.com/APIDevTools/swagger-parser/issues/77
+  const swaggerJsonCopy = JSON.parse(JSON.stringify(swaggerJson));
+  await SwaggerParser.validate(swaggerJsonCopy);
 }