New enum + constructor added for nullSafetyAnalysisResult

pkgs/unified_analytics/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 5.5.0
+
+- Added the `Event.nullSafetyAnalysisResult` constructor
+
 ## 5.4.0
 
 - Added the `Event.codeSizeAnalysis` constructor

pkgs/unified_analytics/lib/src/constants.dart

@@ -82,7 +82,7 @@ const int kLogFileLength = 2500;
 const String kLogFileName = 'dart-flutter-telemetry.log';
 
 /// The current version of the package, should be in line with pubspec version.
-const String kPackageVersion = '5.4.0';
+const String kPackageVersion = '5.5.0';
 
 /// The minimum length for a session.
 const int kSessionDurationMinutes = 30;

pkgs/unified_analytics/lib/src/enums.dart

@@ -76,6 +76,11 @@ enum DashEvent {
     description: 'Information related to the Flutter hot runner',
     toolOwner: DashTool.flutterTool,
   ),
+  nullSafetyAnalysisResult(
+    label: 'null_safety_analysis_results',
+    description: 'Information related to null safety analysis within flutter',
+    toolOwner: DashTool.flutterTool,
+  ),
 
   // Events for language_server below
 

pkgs/unified_analytics/lib/src/event.dart

@@ -43,6 +43,30 @@ final class Event {
           'method': method,
         };
 
+  /// Contains information about null safety migration within a flutter project.
+  ///
+  /// [runtimeMode] - The null safety runtime mode the app should be built in.
+  ///
+  /// [nullSafeMigratedLibraries] - count of libraries with language versions
+  ///   that are greater than 2.12 to indicate migration to null safety.
+  ///
+  /// [nullSafeTotalLibraries] - the total number of packages in the package
+  ///   config.
+  ///
+  /// [languageVersion] - the dart language version for the package.
+  Event.nullSafetyAnalysisResult({
+    required String runtimeMode,
+    required int nullSafeMigratedLibraries,
+    required int nullSafeTotalLibraries,
+    String? languageVersion,
+  })  : eventName = DashEvent.nullSafetyAnalysisResult,
+        eventData = {
+          'runtimeMode': runtimeMode,
+          'nullSafeMigratedLibraries': nullSafeMigratedLibraries,
+          'nullSafeTotalLibraries': nullSafeTotalLibraries,
+          if (languageVersion != null) 'languageVersion': languageVersion,
+        };
+
   /// Event that is emitted periodically to report the performance of the
   /// analysis server's handling of a specific kind of request from the client.
   ///

pkgs/unified_analytics/pubspec.yaml

@@ -4,7 +4,7 @@ description: >-
   to Google Analytics.
 # When updating this, keep the version consistent with the changelog and the
 # value in lib/src/constants.dart.
-version: 5.4.0
+version: 5.5.0
 repository: https://github.com/dart-lang/tools/tree/main/pkgs/unified_analytics
 
 environment:

pkgs/unified_analytics/test/event_test.dart

@@ -409,6 +409,24 @@ void main() {
     expect(constructedEvent.eventData.length, 1);
   });
 
+  test('Event.nullSafetyAnalysisResult constructed', () {
+    Event generateEvent() => Event.nullSafetyAnalysisResult(
+        runtimeMode: 'runtimeMode',
+        nullSafeMigratedLibraries: 4,
+        nullSafeTotalLibraries: 5,
+        languageVersion: 'languageVersion');
+
+    final constructedEvent = generateEvent();
+
+    expect(generateEvent, returnsNormally);
+    expect(constructedEvent.eventName, DashEvent.nullSafetyAnalysisResult);
+    expect(constructedEvent.eventData['runtimeMode'], 'runtimeMode');
+    expect(constructedEvent.eventData['nullSafeMigratedLibraries'], 4);
+    expect(constructedEvent.eventData['nullSafeTotalLibraries'], 5);
+    expect(constructedEvent.eventData['languageVersion'], 'languageVersion');
+    expect(constructedEvent.eventData.length, 4);
+  });
+
   test('Confirm all constructors were checked', () {
     var constructorCount = 0;
     for (var declaration in reflectClass(Event).declarations.keys) {
@@ -417,7 +435,7 @@ void main() {
 
     // Change this integer below if your PR either adds or removes
     // an Event constructor
-    final eventsAccountedForInTests = 21;
+    final eventsAccountedForInTests = 22;
     expect(eventsAccountedForInTests, constructorCount,
         reason: 'If you added or removed an event constructor, '
             'ensure you have updated '