Merge pull request #438 from rennanoliveira/fix-open-api-writer-required

Fix Open Api: issue with multiple required fields

Author: oestrich

features/open_api.feature

@@ -192,7 +192,7 @@ Feature: Generate Open API Specification from test examples
 
           parameter :name, 'The order name', required: true, scope: :data, with_example: true
           parameter :amount, required: false, scope: :data, with_example: true
-          parameter :description, 'The order description', required: false, scope: :data, with_example: true
+          parameter :description, 'The order description', required: true, scope: :data, with_example: true
 
           header "Content-Type", "application/json"
 
@@ -700,7 +700,8 @@ Feature: Generate Open API Specification from test examples
                         }
                       },
                       "required": [
-                        "name"
+                        "name",
+                        "description"
                       ]
                     }
                   }

lib/rspec_api_documentation/writers/open_api_writer.rb

@@ -149,7 +149,10 @@ def extract_schema(fields)
             opts.each { |k, v| current[:properties][field[:name]][k] = v if v }
           end
 
-          current[:required] ||= [] << field[:name] if field[:required]
+          if field[:required]
+            current[:required] ||= []
+            current[:required] << field[:name]
+          end
         end
 
         OpenApi::Schema.new(schema)