fix(core): new and migrate commands should exit with code (#10491)

e2e/workspace-create/src/create-nx-workspace.test.ts

@@ -10,9 +10,9 @@ import {
   readJson,
   runCLI,
   runCreateWorkspace,
-  updateJson,
-  updateFile,
   uniq,
+  updateFile,
+  updateJson,
 } from '@nrwl/e2e/utils';
 import { existsSync, mkdirSync } from 'fs-extra';
 import { execSync } from 'child_process';
@@ -93,9 +93,28 @@ describe('create-nx-workspace', () => {
     });
   });
 
+  it('should fail correctly when preset errors', () => {
+    // Using Angular Preset as the example here to test
+    // It will error when npmScope is of form `<char>-<num>-<char>`
+    // Due to a validation error Angular will throw.
+    const wsName = uniq('angular-1-test');
+    const appName = uniq('app');
+    try {
+      runCreateWorkspace(wsName, {
+        preset: 'angular',
+        style: 'css',
+        appName,
+        packageManager,
+      });
+    } catch (e) {
+      expect(e).toBeTruthy();
+    }
+  });
+
   it('should be able to create an react workspace', () => {
     const wsName = uniq('react');
     const appName = uniq('app');
+
     runCreateWorkspace(wsName, {
       preset: 'react',
       style: 'css',

packages/nx/src/command-line/nx-commands.ts

@@ -293,17 +293,21 @@ ${daemonHelpOutput}
     builder: (yargs) => withNewOptions(yargs),
     handler: async (args) => {
       args._ = args._.slice(1);
-      return (await import('./generate')).newWorkspace(
-        args['nxWorkspaceRoot'] as string,
-        args
+      process.exit(
+        await (
+          await import('./generate')
+        ).newWorkspace(args['nxWorkspaceRoot'] as string, args)
       );
     },
   })
   .command(
     '_migrate [packageAndVersion]',
     false,
     (yargs) => withMigrationOptions(yargs),
-    async (args) => (await import('./migrate')).migrate(process.cwd(), args)
+    async (args) =>
+      process.exit(
+        await (await import('./migrate')).migrate(process.cwd(), args)
+      )
   )
   .help('help')
   .version(nxVersion);