fix(@angular/cli): skip project analytics prompt when using update

The prompt will cause the workspace configuration file to be updated which can result in an unclean repository.

Fixes #16012

Author: clydin

packages/angular/cli/models/command-runner.ts

@@ -64,14 +64,19 @@ export interface CommandMapOptions {
  * Create the analytics instance.
  * @private
  */
-async function _createAnalytics(workspace: boolean): Promise<analytics.Analytics> {
+async function _createAnalytics(workspace: boolean, skipPrompt = false): Promise<analytics.Analytics> {
   let config = await getGlobalAnalytics();
   // If in workspace and global analytics is enabled, defer to workspace level
   if (workspace && config) {
+    const skipAnalytics =
+      skipPrompt ||
+      (process.env['NG_CLI_ANALYTICS'] &&
+        (process.env['NG_CLI_ANALYTICS'].toLowerCase() === 'false' ||
+          process.env['NG_CLI_ANALYTICS'] === '0'));
     // TODO: This should honor the `no-interactive` option.
     //       It is currently not an `ng` option but rather only an option for specific commands.
     //       The concept of `ng`-wide options are needed to cleanly handle this.
-    if (!(await hasWorkspaceAnalyticsConfiguration())) {
+    if (!skipAnalytics && !(await hasWorkspaceAnalyticsConfiguration())) {
       await promptProjectAnalytics();
     }
     config = await getWorkspaceAnalytics();
@@ -231,7 +236,9 @@ export async function runCommand(
       return map;
     });
 
-    const analytics = options.analytics || await _createAnalytics(!!workspace.configFile);
+    const analytics =
+      options.analytics ||
+      (await _createAnalytics(!!workspace.configFile, description.name === 'update'));
     const context = { workspace, analytics };
     const command = new description.impl(context, description, logger);
 

tests/legacy-cli/e2e/tests/misc/ask-analytics-command.ts

@@ -0,0 +1,51 @@
+import { execWithEnv, killAllProcesses, waitForAnyProcessOutputToMatch } from '../../utils/process';
+import { expectToFail } from '../../utils/utils';
+
+export default async function() {
+  try {
+    // Execute a command with TTY force enabled
+    const execution = execWithEnv('ng', ['version'], {
+      ...process.env,
+      NG_FORCE_TTY: '1',
+      NG_CLI_ANALYTICS: 'ci',
+    });
+
+    // Check if the prompt is shown
+    await waitForAnyProcessOutputToMatch(/Would you like to share anonymous usage data/);
+  } finally {
+    killAllProcesses();
+  }
+
+  try {
+    // Execute a command with TTY force enabled
+    const execution = execWithEnv('ng', ['version'], {
+      ...process.env,
+      NG_FORCE_TTY: '1',
+      NG_CLI_ANALYTICS: 'false',
+    });
+
+    // Check if the prompt is shown
+    await expectToFail(() =>
+      waitForAnyProcessOutputToMatch(/Would you like to share anonymous usage data/, 5),
+    );
+  } finally {
+    killAllProcesses();
+  }
+
+  // Should not show a prompt when using update
+  try {
+    // Execute a command with TTY force enabled
+    const execution = execWithEnv('ng', ['update'], {
+      ...process.env,
+      NG_FORCE_TTY: '1',
+      NG_CLI_ANALYTICS: 'ci',
+    });
+
+    // Check if the prompt is shown
+    await expectToFail(() =>
+      waitForAnyProcessOutputToMatch(/Would you like to share anonymous usage data/, 5),
+    );
+  } finally {
+    killAllProcesses();
+  }
+}