mirror of
https://github.com/flutter/flutter.git
synced 2025-06-03 00:51:18 +00:00
Update helper message for --suppress-analytics
(#124810)
Fixes: https://github.com/flutter/flutter/issues/124808
This commit is contained in:
parent
59230759be
commit
3476b96652
@ -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 {
|
|||||||
// 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 {
|
|||||||
'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> {
|
|||||||
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() {
|
|||||||
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,
|
|
||||||
},
|
|
||||||
);
|
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user