Fix a test issue.

iclanton · iclanton · commit 3654f1a68866 · 2019-07-15T21:59:59.000-07:00
diff --git a/apps/rush-lib/src/cli/actions/ChangeAction.ts b/apps/rush-lib/src/cli/actions/ChangeAction.ts
@@ -449,9 +449,11 @@ export class ChangeAction extends BaseRushAction {
   }
 
   private _getBumpOptions(packageName?: string): { [type: string]: string } {
-    let bumpOptions: { [type: string]: string } = this.rushConfiguration.hotfixChangeEnabled ? {
+    let bumpOptions: { [type: string]: string } = (this.rushConfiguration && this.rushConfiguration.hotfixChangeEnabled)
+    ? {
       [ChangeType[ChangeType.hotfix]]: 'hotfix - for changes that need to be published in a separate hotfix package'
-    } : {
+      }
+    : {
       [ChangeType[ChangeType.major]]: 'major - for changes that break compatibility, e.g. removing an API',
       [ChangeType[ChangeType.minor]]: 'minor - for backwards compatible changes, e.g. adding a new API',
       [ChangeType[ChangeType.patch]]: 'patch - for changes that do not affect compatibility, e.g. fixing a bug'
@@ -516,7 +518,7 @@ export class ChangeAction extends BaseRushAction {
           type: 'confirm',
           name: 'isCorrectEmail',
           default: 'Y',
-          message: `Is your email address ${email} ?`
+          message: `Is your email address ${email}?`
         }
       ]).then(({ isCorrectEmail }: { isCorrectEmail: boolean }) => {
         return isCorrectEmail ? email : undefined;
diff --git a/apps/rush-lib/src/cli/test/__snapshots__/CommandLineHelp.test.ts.snap b/apps/rush-lib/src/cli/test/__snapshots__/CommandLineHelp.test.ts.snap
@@ -141,7 +141,9 @@ Optional arguments:
 `;
 
 exports[`CommandLineHelp prints the help for each action: change 1`] = `
-"usage: rush change [-h] [-v] [--no-fetch] [-b BRANCH]
+"usage: rush change [-h] [-v] [--no-fetch] [-b BRANCH] [--email EMAIL] [--bulk]
+                   [--message MESSAGE] [--bump-type {major,minor,patch}]
+                   
 
 Asks a series of questions and then generates a <branchname>-<timestamp>.json 
 file in the common folder. The \`publish\` command will consume these files and 
@@ -172,6 +174,18 @@ Optional arguments:
                         which projects were changed. If this parameter is not 
                         specified, the checked out branch is compared against 
                         the \\"master\\" branch.
+  --email EMAIL         The email address to use in changefiles. If this 
+                        parameter is not provided, the email address will be 
+                        detected or prompted for in intractive mode.
+  --bulk                If this flag is specified, apply the same change 
+                        message and bump type to all changed projects. The 
+                        --message and the --bump-type parameters must be 
+                        specified if the --bulk parameter is specified
+  --message MESSAGE     The message to apply to all changed projects if the 
+                        --bulk flag is provided.
+  --bump-type {major,minor,patch}
+                        The bump type to apply to all changed projects if the 
+                        --bulk flag is provided.
 "
 `;
 
