fix(oas2): use forOwn to iterate over objects

`forEach` will treat object as an array if it contains a property
`length`

Author: kylef

packages/fury-adapter-swagger/lib/generator.js

@@ -56,7 +56,7 @@ const bodyFromSchema = (schema, payload, parser, contentType = 'application/json
         const boundary = parseBoundary(contentType);
         let content = '';
 
-        _.forEach(body, (value, key) => {
+        _.forOwn(body, (value, key) => {
           content += `--${boundary}\r\n`;
           content += `Content-Disposition: form-data; name="${key}"\r\n\r\n`;
           content += `${value}\r\n`;

packages/fury-adapter-swagger/lib/json-schema.js

@@ -379,7 +379,7 @@ const convertSchemaDefinitions = (definitions) => {
   const jsonSchemaDefinitions = {};
 
   if (definitions) {
-    _.forEach(definitions, (schema, key) => {
+    _.forOwn(definitions, (schema, key) => {
       jsonSchemaDefinitions[key] = convertSchema(schema, { definitions }, { definitions }, false);
     });
   }

packages/fury-adapter-swagger/lib/parser.js

@@ -192,7 +192,7 @@ class Parser {
 
         const paths = _.omitBy(swagger.paths, isExtension);
 
-        _.forEach(paths, (pathValue, href) => {
+        _.forOwn(paths, (pathValue, href) => {
           this.handleSwaggerPath(pathValue, href);
         });
 
@@ -631,7 +631,7 @@ class Parser {
     const dataStructures = new Category();
     dataStructures.classes.push('dataStructures');
 
-    _.forEach(definitions, (schema, key) => {
+    _.forOwn(definitions, (schema, key) => {
       this.withPath(key, () => {
         try {
           const dataStructure = generator.generateDataStructure(schema);
@@ -718,7 +718,7 @@ class Parser {
         .value();
 
       // Each path is an object with methods as properties
-      _.forEach(relevantMethods, (methodValue, method) => {
+      _.forOwn(relevantMethods, (methodValue, method) => {
         this.handleSwaggerMethod(resource, href, pathObjectParameters, methodValue, method);
       });
 
@@ -839,7 +839,7 @@ class Parser {
       }
 
       // Transactions are created for each response in the document
-      _.forEach(relevantResponses, (responseValue, statusCode) => {
+      _.forOwn(relevantResponses, (responseValue, statusCode) => {
         this.handleSwaggerResponse(
           transition, method, methodValue,
           transitionParams, responseValue, statusCode,