Merge pull request #722 from postmanlabs/feature/fix-non-array-required-issue

#708 Fixes issue where if string is defined for required field, conversion was failing.

Author: VShingala

assets/json-schema-faker.js

@@ -24258,7 +24258,7 @@ function extend() {
 
     const properties = value.properties || {};
     const patternProperties = value.patternProperties || {};
-    const requiredProperties = typeof value.required === 'boolean' ? [] : (value.required || []).slice();
+    const requiredProperties = (!Array.isArray(value.required)) ? [] : (value.required || []).slice();
     const allowsAdditional = value.additionalProperties !== false;
 
     const propertyKeys = Object.keys(properties);

test/unit/faker.test.js

@@ -136,4 +136,22 @@ describe('JSON SCHEMA FAKER TESTS', function () {
     expect(fakedData.length >= 2).to.be.true;
     expect(fakedData.length <= 63).to.be.true;
   });
+
+  it('Should successsfully generate data iff required is defined as string', function () {
+    const schema = {
+      type: 'object',
+      required: 'timebase',
+      properties: {
+        timebase: { type: 'string' },
+        linkid: { type: 'string' },
+        chartRef: { type: 'string' }
+      }
+    };
+
+    var fakedData = schemaFaker(schema);
+    expect(fakedData).to.be.an('object');
+    expect(fakedData.timebase).to.be.a('string');
+    expect(fakedData.linkid).to.be.a('string');
+    expect(fakedData.chartRef).to.be.a('string');
+  });
 });