[Fixes YousefED#106] Don't add ignored properties to "required" arrays (YousefED#125)

* [Fixes YousefED#106] Don't add ignored properties to "required" arrays

* [squashme] PR feedback

Author: RyanLB

test/programs/ignored-required/main.ts

@@ -0,0 +1,14 @@
+interface MyObject {
+    /**
+     * @ignore
+     */
+    ignored: boolean;
+
+    /**
+     * @ignore
+     */
+    ignoredOptional?: boolean;
+    
+    required: boolean;
+    optional?: boolean;
+}

test/programs/ignored-required/schema.json

@@ -0,0 +1,15 @@
+{
+  "$schema": "http://json-schema.org/draft-04/schema#",
+  "properties": {
+    "required": {
+      "type": "boolean"
+    },
+    "optional": {
+      "type": "boolean"
+    }
+  },
+  "required": [
+        "required"
+    ],
+  "type": "object"
+}

test/schema.test.ts

@@ -69,6 +69,8 @@ describe("schema", () => {
     assertSchema("class-single", "main.ts", "MyObject");
     assertSchema("class-extends", "main.ts", "MyObject");
 
+    assertSchema("ignored-required", "main.ts", "MyObject");
+
     assertSchema("interface-single", "main.ts", "MyObject");
     assertSchema("interface-multi", "main.ts", "MyObject");
     assertSchema("interface-extends", "main.ts", "MyObject");

typescript-json-schema.ts

@@ -585,7 +585,9 @@ export class JsonSchemaGenerator {
             }
             if (this.args.generateRequired) {
                 const requiredProps = props.reduce((required: string[], prop: ts.Symbol) => {
-                    if (!(prop.flags & ts.SymbolFlags.Optional) && !(<any>prop).mayBeUndefined) {
+                    let def = {};
+                    this.parseCommentsIntoDefinition(prop, def, {});
+                    if (!(prop.flags & ts.SymbolFlags.Optional) && !(<any>prop).mayBeUndefined && !def.hasOwnProperty("ignore")) {
                         required.push(prop.getName());
                     }
                     return required;