Merge pull request #969 from OpenAPITools/fix_vendor_extension

Fix missing top level extensions in OAS v2 spec

Author: gracekarina

modules/swagger-parser-v2-converter/src/main/java/io/swagger/v3/parser/converter/SwaggerConverter.java

@@ -139,6 +139,10 @@ public SwaggerParseResult convert(SwaggerDeserializationResult parse) {
 
         Swagger swagger = parse.getSwagger();
 
+        if (swagger.getVendorExtensions() != null) {
+            openAPI.setExtensions(convert(swagger.getVendorExtensions()));
+        }
+
         if (swagger.getExternalDocs() != null) {
             openAPI.setExternalDocs(convert(swagger.getExternalDocs()));
         }

modules/swagger-parser-v2-converter/src/test/java/io/swagger/parser/test/V2ConverterTest.java

@@ -727,6 +727,13 @@ public void testIssue755() throws Exception {
         final OpenAPI oas = getConvertedOpenAPIFromJsonFile(ISSUE_755_YAML);
         assertNotNull(oas);
     }
+
+    @Test(description = "OpenAPI v2 converter - top-level extensions should be preserved")
+    public void testTopLevelExtensions() throws Exception {
+        final OpenAPI oas = getConvertedOpenAPIFromJsonFile(PARAMETER_CONVERSION_JSON);
+        assertNotNull(oas);
+        assertEquals((String)oas.getExtensions().get("x-some-extensions"), "hello");
+    }
 
     @Test(description = "OpenAPI v2 converter - Conversion param extensions should be preserved")
     public void testIssue820() throws Exception {

modules/swagger-parser-v2-converter/src/test/resources/parameter-conversion.json

@@ -1,5 +1,6 @@
 {
     "swagger": "2.0",
+    "x-some-extensions": "hello",
     "info": {
         "version": "1.0",
         "title": "x-example"