Handling missing body properties

lib/trie.js

@@ -15,7 +15,7 @@ function Node (options) {
   // stores all direct folder descendants of this node
   this.children = {};
 
-  this.requests = options ? options.requests : []; // request will be an array of objects
+  this.requests = []; // request will always be an array of objects
 
   this.addChildren = function (child, value) {
     this.children[child] = value;

lib/util.js

@@ -280,7 +280,7 @@ module.exports = {
       // https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.2.md#pathItemObject
       getPathMethods = function(pathKeys) {
         var methods = [];
-        pathKeys.forEach(function(element) {
+        pathKeys && pathKeys.forEach(function(element) {
           if (METHODS.includes(element)) {
             methods.push(element);
           }
@@ -321,8 +321,6 @@ module.exports = {
             currentNode.addChildren(currentPath[i], new Node({
               name: currentPath[i],
               requestCount: 0,
-              requests: [],
-              children: {},
               type: 'item-group'
             }));
           }
@@ -1012,7 +1010,7 @@ module.exports = {
             type: typeof value
           };
           params = this.convertParamsWithStyle(encoding[key], value);
-          params.forEach((element) => {
+          params && params.forEach((element) => {
             if (typeof element.value === 'object') { element.value = JSON.stringify(element.value); }
             // element.value = JSON.stringify(element.value);
             delete element.description;
@@ -1387,7 +1385,7 @@ module.exports = {
       item.request.body = pmBody.body;
       item.request.addHeader(pmBody.contentHeader);
       // extra form headers if encoding is present in request Body.
-      pmBody.formHeaders.forEach((element) => {
+      pmBody.formHeaders && pmBody.formHeaders.forEach((element) => {
         item.request.addHeader(element);
       });
     }

test/data/valid_openapi/wrong-body.yaml

@@ -0,0 +1,20 @@
+openapi: 3.0.0
+info:
+    description:
+        Test Service
+    version: 0.1
+    title: Test Service Name
+servers:
+    -   url: https://test.com
+        description: Test API
+paths:
+    /entity:
+        put:
+            summary: Update entity
+            operationId: updateEntity
+            requestBody:
+                schema: object # This is wrong,
+                # but the converter should just ignore the formHeaders if not present
+            responses:
+                200:
+                    description: 200 OK

test/unit/convert.temp.js

@@ -24,9 +24,10 @@ describe('The converter must generate a collection conforming to the schema', fu
           expect(err).to.be.null;
 
           let collection = conversionResult.output[0].data;
-          // eslint-disable-next-line no-console
-          // console.log(JSON.stringify(collection, null, 2));
-          fs.writeFileSync('./temp/temp-collection.json', JSON.stringify(collection, null, 2));
+          fs.writeFileSync(
+            path.join(__dirname, '../data/.temp/temp-collection.json'),
+            JSON.stringify(collection, null, 2)
+          );
           done();
         });
     });