[UA] Add success and label (#2328)

see : flutter/flutter#181559 (comment)

<details>
  <summary>Contribution guidelines:</summary><br>

- See our [contributor guide](https://github.com/dart-lang/.github/blob/main/CONTRIBUTING.md) for general expectations for PRs.
- Larger or significant changes should be discussed in an issue before creating a PR.
- Contributions to our repos should follow the [Dart style guide](https://dart.dev/guides/language/effective-dart) and use `dart format`.
- Most changes should add an entry to the changelog and may need to [rev the pubspec package version](https://github.com/dart-lang/sdk/blob/main/docs/External-Package-Maintenance.md#making-a-change).
- Changes to packages require [corresponding tests](https://github.com/dart-lang/.github/blob/main/CONTRIBUTING.md#Testing).

Many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

**Note**: The Dart team is trialing Gemini Code Assist. Don't take its comments as final Dart team feedback. Use the suggestions if they're helpful; otherwise, wait for a human reviewer.

</details>

Author: AbdeMohlbi

pkgs/unified_analytics/CHANGELOG.md

@@ -1,5 +1,6 @@
-## 8.0.12-wip
+## 8.0.12
 - Require Dart 3.10
+- Added `success` indicator and `label` to `Event.flutterTrackAndroidDependencies` 
 
 ## 8.0.11
 - Added `Event.flutterTrackAndroidDependencies` to track android dependencies.

pkgs/unified_analytics/lib/src/constants.dart

@@ -87,7 +87,7 @@ const int kMaxLogFileSize = 25 * (1 << 20);
 const String kLogFileName = 'dart-flutter-telemetry.log';
 
 /// The current version of the package, should be in line with pubspec version.
-const String kPackageVersion = '8.0.12-wip';
+const String kPackageVersion = '8.0.12';
 
 /// The minimum length for a session.
 const int kSessionDurationMinutes = 30;

pkgs/unified_analytics/lib/src/event.dart

@@ -733,6 +733,10 @@ final class Event {
 
   /// Provides information about the android dependencies of a project.
   ///
+  /// * [success] - whether the build succeded.
+  ///
+  /// * [label] - the error label if the build fails.
+  ///
   /// * [isModule] - whether the project is an add-to-app Flutter module.
   ///
   /// * [agpVersion] - the Android Gradle plugin of the build target.
@@ -750,6 +754,8 @@ final class Event {
   ///
   /// * [gradleVersion] - the Gradle version used by the build target.
   Event.flutterTrackAndroidDependencies({
+    required bool success,
+    String? label,
     bool? isModule,
     String? agpVersion,
     int? minSDK,
@@ -761,6 +767,8 @@ final class Event {
   }) : this._(
          eventName: DashEvent.flutterTrackAndroidDependencies,
          eventData: {
+           'success': success,
+           'label': ?label,
            'isModule': ?isModule,
            'agpVersion': ?agpVersion,
            'minSDK': ?minSDK,

pkgs/unified_analytics/pubspec.yaml

@@ -5,7 +5,7 @@ description: >-
 # LINT.IfChange
 # When updating this, keep the version consistent with the changelog and the
 # value in lib/src/constants.dart.
-version: 8.0.12-wip
+version: 8.0.12
 # LINT.ThenChange(lib/src/constants.dart)
 repository: https://github.com/dart-lang/tools/tree/main/pkgs/unified_analytics
 issue_tracker: https://github.com/dart-lang/tools/issues?q=is%3Aissue+is%3Aopen+label%3Apackage%3Aunified_analytics

pkgs/unified_analytics/test/event_test.dart

@@ -539,6 +539,7 @@ void main() {
   group('Event.flutterTrackAndroidDependencies', () {
     test('constructed', () {
       Event generateEvent() => Event.flutterTrackAndroidDependencies(
+        success: true,
         isModule: true,
         agpVersion: '8.2.2',
         minSDK: 24,
@@ -556,6 +557,8 @@ void main() {
         constructedEvent.eventName,
         DashEvent.flutterTrackAndroidDependencies,
       );
+      expect(constructedEvent.eventData['success'], isTrue);
+      expect(constructedEvent.eventData['label'], isNull);
       expect(constructedEvent.eventData['isModule'], isTrue);
       expect(constructedEvent.eventData['agpVersion'], '8.2.2');
       expect(constructedEvent.eventData['minSDK'], 24);
@@ -564,11 +567,13 @@ void main() {
       expect(constructedEvent.eventData['jdkVersion'], 24);
       expect(constructedEvent.eventData['ndkVersion'], '28.0.13004108');
       expect(constructedEvent.eventData['gradleVersion'], '8.10.2');
-      expect(constructedEvent.eventData.length, 8);
+      expect(constructedEvent.eventData.length, 9);
     });
 
     test('constructor arguments default to null if not specified', () {
       Event generateEvent() => Event.flutterTrackAndroidDependencies(
+        success: false,
+        label: 'failed to download gradle from ...',
         isModule: true,
         agpVersion: '8.2.2',
         targetSDK: 36,
@@ -583,6 +588,11 @@ void main() {
         constructedEvent.eventName,
         DashEvent.flutterTrackAndroidDependencies,
       );
+      expect(constructedEvent.eventData['success'], isFalse);
+      expect(
+        constructedEvent.eventData['label'],
+        'failed to download gradle from ...',
+      );
       expect(constructedEvent.eventData['isModule'], isTrue);
       expect(constructedEvent.eventData['agpVersion'], '8.2.2');
       expect(constructedEvent.eventData['minSDK'], null);
@@ -591,7 +601,7 @@ void main() {
       expect(constructedEvent.eventData['jdkVersion'], 24);
       expect(constructedEvent.eventData['ndkVersion'], null);
       expect(constructedEvent.eventData['gradleVersion'], '8.10.2');
-      expect(constructedEvent.eventData.length, 5);
+      expect(constructedEvent.eventData.length, 7);
     });
   });
   test('Event.codeSizeAnalysis constructed', () {