Unverified Commit ca3a900d authored by Elias Yishak's avatar Elias Yishak Committed by GitHub

[reland] Update helper message for `--suppress-analytics` (#125145)

Relanding: https://github.com/flutter/flutter/pull/124810

Fixes: https://github.com/flutter/flutter/issues/124808
parent 4c6929d4
...@@ -88,14 +88,6 @@ Future<int> run( ...@@ -88,14 +88,6 @@ Future<int> run(
await globals.analytics.setTelemetry(value); await globals.analytics.setTelemetry(value);
} }
// If the user has opted out of legacy analytics, we will continue
// to opt them out of unified analytics and inform them
if (!globals.flutterUsage.enabled && globals.analytics.telemetryEnabled) {
await globals.analytics.setTelemetry(false);
globals.logger.printStatus(
'Please note that analytics reporting was already disabled, and will continue to be disabled.\n');
}
await runner.run(args); await runner.run(args);
// Triggering [runZoned]'s error callback does not necessarily mean that // Triggering [runZoned]'s error callback does not necessarily mean that
...@@ -285,6 +277,11 @@ Future<int> _exit(int code, {required ShutdownHooks shutdownHooks}) async { ...@@ -285,6 +277,11 @@ Future<int> _exit(int code, {required ShutdownHooks shutdownHooks}) async {
// Ensure that the consent message has been displayed for unified analytics // Ensure that the consent message has been displayed for unified analytics
if (globals.analytics.shouldShowMessage) { if (globals.analytics.shouldShowMessage) {
globals.logger.printStatus(globals.analytics.getConsentMessage); globals.logger.printStatus(globals.analytics.getConsentMessage);
if (!globals.flutterUsage.enabled) {
globals.printStatus(
'Please note that analytics reporting was already disabled, '
'and will continue to be disabled.\n');
}
// Because the legacy analytics may have also sent a message, // Because the legacy analytics may have also sent a message,
// the conditional below will print additional messaging informing // the conditional below will print additional messaging informing
...@@ -296,8 +293,7 @@ Future<int> _exit(int code, {required ShutdownHooks shutdownHooks}) async { ...@@ -296,8 +293,7 @@ Future<int> _exit(int code, {required ShutdownHooks shutdownHooks}) async {
'the flutter tool is migrating to a new analytics system. ' 'the flutter tool is migrating to a new analytics system. '
'Disabling analytics collection will disable both the legacy ' 'Disabling analytics collection will disable both the legacy '
'and new analytics collection systems. ' 'and new analytics collection systems. '
'You can disable analytics reporting by running either `flutter --disable-telemetry` ' 'You can disable analytics reporting by running `flutter --disable-telemetry`\n');
'or `flutter config --no-analytics\n');
} }
// Invoking this will onboard the flutter tool onto // Invoking this will onboard the flutter tool onto
......
...@@ -77,7 +77,7 @@ class FlutterCommandRunner extends CommandRunner<void> { ...@@ -77,7 +77,7 @@ class FlutterCommandRunner extends CommandRunner<void> {
help: 'Allow Flutter to check for updates when this command runs.'); help: 'Allow Flutter to check for updates when this command runs.');
argParser.addFlag('suppress-analytics', argParser.addFlag('suppress-analytics',
negatable: false, negatable: false,
help: 'Suppress analytics reporting when this command runs.'); help: 'Suppress analytics reporting for the current CLI invocation.');
argParser.addFlag('disable-telemetry', argParser.addFlag('disable-telemetry',
negatable: false, negatable: false,
help: 'Disable telemetry reporting when this command runs.'); help: 'Disable telemetry reporting when this command runs.');
......
...@@ -346,34 +346,6 @@ void main() { ...@@ -346,34 +346,6 @@ void main() {
ProcessManager: () => FakeProcessManager.any(), ProcessManager: () => FakeProcessManager.any(),
}, },
); );
testUsingContext(
'legacy analytics disabled will disable new analytics',
() async {
io.setExitFunctionForTests((int exitCode) {});
await runner.run(
<String>[],
() => <FlutterCommand>[],
// This flutterVersion disables crash reporting.
flutterVersion: '[user-branch]/',
shutdownHooks: ShutdownHooks(),
);
expect(globals.flutterUsage.enabled, false);
expect(globals.analytics.telemetryEnabled, false);
expect(testLogger.statusText.contains(
'Please note that analytics '
'reporting was already disabled'), true);
},
overrides: <Type, Generator>{
Analytics: () => FakeAnalytics(),
FileSystem: () => MemoryFileSystem.test(),
ProcessManager: () => FakeProcessManager.any(),
Usage: () => legacyAnalytics,
},
);
}); });
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment