Removed process.exit(0) from the generate module so the generate command can be passed through to plugins (#351)

* Removed process.exit(0) from generate() module the generate command can be passed through to plugins

* Updated unit test to reflect new logic

Author: Blackbaud-PaulCrowder

cli/generate.js

@@ -155,8 +155,6 @@ function generate(argv) {
         generateComponent(name);
         break;
     }
-
-    process.exit(0);
   } catch (err) {
     process.exit(1);
   }

test/cli-generate.spec.js

@@ -74,6 +74,10 @@ describe('cli generate', () => {
     });
 
     if (expectedExitCode === 0) {
+      // process.exit() should not be called on success so that other Builder
+      // plugins can tap into the generate command.
+      expect(process.exit).not.toHaveBeenCalled();
+
       expect(fsMock.ensureDirSync).toHaveBeenCalledWith(
         path.resolve('src', 'app', ...expectedPathParts)
       );
@@ -98,9 +102,9 @@ export class ${expectedClassName}`
 
       validateSpecFile(`describe('${expectedDescribe}', () => {`);
       validateSpecFile(`const fixture = TestBed.createComponent(${expectedClassName});`);
+    } else {
+      expect(process.exit).toHaveBeenCalledWith(expectedExitCode);
     }
-
-    expect(process.exit).toHaveBeenCalledWith(expectedExitCode);
   }
 
   afterEach(() => {