NgModule still generated by default in 1.x

Fixes #373

Author: Luis Fernando Planella Gonzalez

lib/ng-openapi-gen.ts

@@ -404,6 +404,11 @@ export class NgOpenApiGen {
   }
 
   private setDefaults(): void {
+    if (this.options.module === undefined) {
+      this.options.module = false;
+    } else if (this.options.module === true) {
+      this.options.module = 'ApiModule';
+    }
     if (!this.options.enumStyle) {
       this.options.enumStyle = 'alias';
     }

test/all-types.config.json

@@ -3,5 +3,7 @@
   "input": "all-types.json",
   "output": "out/all-types/",
   "indexFile": true,
-  "enumStyle": "pascal"
+  "enumStyle": "pascal",
+  "module": "AllTypesModule",
+  "services": true
 }

test/all-types.spec.ts

@@ -535,12 +535,11 @@ describe('Generation tests using all-types.json', () => {
     const parser = new TypescriptParser();
     parser.parseSource(ts).then(ast => {
       expect(ast.exports.length).toBe(5);
-      expect(ast.exports.some((ex: NamedExport) => ex.from === './api-configuration')).toBeDefined();
-      expect(ast.exports.some((ex: NamedExport) => ex.from === './base-service')).toBeDefined();
-      expect(ast.exports.some((ex: NamedExport) => ex.from === './request-builder')).toBeDefined();
-      expect(ast.exports.some((ex: NamedExport) => ex.from === './strict-http-response')).toBeDefined();
-      expect(ast.exports.some((ex: NamedExport) => ex.from === './api.module')).toBeDefined();
-
+      expect(ast.exports.find((ex: NamedExport) => ex.from === './api-configuration')).toBeDefined();
+      expect(ast.exports.find((ex: NamedExport) => ex.from === './base-service')).toBeDefined();
+      expect(ast.exports.find((ex: NamedExport) => ex.from === './request-builder')).toBeDefined();
+      expect(ast.exports.find((ex: NamedExport) => ex.from === './strict-http-response')).toBeDefined();
+      expect(ast.exports.find((ex: NamedExport) => ex.from === './all-types.module')).toBeDefined();
     });
   });
 });

test/noModule.spec.ts

@@ -18,12 +18,10 @@ describe('Generation tests with index and no ApiModule', () => {
     const ts = gen.templates.apply('index', ref);
     const parser = new TypescriptParser();
     parser.parseSource(ts).then(ast => {
-      expect(ast.exports.length).toBe(4);
+      expect(ast.exports.length).toBe(3);
       expect(ast.exports.some((ex: NamedExport) => ex.from === './api-configuration')).toBeDefined();
-      expect(ast.exports.some((ex: NamedExport) => ex.from === './base-service')).toBeDefined();
       expect(ast.exports.some((ex: NamedExport) => ex.from === './request-builder')).toBeDefined();
       expect(ast.exports.some((ex: NamedExport) => ex.from === './strict-http-response')).toBeDefined();
-
     });
   });